Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 347064 - NPE in JavaScriptDebugHover
Summary: NPE in JavaScriptDebugHover
Status: RESOLVED FIXED
Alias: None
Product: JSDT
Classification: WebTools
Component: Debug (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.4   Edit
Assignee: Michael Rennie CLA
QA Contact: Michael Rennie CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-05-24 15:18 EDT by Michael Rennie CLA
Modified: 2011-07-07 12:53 EDT (History)
1 user (show)

See Also:

Attachments
fix (1.14 KB, patch)
2011-05-24 15:18 EDT, Michael Rennie 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 Michael Rennie CLA 2011-05-24 15:18:27 EDT 
Created attachment 196478 [details]
fix

java.lang.NullPointerException
	at org.eclipse.wst.jsdt.debug.internal.ui.eval.JavaScriptDebugHover.getHoverInfo2(JavaScriptDebugHover.java:78)
	at org.eclipse.wst.jsdt.internal.ui.text.java.hover.BestMatchHover.getHoverInfo2(BestMatchHover.java:147)
	at org.eclipse.wst.jsdt.internal.ui.text.java.hover.JavaEditorTextHoverProxy.getHoverInfo2(JavaEditorTextHoverProxy.java:78)
	at org.eclipse.jface.text.TextViewerHoverManager$4.run(TextViewerHoverManager.java:166)

The problem is coming from JavaScriptValue#getVariables(), the current implementation is not following the API, i.e. it should not be returning null.
Comment 1 Michael Rennie CLA 2011-07-07 12:53:52 EDT 
applied patch to HEAD