Change in ovirt-engine[master]: userportal: ignore the NetworkUser role

tjelinek at redhat.com tjelinek at redhat.com
Fri Mar 1 07:55:11 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list