Bug 995754

Summary: webadmin [Tree]: fix hardcoded non-i18n strings in tree context info messages
Product: [Retired] oVirt Reporter: Greg Sheremeta <gshereme>
Component: ovirt-engine-webadminAssignee: Greg Sheremeta <gshereme>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: low Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: acathrow, derez, ecohen, iheim, mgoldboi, yeylon
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ux
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-09-23 07:27:16 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Greg Sheremeta 2013-08-10 21:02:54 UTC
Description of problem:
In several model classes, there are hard-coded non-internationalized strings that are used to provide context help to the user.

Example:
In HostListModel.java
innerHostModel.getName().setInfo("Cannot edit Host's Name in this tree context"); //$NON-NLS-1$

At current count there are about 20 of these in uicommonweb.

How reproducible:
Always

Steps to Reproduce:
1. Select a language other than English
2. Select an actual host in the tree.
3. Click edit.
4. You cannot edit the host, and the context help reason shown is in English.

Actual results:
The reason is in English.

Expected results:
These strings should be internationalized and not hardcoded. The reason should be in the user's selected language.

Additional info:

Comment 1 Greg Sheremeta 2013-08-10 21:09:25 UTC
Step to Reproduce #4 should say
4. You cannot edit the host *name*, and the context help reason shown is in English.

Comment 2 Einav Cohen 2013-08-10 21:30:17 UTC
*** Bug 995755 has been marked as a duplicate of this bug. ***

Comment 3 Einav Cohen 2013-08-10 21:33:56 UTC
according to the comment posted in http://gerrit.ovirt.org/#/c/17886/, setInfo is deprecated and should be replaced with "setChangeProhibitionReason" (thanks Derez). 
all occurrences of "setInfo" in uicommonweb should be replaced with "setChangeProhibitionReason", and while at it - be localized as well (i.e. extracted to UIConstants).

Comment 4 Greg Sheremeta 2013-08-15 21:49:26 UTC
done

Comment 5 Itamar Heim 2013-08-21 16:41:45 UTC
as RC is built, moving to ON_QA (hopefully did not catch incorrect bugs when doing this)

Comment 6 Itamar Heim 2013-09-23 07:27:16 UTC
closing as this should be in 3.3 (doing so in bulk, so may be incorrect)