Moti Asayag has submitted this change and it was merged.
Change subject: webadmin: Refactor RoleTreeView
......................................................................
webadmin: Refactor RoleTreeView
RoleTreeView had too long and complicated method
with too many indentation levels. The patch reduces
the complexity and allows new entities to be added
more easily to the Action Groups tree.
Change-Id: I82431f3e96be13d3815e9e89130f4cd674394bc8
Related-To:
https://bugzilla.redhat.com/1002483
Signed-off-by: Moti Asayag <masayag(a)redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/configure/roles_ui/RoleTreeView.java
1 file changed, 207 insertions(+), 341 deletions(-)
Approvals:
Moti Asayag: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/19494
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I82431f3e96be13d3815e9e89130f4cd674394bc8
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server