Tomas Jelinek has uploaded a new change for review. Change subject: userportal: Extended view/Basic view visibility fixes ......................................................................
userportal: Extended view/Basic view visibility fixes There where two problems with determining, if the Extended view should be shown or not: 1: The Everyone has a QuotaConsumer role which enabled everyone to see the Extended view. Fixed by ignoring the Everyone QuotaConsumer when deciding if show or not show the extended view 2: After logout/login as user with different role, the visibility of the header buttons (Basic, Extended) was not updated. Fixed by moving the code which refreshes this from HeaderPresenterWidget.onBind to HeaderPresenterWidget.onReset because the HeaderPresenterWidget is a singleton. Change-Id: I16ec262e7e5b46ccfdba28d091726d8368cae054 Signed-off-by: Tomas Jelinek <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalLoginModel.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/presenter/HeaderPresenterWidget.java 2 files changed, 21 insertions(+), 6 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/24/5424/1 -- To view, visit http://gerrit.ovirt.org/5424 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I16ec262e7e5b46ccfdba28d091726d8368cae054 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
