Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 244461

Summary: Duplicating colon in error message prefix
Product: [Eclipse Project] PDE Reporter: Inocencio Richiez <irichiez>
Component: UIAssignee: Kit Lo <kitlo>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: contact, kitlo, Lars.Vogel, Vikas.Chandra
Version: 3.5   
Target Milestone: 4.11 M1   
Hardware: PC   
OS: Windows XP   
See Also: https://git.eclipse.org/r/133797
https://git.eclipse.org/c/pde/eclipse.pde.ui.git/commit/?id=b9a0a04566089d81f1909d3ea4733a11c62eb944
Whiteboard:
Attachments:
Description Flags
message with double semicolons
none
Test result in English.png
none
Test result in Traditional Chinese.png none

Description Inocencio Richiez CLA 2008-08-18 14:05:43 EDT
Created attachment 110271 [details]
message with double semicolons

Build ID: I20080609-1311

Steps To Reproduce:
1.Set your enviroment to Traditional Chinese (zh_TW)
2. Create a Plugin project
2.In the Plug-in Content page, set the Plug-in Version to e.g. *1.0.0 (to force the error message)
3.The error message displayed ont he wizard contain two (2) semicolons.


More information:
Comment 1 Kit Lo CLA 2009-06-22 23:07:27 EDT
First of all, it's DBCS (double byte languages, which includes Simplified Chinese, Traditional Chinese, Japanese, Korean) languages' convention to put the mnemonic char in parenthesis at the end of the string.

However, looks like PDE is reusing the field prompt string in the message, causing "(V)" to be displayed. That's the first problem.

I also noticed that when Simplified Chinese and Traditional Chinese translate the single byte colon into a double byte colon, an extra colon is inserted "programmatically", I think.

Both problems are coding problems in PDE. Assigning to PDE to investigate.
Comment 2 Benjamin Cabé CLA 2009-06-23 08:28:54 EDT
Thank you very much Kit, the two points you are highlighting are indeed issues in PDE itself.
Comment 3 Eclipse Genie CLA 2018-10-02 01:26:42 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 4 Lars Vogel CLA 2018-12-03 09:31:47 EST
Please reopen if you plan to provide a patch.
Comment 5 Kit Lo CLA 2018-12-07 18:31:07 EST
We should format message with MessageFormat instead of programmatically removing mnemonic and adding colon.
Comment 6 Eclipse Genie CLA 2018-12-07 18:45:25 EST
New Gerrit change created: https://git.eclipse.org/r/133705
Comment 7 Kit Lo CLA 2018-12-07 19:08:45 EST
Created attachment 276867 [details]
Test result in English.png

English message is still displaying correctly
Comment 8 Kit Lo CLA 2018-12-07 19:12:19 EST
Created attachment 276868 [details]
Test result in Traditional Chinese.png

Message in Traditional Chinese with mnemonic "(V)" is removed and no duplicating colon
Comment 9 Eclipse Genie CLA 2018-12-08 10:43:11 EST
New Gerrit change created: https://git.eclipse.org/r/133734
Comment 10 Kit Lo CLA 2018-12-08 11:29:51 EST
Lars, I abandoned the second Gerrit change and tried to commit the update with the same change-id, but received this message: "same Change-Id in multiple changes.
Squash the commits with the same Change-Id or ensure Change-Ids are unique for each commit"

Please provide the instructions for adding the update as patch set 2. Thanks!
Comment 11 Lars Vogel CLA 2018-12-10 11:10:59 EST
(In reply to Kit Lo from comment #10)
> Lars, I abandoned the second Gerrit change and tried to commit the update
> with the same change-id, but received this message: "same Change-Id in
> multiple changes.
> Squash the commits with the same Change-Id or ensure Change-Ids are unique
> for each commit"
> 
> Please provide the instructions for adding the update as patch set 2. Thanks!

See http://www.vogella.com/tutorials/Gerrit/article.html#gerrit_problems_squashcommits and http://www.vogella.com/tutorials/Gerrit/article.html#combining-changes-from-two-commits-gerrit-change-reviews for detailed instructions.
Comment 12 Eclipse Genie CLA 2018-12-10 13:57:40 EST
New Gerrit change created: https://git.eclipse.org/r/133797
Comment 13 Kit Lo CLA 2018-12-10 14:08:59 EST
Lars, thanks for the instructions. Very helpful. However, I saw a commit in between mine. Just to be safe, I started fresh again. Please review the latest Gerrit change: https://git.eclipse.org/r/133797 Thanks!
Comment 14 Lars Vogel CLA 2018-12-13 05:50:29 EST
Thanks, Kit.
Comment 16 Vikas Chandra CLA 2019-01-07 04:25:38 EST
Can someone verify this fix?
Comment 17 Lars Vogel CLA 2019-01-07 04:50:46 EST
(In reply to Vikas Chandra from comment #16)
> Can someone verify this fix?

Kit, please verify.
Comment 18 Kit Lo CLA 2019-01-07 21:19:40 EST
I verified the fix in 4.11-I-build I20190103-1800