Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 359940 - [nls tooling] NLS hover shown in place of Java hover
Summary: [nls tooling] NLS hover shown in place of Java hover
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.8 M3   Edit
Assignee: Dani Megert CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-04 23:20 EDT by Deepak Azad CLA
Modified: 2011-10-26 04:06 EDT (History)
1 user (show)

See Also:

Attachments
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 Deepak Azad CLA 2011-10-04 23:20:53 EDT 
Hover over any reference of a public string field of org.eclipse.jdt.internal.corext.refactoring.nls.NLSRefactoring class. 

=> The hover says - "The properties file could not be detected", which is not right.
Comment 1 Deepak Azad CLA 2011-10-04 23:53:54 EDT 
Ha! 

NLSRefactoring has a field called BUNDLE_NAME, and for finding the accessor class  we look for hard-coded fields bundle name String field names - "BUNDLE_NAME", "RESOURCE_BUNDLE", "bundleName" (see NLSHintHelper line 315)

I suppose the only thing that can be done is to rename NLSRefactoring.BUNDLE_NAME e.g. to NLSRefactoring.BUNDLE_NAME_FIELD
Comment 2 Dani Megert CLA 2011-10-05 08:08:53 EDT 
Mixed in master: de2edee12b0bcf250a1bd184bd7daaf6e13e0172
Comment 3 Dani Megert CLA 2011-10-26 04:06:03 EDT 
Verified in I20111025-1800.