Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 339600

Summary: [content assist] Function does not show up in content assist if it has a parameter and it is not the first thing in the script tag
Product: [WebTools] JSDT Reporter: Chris Jaun <cmjaun>
Component: WebAssignee: Chris Jaun <cmjaun>
Status: RESOLVED WORKSFORME QA Contact: Chris Jaun <cmjaun>
Severity: normal    
Priority: P3 CC: jpadilla
Version: 3.2.3   
Target Milestone: Future   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
patch none

Description Chris Jaun CLA 2011-03-10 16:35:19 EST
1. Create a web project.
2. Create a JS file and a HTML file in webcontent.
3. The JS file....

function doSomethingNeat() {
}

function doSomethingAwesome(a) {
}

4. Add a script region to the HTML file.
5. Type doS and attempt content assist. You should see both functions as proposals. Insert either one.
6. Attempt content assist again...this time you will only see a proposal for doSomethingNeat.

The function with the parameter does not show up.
Comment 1 Chris Jaun CLA 2011-03-15 17:17:09 EDT
Created attachment 191253 [details]
patch

This patch fixes the specific problem described by this defect, but I found some other issues.

1. With this patch applied the parameter name shows up as arg1 instead of the actual name.
2. When I am on the first line of the script region and invoke content assist, both method proposals are made, but the one with a parameter does not display the parameter in the proposal.
Comment 2 Nitin Dahyabhai CLA 2011-04-27 22:30:53 EDT
Chris, can you clone this so we can fix it in 3.3 RC1 separately?
Comment 3 Jorge Padilla CLA 2013-06-10 10:03:15 EDT
Hi Chris, I couldn't recreate this problem in the current driver.  I took a look at the MASTER branch in Git and it looks the problem has been fixed in a very similar way to the attached patch.  Maybe this was fixed with a duplicate bug?  I think this bug can be closed as no longer valid.
Comment 4 Chris Jaun CLA 2013-06-10 10:58:00 EDT
Thanks, Jorge. I will confirm your findings and close this bug.
Comment 5 Chris Jaun CLA 2013-07-10 11:45:43 EDT
Confirmed that this is working now in WTP 3.5. Resolving.