Change in ovirt-engine[master]: webadmin, userportal: ClientStorage improvements
vszocs at redhat.com
vszocs at redhat.com
Tue Jun 2 15:10:09 UTC 2015
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin,userportal: ClientStorage improvements
......................................................................
webadmin,userportal: ClientStorage improvements
1. All key/value pairs persisted through ClientStorage interface
now use application-specific key prefix to avoid clashes between
WebAdmin vs. UserPortal instances running on the same browser
(but not necessarily at the same time) where one instance might
read data originally persisted by another instance.
Example key names:
ENGINE_WebAdmin_MyKey
ENGINE_UserPortal_MyKey
2. ClientStorageImpl behavior modified as following:
* get{Local,Session}Item -- first use prefixed key, if missing
fall back to un-prefixed key for backwards compatibility
* set{Local,Session}Item -- use prefixed key (only)
3. Removed unused ClientStorage interface methods:
* removeLocalItem
* removeSessionItem
4. Added ClientStorageImplTest.
5. In UserPortal SystemModule, removed unnecessary GIN bindings.
Change-Id: Iaef8f72feec5dd083fbe4cc962a222dce4f7a99f
Signed-off-by: Vojtech Szocs <vszocs at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/ClientStorage.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/ClientStorageImpl.java
A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/ClientStorageKeyPrefix.java
A frontend/webadmin/modules/gwt-common/src/test/java/org/ovirt/engine/ui/common/system/ClientStorageImplTest.java
M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/gin/SystemModule.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/SystemModule.java
6 files changed, 193 insertions(+), 37 deletions(-)
Approvals:
Tomas Jelinek: Looks good to me, approved
Alexander Wels: Looks good to me, approved
Jenkins CI: Verified; Looks good to me, but someone else must approve
Vojtech Szocs: Verified
--
To view, visit https://gerrit.ovirt.org/41404
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaef8f72feec5dd083fbe4cc962a222dce4f7a99f
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
More information about the Engine-commits
mailing list