Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 335146 - web_ui.properties changed without proper approval
Summary: web_ui.properties changed without proper approval
Status: VERIFIED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.servlet (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P2 major (vote)
Target Milestone: 3.2.3   Edit
Assignee: Roberto Sanchez Herrera CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard: PMC_approved
Keywords:
Depends on: 327802
Blocks:
  Show dependency tree
 
Reported: 2011-01-24 00:04 EST by Masayuki Fuse CLA
Modified: 2011-02-02 05:14 EST (History)
4 users (show)

See Also:
david_williams: pmc_approved+
raghunathan.srinivasan: pmc_approved+
ccc: pmc_approved? (naci.dai)
ccc: pmc_approved? (deboer)
ccc: pmc_approved? (neil.hauge)
ccc: pmc_approved? (kaloyan)
cbridgha: review+


Attachments
screen shot (14.01 KB, image/png)
2011-01-24 00:07 EST, Masayuki Fuse CLA
no flags Details
Restore the content of org.eclipse.jst.servlet.ui\web_ui.properties (1.94 KB, patch)
2011-01-24 12:50 EST, Roberto Sanchez Herrera CLA
ccc: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Masayuki Fuse CLA 2011-01-24 00:04:20 EST
Build Identifier: 3.2.2.v201101121727 + NPpack1-wtp-sdk-noop-R-3.2.2-2010091517344

Japanese folder label for "Java Resources" contains {0}

Reproducible: Always

Steps to Reproduce:
1. Create a Dynamic Web Project
2. Expand the project on the Project Explore view
3.
Comment 1 Masayuki Fuse CLA 2011-01-24 00:07:40 EST
Created attachment 187395 [details]
screen shot
Comment 2 Carl Anderson CLA 2011-01-24 09:45:04 EST
Fuse-san,  WTP does not provide translations for its content.  You will need to figure out where you are getting your translation from, and contact the appropriate provider.
Comment 3 Carl Anderson CLA 2011-01-24 11:25:12 EST
This is bad.

After further investigation (with Kit Lo's help), I discovered that bug 327802 changed the content of org.eclipse.jst.servlet.ui\web_ui.properties without proper PMC approval.  This causes breakages in the UI of adopters.  We need to "undo" the UI part of that change for the R3_2_maintenance stream.  I am updating the description to properly reflect the problem.
Comment 4 Roberto Sanchez Herrera CLA 2011-01-24 12:50:36 EST
Created attachment 187450 [details]
Restore the content of org.eclipse.jst.servlet.ui\web_ui.properties
Comment 5 Chuck Bridgham CLA 2011-01-25 14:23:06 EST
approved
Comment 6 Carl Anderson CLA 2011-01-25 20:58:34 EST
Please note that, while a properties file is being changed, there is no UI/NLS changes between WTP 3.2.2 and the properties file that will be released in WTP 3.2.3 if this change is approved.  If the change is not approved, we will need to seek PMC approval for the NLS change that was previously made.

    * Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such. 
This was a UI breakage that needs to be reverted, as it breaks translations of WTP by adopters (including IBM).

    * Is there a work-around? If so, why do you believe the work-around is insufficient?
The work-around would be for all translations of WTP to change their properties files.  That is not an acceptable option.

    * How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added?
The UI was tested by hand to verify that the correct string is displayed.

    * Give a brief technical overview. Who has reviewed this fix?
Roberto reverted the change made by another developer- the properties file is now the same as it was in WTP 3.2.0-3.2.2, and the proper, empty, string is now substituted.  Chuck Bridgham and Carl Anderson have reviewed this fix.

    * What is the risk associated with this fix?
Extremely low
Comment 7 David Williams CLA 2011-01-25 21:25:33 EST
thanks for catching (and fixing) this maintenance "break".
Comment 8 Carl Anderson CLA 2011-01-25 22:11:58 EST
Committed to R3_2_maintenance for WTP 3.2.3.

Note that this will *NOT* be committed to HEAD, since it is perfectly acceptable to change a properties file like this before WTP 3.3 M6.
Comment 9 David Williams CLA 2011-01-25 22:20:45 EST
(In reply to comment #8)
> ... since it is perfectly
> acceptable to change a properties file like this before WTP 3.3 M6.

Yes, it is. But still ... sounds like a pretty "breaking change" if a translator doesn't notice or translated files not udpated? Its not like a word to end user is wrong ... but actually creates wrong folders? Are translations such as "babel" controlled so that "new version of translations" for every new release? Perhaps so, but perhaps deserves mention in our "migration guide"? (and I really do not know if this deserves so much attention ... just asking).
Comment 10 Kit Lo CLA 2011-01-25 23:06:26 EST
>Are translations such as "babel" controlled so that "new version of translations" for every new release? Perhaps so, but perhaps deserves mention in our "migration guide"?

It's normal and acceptable to modify the UI strings between releases. Babel does extract all strings for every new release. I don't think this needs to be mentioned in the migration guide.
Comment 11 David Williams CLA 2011-01-26 08:15:44 EST
(In reply to comment #10)
> >Are translations such as "babel" controlled so that "new version of translations" for every new release? Perhaps so, but perhaps deserves mention in our "migration guide"?
> 
> It's normal and acceptable to modify the UI strings between releases. Babel
> does extract all strings for every new release. I don't think this needs to be
> mentioned in the migration guide.

Excellent. Thanks for your expertise, Kit.
Comment 12 Masayuki Fuse CLA 2011-02-02 05:14:37 EST
Verified the fixed in M-3.2.3-20110127202034. Thanks