From 5ecab7b05e30ba1d1874ff7de96936d990617ca3 Mon Sep 17 00:00:00 2001 From: Mathieu Baudier Date: Wed, 15 Nov 2017 13:23:20 +0100 Subject: [PATCH] Fix marking as workgroup --- .../ui/workbench/internal/useradmin/parts/GroupMainPage.java | 5 +++++ 1 file changed, 5 insertions(+) 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); -- 2.30.2