Eli Mesika has submitted this change and it was merged.

Change subject: core: Fixing NPE when group is assigned with permissions
......................................................................


core: Fixing NPE when group is assigned with permissions

The patch associated with Idb1a7146c29eb74f97e10043d65b5a67f1430021
introduced a regression in assigning a group permissions.
Prior to that patch the entity DbGroup did not exist, hence
the CTOR of LdapGroup that has DbGroup did not exist as well.
The memberOf initialization is fixed here for cases that the flows do not
set the memberOf attribute

Change-Id: If92ce2718a32658b4118e4e7f9334b631fa6c839
Signed-off-by: Yair Zaslavsky <yzasl...@redhat.com>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/DbGroup.java
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/LdapGroup.java
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved
  Oved Ourfali: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/22612
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If92ce2718a32658b4118e4e7f9334b631fa6c839
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to