From: Mathieu Baudier <mbaudier@argeo.org>
Date: Wed, 15 Nov 2017 12:23:20 +0000 (+0100)
Subject: Fix marking as workgroup
X-Git-Tag: argeo-commons-2.1.70~15
X-Git-Url: http://git.argeo.org/?a=commitdiff_plain;h=5ecab7b05e30ba1d1874ff7de96936d990617ca3;p=lgpl%2Fargeo-commons.git

Fix marking as workgroup
---

diff --git a/org.argeo.cms.ui.workbench/src/org/argeo/cms/ui/workbench/internal/useradmin/parts/GroupMainPage.java b/org.argeo.cms.ui.workbench/src/org/argeo/cms/ui/workbench/internal/useradmin/parts/GroupMainPage.java
index 1f0011d49..6987a217d 100644
--- a/org.argeo.cms.ui.workbench/src/org/argeo/cms/ui/workbench/internal/useradmin/parts/GroupMainPage.java
+++ b/org.argeo.cms.ui.workbench/src/org/argeo/cms/ui/workbench/internal/useradmin/parts/GroupMainPage.java
@@ -214,8 +214,13 @@ public class GroupMainPage extends FormPage implements ArgeoNames {
 						return; // already marked as workgroup, do nothing
 					else
 						try {
+							// improve transaction management
+							userAdminWrapper.beginTransactionIfNeeded();
 							nodeInstance.createWorkgroup(new LdapName(group.getName()));
 							setProperty(group, businessCategory, WORKGROUP);
+							userAdminWrapper.commitOrNotifyTransactionStateChange();
+							userAdminWrapper
+									.notifyListeners(new UserAdminEvent(null, UserAdminEvent.ROLE_CHANGED, group));
 							part.refresh();
 						} catch (InvalidNameException e1) {
 							throw new CmsException("Cannot create Workgroup for " + group.toString(), e1);