Fix marking as workgroup
authorMathieu Baudier <mbaudier@argeo.org>
Wed, 15 Nov 2017 12:23:20 +0000 (13:23 +0100)
committerMathieu Baudier <mbaudier@argeo.org>
Wed, 15 Nov 2017 12:23:20 +0000 (13:23 +0100)
org.argeo.cms.ui.workbench/src/org/argeo/cms/ui/workbench/internal/useradmin/parts/GroupMainPage.java

index 1f0011d49d047a769ec54d67a19a456b07cf8a7a..6987a217d807c06450aa927e36817dda7ce0a44d 100644 (file)
@@ -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);