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 235439 - Broken file parsing after using prototype's property
Summary: Broken file parsing after using prototype's property
Status: VERIFIED FIXED
Alias: None
Product: javascript
Classification: Unclassified
Component: Editor (show other bugs)
Version: 7.4
Hardware: PC Linux
: P2 normal (vote)
Assignee: Petr Pisl
URL:
Keywords: 7.4_HR_FIX
Depends on:
Blocks:
 
Reported: 2013-09-03 08:12 UTC by Vladimir Riha
Modified: 2013-09-10 15:25 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
IDE log (50.57 KB, text/plain)
2013-09-03 08:12 UTC, Vladimir Riha 		Details
sample file (1.07 KB, text/plain)
2013-09-03 08:12 UTC, Vladimir Riha 		Details
screenshot (113.85 KB, image/png)
2013-09-03 08:13 UTC, Vladimir Riha 		Details
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 Vladimir Riha 2013-09-03 08:12:09 UTC 
Created attachment 139606 [details]
IDE log

Please check the attached JS file. The iteration over ChartP3.prototype.milestones on line 5 causes that function ChartP3 declared on line 13 is:
 - not displayed in Navigator at all, Navigator shows only WidgetMilestones
 - missing "semantic" coloring (e.g. name of functions inside and ChartP3 as such is not bold)
 - not recognized in code completion, for instance

var a = new ChartP3();
a.

is not offering predictionLinearData, codeFreezeQC etc.

 - usage of ChartP3 in code shows "Global variable ChartP3 is not declared" (e.g. line 5) and it has purple color instead of black


I'm making it P2 as I think it would be good to be fixed for 7.4.




Product Version: NetBeans IDE Dev (Build 201309020001)
Java: 1.7.0_40; Java HotSpot(TM) Client VM 24.0-b55
Runtime: Java(TM) SE Runtime Environment 1.7.0_40-b39
System: Linux version 3.2.0-48-generic-pae running on i386; UTF-8; en_US (n
Comment 1 Vladimir Riha 2013-09-03 08:12:34 UTC 
Created attachment 139607 [details]
sample file
Comment 2 Vladimir Riha 2013-09-03 08:13:16 UTC 
Created attachment 139608 [details]
screenshot
Comment 3 Petr Pisl 2013-09-03 11:24:25 UTC 
I have the commit on my disk. It will be pushed, when the trunk will be open for commits.
Comment 4 Petr Pisl 2013-09-03 12:32:45 UTC 
Fixed in web-main: http://hg.netbeans.org/web-main/rev/fcd6cb67a0dc

Vlado, could you please verify it? Thanks
Comment 5 Vladimir Riha 2013-09-03 12:36:08 UTC 
Of course, I'll wait for trunk build (tomorrow?). Thanks for quick fix
Comment 6 Petr Pisl 2013-09-03 12:52:25 UTC 
Petr H. Could you please review the fix? Thanks
Comment 7 Petr Hejl 2013-09-03 13:02:06 UTC 
As far as I can tell the patch is ok. Please also check with FindBugs.
Comment 8 Vladimir Riha 2013-09-04 07:51:24 UTC 
Verified in trunk, please continue with integration to release74 as described
in [1]. Thank you


[1] http://wiki.netbeans.org/NetBeans_74_HighResistance



Product Version: NetBeans IDE Dev (Build 201309040001)
Java: 1.7.0_40; Java HotSpot(TM) Client VM 24.0-b55
Runtime: Java(TM) SE Runtime Environment 1.7.0_40-b39
System: Linux version 3.2.0-48-generic-pae running on i386; UTF-8; en_US (nb)
Comment 9 Petr Pisl 2013-09-05 08:48:40 UTC 
Transplanted to the release74 http://hg.netbeans.org/releases/rev/4702c39f7e7e
Comment 10 Vladimir Riha 2013-09-10 15:25:02 UTC 
Verified in RC1

Product Version: NetBeans IDE 7.4 RC1 (Build 201309092300)
Java: 1.7.0_40; Java HotSpot(TM) Client VM 24.0-b56
Runtime: Java(TM) SE Runtime Environment 1.7.0_40-b43
System: Linux version 3.2.0-48-generic-pae running on i386; UTF-8; en_US (nb)