Bug 235475 - TVT34:TCT723: two more strings to externalize in file/new menu II
Summary: TVT34:TCT723: two more strings to externalize in file/new menu II
Status: VERIFIED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.xml (show other bugs)
Version: 3.0   Edit
Hardware: PC All
: P2 normal (vote)
Target Milestone: 3.0 RC4   Edit
Assignee: Nick Sandonato CLA
QA Contact: Nitin Dahyabhai CLA
URL: 723
Whiteboard: PMC_approved
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-06-03 17:18 EDT by CDE Administration CLA
Modified: 2008-06-12 12:34 EDT (History)
8 users (show)

See Also:
david_williams: pmc_approved+
for.work.things: pmc_approved? (raghunathan.srinivasan)
for.work.things: pmc_approved? (naci.dai)
for.work.things: pmc_approved? (deboer)
neil.hauge: pmc_approved+
kaloyan: pmc_approved+
for.work.things: review+


Attachments
file-new.png (252.02 KB, image/png)
2008-06-03 17:18 EDT, CDE Administration CLA
no flags Details
patch (1.28 KB, patch)
2008-06-04 17:39 EDT, Nick Sandonato CLA
bjorn.freeman-benson: iplog+
Details | Diff
org.eclipse.wst.xml.ui.patch (1.96 KB, patch)
2008-06-04 23:40 EDT, Amy Wu CLA
bjorn.freeman-benson: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description CDE Administration CLA 2008-06-03 17:18:31 EDT
<response_by> Stepan Kvapilik at 2008.06.03.15.14.39 </response_by>
Hi, I was asked to separate the TCT#647 to two problems, so here is the 2nd one:

OS: Linux
Build date: 0602
Component/Function name: any
Blocking: NO
Tester Name: Ji?? Motjekz?k (Czech)

Steps to recreate the problem:
In eclipse, go to the file/new/other menu.

Problem:
I found one more string which should be externalized in this menu.

- The menu item "XML" and its subitem "XML" are taken from the same string "_UI_WIZARD_NEW_XML". As the meaning of the menu item is "group for XML files", while the inner item means "concrete XML file type", the translation of these two strings should be separated.

The problem was checked in the EN build with string IDs.

Attaching screenshot for reference.

Jiri

<response_by> Karl Mittmann at 2008.06.03.15.55.09 </response_by>
This article was reassigned from Category:''TVT/Testing,Inbox''.
Comment 1 CDE Administration CLA 2008-06-03 17:18:39 EDT
Created attachment 103441 [details]
file-new.png
Comment 2 CDE Administration CLA 2008-06-03 17:18:44 EDT
<cde:tctdetail>
Testcase: N/A
Project: WSW34
Component: Xfer - Web Tools/wst.xml
Priority: 2
Subject: two more strings to externalize in file/new menu II
Article ID: 723
Originator: ibm@hiero.cz
</cde:tctdetail>
Comment 3 Nick Sandonato CLA 2008-06-04 17:39:10 EDT
Created attachment 103656 [details]
patch

The Wizard Category "XML" is now a separate string from the new file "XML".
Comment 4 Amy Wu CLA 2008-06-04 17:53:01 EDT
review and tested fix. it looks good.
Comment 5 Amy Wu CLA 2008-06-04 18:06:45 EDT
* 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 is an important bug to fix because we want to fix externalized strings to be a good citizen for all countries. Not fixing this will be confusing if translators want to distinguish the difference between an XML category versus an XML file.

* Is there a work-around? If so, why do you believe the work-around is insufficient?
No

* How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added?
Fix has been manually tested by me and Nick Sandonato

* Give a brief technical overview. Who has reviewed this fix? 
Both the label for the XML category and the label for the New XML wizard in the New files dialog were using the same string.  Fix is to create a new string for the XML category label.  Nick and I have reviewed the fix.

* What is the risk associated with this fix?
None.
Comment 6 David Williams CLA 2008-06-04 20:16:49 EDT
I'm fine with this, but would also ask you remove the extraneous 
<keywordReference id="preferenceKeywords.severities"/> 
Comment 7 Neil Hauge CLA 2008-06-04 20:48:14 EDT
Approved.
Comment 8 Amy Wu CLA 2008-06-04 23:40:08 EDT
Created attachment 103689 [details]
org.eclipse.wst.xml.ui.patch

(In reply to comment #6)
> I'm fine with this, but would also ask you remove the extraneous 
> <keywordReference id="preferenceKeywords.severities"/> 

Actually, it's not extraneous.  There are additional preference keywords.
That line should be changed to:
<keywordReference id="org.eclipse.wst.xml.ui.severities"/>
And then the keyword id should be defined like so:
		<keyword
			label="%preferenceKeywords.severities"
			id="org.eclipse.wst.xml.ui.severities"/>

This way, if you filter preferences pages on severity keywords like "error" "ignore" "warning" you will get the XML Files preference page.

This additional fix is low risk.  The preference keywords are already defined/externalized in plugin.properties.  The keyword id just needed to be defined.
Comment 9 Amy Wu CLA 2008-06-05 15:24:21 EDT
Fix committed and released. Tagged v200806051910.
Comment 10 Nick Sandonato CLA 2008-06-10 10:12:19 EDT
Verified in S-3.0RC4-20080610084135.
Comment 11 CDE Administration CLA 2008-06-12 12:34:09 EDT
<response_by> Stepan Kvapilik at 2008.06.12.10.52.13 </response_by>
Thank you for externalizing, closing TCT.

Jiri
Comment 12 CDE Administration CLA 2008-06-12 12:34:17 EDT
<cde:tctdetail>
Testcase: N/A
Project: 
Component: Xfer - Web Tools/wst.xml
Priority: 2
Subject: two more strings to externalize in file/new menu II
Article ID: 723
Originator: ibm@hiero.cz
</cde:tctdetail>