Bug 106210 - [plan item][nls tooling] Improve NLS tooling
Summary: [plan item][nls tooling] Improve NLS tooling
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.1   Edit
Hardware: All All
: P4 enhancement with 1 vote (vote)
Target Milestone: 3.2 M6   Edit
Assignee: Benno Baumgartner CLA
QA Contact:
URL:
Whiteboard:
Keywords: plan
Depends on:
Blocks:
 
Reported: 2005-08-05 12:53 EDT by Mike Wilson CLA
Modified: 2006-03-06 11:27 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Wilson CLA 2005-08-05 12:53:15 EDT
The Eclipse NLS tooling should better support the the new Eclipse string externalization pattern added in 
3.1, along with ways to help developers with unused keys and unused property file entries. [JDT UI]
Comment 1 Dani Megert CLA 2005-08-08 06:45:57 EDT
This item is JDT Text and not JDT UI.
Comment 2 Dani Megert CLA 2005-08-08 06:46:19 EDT
Mike, can you please update the item on the 3.2 plan? Thanks.
Comment 3 Mike Wilson CLA 2005-08-08 10:51:53 EDT
Jeem, said he would change it for me.
Comment 4 Max Rotvel CLA 2005-09-13 08:11:18 EDT
Here's some things that needs fixed in the Externalize Strings wizard: 

1) Currently when you add or edit keys the changes are reflected in the 
specified properties file (say 'strings.properties'). But if you have more files 
in your bundle ('strings_dk.properties', 'strings_it.properties' etc) you have 
to edit them manually. So it should be possible to specify more than one 
property file (perhaps using wildcards?) and/or the wizard should (optionally?) 
try to find all property files based on the root part (in this case 'strings') 
and display them in the 'changes to be performed' list.

2) It's cumbersome to remove keys. First you need to start the wizard and 
internalize the key (and, as above, this only works on one property file) and 
then you can remove the string from your code. The wizard should detect keys 
that are no longer in use and offer to remove them from the accessor class and 
the relevant property file(s).

3) And finally, when you have more than one source file externalized in one set 
of accessor class/property file(s), _please_ consider making it possible to open 
 all of these source files at the same time in the wizard. It's a pain having to 
 open wizard for source file 1, edit keys, perform changes, open wizard for file 
 2 etc...

If any of these points needs a seperate bug#, feel free to move them or ask me 
and I'll do it.

Regards /Max

 
Comment 5 Dani Megert CLA 2005-09-13 08:21:55 EDT
Please file a separate PRs. 1) and 3) are enhancement requests which will
most-likely not be addressed for 3.2 unless there's a good quality contribution.
2) looks like a bug.
Comment 6 Mike Wilson CLA 2006-02-14 15:26:11 EST
The text of this item in the plan has changed. The new text is:
"
The Eclipse NLS tooling should better support the new Eclipse string externalization pattern added in 3.1. [JDT Text]
"
Comment 7 Dani Megert CLA 2006-02-28 08:48:36 EST
Benno, drop by for more info.
Comment 8 Benno Baumgartner CLA 2006-03-06 11:27:02 EST
fixed > I20060301-0800

NLSStringHover: Show the value of the nls constant when hovering over a reference to an NLS constant.
NLSKeyHyperlink: Jump directly to the definition of nls constant in the property file when clicking on the link of an NLS constant reference.
PropertyKeyHyperlink: Jump to users of an nls value when clicking on the link in the properties file (or show a dialog if multiple users)
NLSSearchResultRequestor: Search for and show 
      +Unusued property keys 
      +Referenced but undefined property keys

All this is supposed to work and behave the same with the traditional nls style and the eclipse nls style.

Max, 1) and 3) are enhancement requests, as Dani said, file separate bugs. I think also 2) is an enhancement, or did the dialog ever detect unused keys? You can use the NLS search to find unused/undefined nls keys.