Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 339600 - [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
Summary: [content assist] Function does not show up in content assist if it has a para...
Status: RESOLVED WORKSFORME
Alias: None
Product: JSDT
Classification: WebTools
Component: Web (show other bugs)
Version: 3.2.3   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: Future   Edit
Assignee: Chris Jaun CLA
QA Contact: Chris Jaun CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-10 16:35 EST by Chris Jaun CLA
Modified: 2013-07-10 11:45 EDT (History)
1 user (show)

See Also:

Attachments
patch (1.48 KB, patch)
2011-03-15 17:17 EDT, Chris Jaun CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.