This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 151141

Summary: Code Completion with Default Parameters
Product: php Reporter: compucoder <compucoder>
Component: EditorAssignee: Tomasz Slota <tslota>
Status: RESOLVED DUPLICATE    
Severity: blocker    
Priority: P3    
Version: 6.x   
Hardware: All   
OS: All   
Issue Type: DEFECT Exception Reporter:

Description compucoder 2008-10-23 13:31:25 UTC
Added as per request of Petr Pisl:

I think the code completion with respect to methods and functions with default paramaters may need to be looked at.

For example, if you have 1 function with 10 parameters, 1 is mandatory and 9 have default values, your CC list actually
shows 10 functions with each working its way up to the 10 parameters. I have never seen this done like this on any IDE
ever. Wouldn't it just be better to show the function once and in the CC list show the parameters with their defaults? I
think you need to show the defaults regardless as you should display that to the user.

I know you have that red box that pre-fills in the parameter names so you can easily edit these and I think you can
still do that - just do it for the parameters without defaults. To be honest though, I don't think you should even be
doing that for functions / methods at all. I can see it for templates. Why not just show CC parameter lists for the
functions and when they put a comma then adjust the CC list accordingly? I think that falls more into the style of what
people expect.

Visual Studio would be a good basis to mimic for this I think. it does a great job showing CC lists for helping fill out
functions and methods.

Thanks.
Comment 1 Tomasz Slota 2008-10-23 13:53:01 UTC
This was the easiest way to implement some sort of support for optional args. The concept should be familiar for most developers and I was hoping most of 
them would find it helpful. This issue is very similar to issue 139842, they should be tracked together


*** This issue has been marked as a duplicate of 139842 ***