Tomas Jelinek has submitted this change and it was merged.
Change subject: userportal: ignore the NetworkUser role
......................................................................
userportal: ignore the NetworkUser role
Since 97e84c622a7cfa9f709eb4d814d42d90448edc91 there is a
NetworkUser role and this role is assigned to everyone.
Since the engineUser does not contain this role, UP
evaluates, that this specific user has more roles than the
engineUser so he is an admin and should see the extended tab.
Fixed by ignoring the NetworkUser in this algorithm
the same way as e.g. the QuotaConsumer is.
Change-Id: Iaf696b51159ba016d88124b789d5bbbab7b736fe
Bug-Url:
https://bugzilla.redhat.com/916328
Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/auth/ApplicationGuids.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalLoginModel.java
2 files changed, 9 insertions(+), 1 deletion(-)
Approvals:
Tomas Jelinek: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/12526
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf696b51159ba016d88124b789d5bbbab7b736fe
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>