Bug 870998 - [ja_JP] Test case failure: Storage Tab: General sub-tab: unnecessary colon (":") between "Used: xxxxGB" line and NFS Export Path
Summary: [ja_JP] Test case failure: Storage Tab: General sub-tab: unnecessary colon ("...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-webadmin-portal
Version: 3.1.0
Hardware: Unspecified
OS: Linux
medium
high
Target Milestone: ---
: 3.2.0
Assignee: Einav Cohen
QA Contact: Pavel Stehlik
URL:
Whiteboard: ux
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-10-29 11:23 UTC by Yuko Katabami
Modified: 2013-06-11 08:22 UTC (History)
9 users (show)

Fixed In Version: sf3
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 883896 (view as bug list)
Environment:
Last Closed: 2013-06-11 08:18:16 UTC
oVirt Team: ---
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
Storage main tab - General sub-tab (Japanese) (80.68 KB, image/png)
2012-12-04 22:27 UTC, Einav Cohen 		no flags Details
Fixed colon removed. (92.57 KB, image/png)
2013-01-09 05:15 UTC, Lijun Li 		no flags Details
View All

Description Yuko Katabami 2012-10-29 11:23:36 UTC 
Filed from caserun (INSERT URL HERE)

Version-Release number of selected component (if applicable):
unspecified

Steps to Reproduce: 
  * Portal should run in local language


  * Click on Storage Tab in Portal

  * In the Detail Pane, select General tab.






Actual results: 
There is a colon ":" between "Used: xxxxGB" line and NFS Export Path line.

Expected results:
It should not be there?
Comment 1 Shahar Havivi 2012-11-04 14:25:40 UTC 
The reason that you see : in Japanese is because of the Zenata string:
empty=\u3000

Please remove it, I saw 4 more entries at this file:
CommonApplicationConstants_ja.properties
Constants_ja.properties
ApplicationConstants_ja.properties
ApplicationConstants_ja.properties

I will add additional code fix for this bug.
Comment 2 Shahar Havivi 2012-11-04 15:03:45 UTC 
posted at: http://gerrit.ovirt.org/#/c/9012/
Comment 3 Shahar Havivi 2012-11-04 15:27:32 UTC 
merged at: http://gerrit.ovirt.org/gitweb?p=ovirt-engine.git;a=commit;h=9e5e2498cecce70387679cb073a5327b0950fb20
Comment 5 Einav Cohen 2012-11-08 17:12:07 UTC 
Hi Yuko, just making sure that you saw Shahar's Comment #1. Thanks.
Comment 6 Yuko Katabami 2012-11-08 23:41:59 UTC 
Hi Einav and Shahar,

Sorry for my late response.
I have removed all entries of the space charaters where supposed to be empty and double checked.
Please let me know if you still have any problems.
Comment 7 Einav Cohen 2012-12-04 22:27:36 UTC 
Created attachment 657783 [details]
Storage main tab - General sub-tab (Japanese)
Comment 8 Einav Cohen 2012-12-04 22:30:07 UTC 
Yuko, I pulled updated strings from Zanata into the code and it seems that the problem has been fixed, however I am not absolutely sure, since no screen-shot has been provided when the BZ was reported.

Can you please confirm that the problem is indeed resolved (see attachment 657783 [details])?

[if so - feel free to move this BZ to MODIFIED. Thanks]
Comment 9 Yuko Katabami 2012-12-04 22:34:10 UTC 
Confirmed that it has been fixed. Thank you very much.
Comment 11 Einav Cohen 2012-12-20 15:33:54 UTC 
CodeChange (nothing to test here on English) - moving to VERIFIED.
Comment 12 Einav Cohen 2013-01-09 04:19:51 UTC 
(In reply to comment #11)
> CodeChange (nothing to test here on English) - moving to VERIFIED.

Fix still needs to be verified in Japanese (fix has simply removed the usage of the localized empty string) -> removing CodeChange keyword, moving to ON_QA on latest RHEV-M 3.2 version (sf3).
Comment 13 Lijun Li 2013-01-09 05:12:04 UTC 
Verified on the latest rhevm3.2 sf3 build, it's fixed.
[root@rhevm32 ~]# rpm -q rhevm rhevm-webadmin-portal
rhevm-3.2.0-4.el6ev.noarch
rhevm-webadmin-portal-3.2.0-4.el6ev.noarch

Please refer to the attached screen shot.

Thanks,
Robert
Comment 14 Lijun Li 2013-01-09 05:15:36 UTC 
Created attachment 675256 [details]
Fixed colon removed.
Comment 15 Itamar Heim 2013-06-11 08:18:16 UTC 
3.2 has been released
Comment 16 Itamar Heim 2013-06-11 08:22:49 UTC 
3.2 has been released
Note You need to log in before you can comment on or make changes to this bug.