Change in ovirt-engine[master]: userportal: createRequestToken should use getHashName (#8507...
derez at redhat.com
derez at redhat.com
Mon Sep 3 05:47:17 UTC 2012
Daniel Erez has submitted this change and it was merged.
Change subject: userportal: createRequestToken should use getHashName (#850749)
......................................................................
userportal: createRequestToken should use getHashName (#850749)
https://bugzilla.redhat.com/850749
* AbstractSideTabWithDetailsPresenter -> createRequestToken:
subTabName should be composed of the HashName and not the Title
(i.e. as in the WebAdmin).
Getting the tab's name by its title:
- is wrong due to translation issues (HashName is not translated).
- caused a problem in Templates-Disks sub-tab revealing
after changing 'Virtual Disks' to 'Disks'.
* Renaming HashName from 'virtual_disks' to 'disks' just to
keep on consistency.
Change-Id: I769538944b014ddd69a37ed4c151b607d6266ca1
Signed-off-by: Daniel Erez <derez at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolDiskListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateDiskListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmDiskListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmMonitorModel.java
M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/place/ApplicationPlaces.java
M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/presenter/AbstractSideTabWithDetailsPresenter.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/place/ApplicationPlaces.java
7 files changed, 9 insertions(+), 8 deletions(-)
Approvals:
Daniel Erez: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/7687
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I769538944b014ddd69a37ed4c151b607d6266ca1
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
More information about the Engine-commits
mailing list