Bug 241552

Summary:

JavaScript function arguments should be displayed in the HTML editor

Product:

[WebTools] JSDT

Reporter:

Alon Peled <alon>

Component:

General

Assignee:

Chris Jaun <cmjaun>

Status:

RESOLVED FIXED

QA Contact:

Nitin Dahyabhai <thatnitind>

Severity:

enhancement

Priority:

CC:

childsb, cmjaun

Version:

3.0

Keywords:

contributed

Target Milestone:

3.0.5

Flags:

thatnitind: review+

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
Content assist in web pages now obeys the preference for adding method params	thatnitind: iplog+

Description Alon Peled

2008-07-21 11:28:40 EDT

When using the code assist, all the functions from included JS file have no arguments displayed.
Once you select an entry, the required parameters are missing and you have no indication that the code entered is invalid.

Comment 1 Alon Peled

2008-07-31 04:39:28 EDT

The common practice for using JavaScript in HTML is based on includes - I would like to see this bug getting higher priority.

Comment 2 Bradley Childs

2008-11-10 17:02:31 EST

hi alon,

this is working as designed.  the same function is found inside standalone JS editor (no arguments from other source files only function name)

i'm marking this as enhancement  but i agree it's something we should look at fixing for standalone and HTML/embedded editor.

-brad

Comment 3 Alon Peled

2008-11-11 02:13:22 EST

AFAIK there is no include inside a JS file, only in HTML files. Therefore it is irrelevant for the JS editor.
I am afraid that an enhancement with P3 priority will never be looked at again.

Comment 4 Bradley Childs

2008-11-11 12:03:19 EST

the similar situation in standalone javascript is call/use of functions in javascript source files within the same project and marked as source folder or file.

i'll give this higher priority.

Comment 5 Chris Jaun

2009-03-23 16:30:53 EDT

Created attachment 129638 [details]
Content assist in web pages now obeys the preference for adding method params

Comment 6 Chris Jaun

2009-03-23 16:34:07 EDT

The content assist collector for web pages never checked the preference for whether to add the method params or not, so it was creating the wrong type of collector object.

Had to change a few things around to get this to work, but it seems to be in good shape now.

Comment 7 Chris Jaun

2009-03-23 16:42:48 EDT

This fix can go into the 3.0.5 and 3.1 streams.

Comment 8 Nitin Dahyabhai

2009-04-13 22:29:52 EDT

Released to those streams.  Test well.