X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.security.ui.admin%2Fsrc%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Fcommands%2FNewGroup.java;h=d797c5d748fae47b1c09b59a63009acc0c259466;hb=63c252bcf4aa2cb508a45a839d032d64daecfa82;hp=22780f27cf3bd0f625343591d7bce35d452e82df;hpb=28f2a8c1668ce2e9d2bb2a6601a659c2bcc2676b;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/commands/NewGroup.java b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/commands/NewGroup.java
index 22780f27c..d797c5d74 100644
--- a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/commands/NewGroup.java
+++ b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/commands/NewGroup.java
@@ -17,10 +17,8 @@ package org.argeo.security.ui.admin.commands;
 
 import java.util.Dictionary;
 
-import javax.transaction.Status;
 import javax.transaction.UserTransaction;
 
-import org.argeo.ArgeoException;
 import org.argeo.eclipse.ui.EclipseUiUtils;
 import org.argeo.eclipse.ui.dialogs.ErrorFeedback;
 import org.argeo.jcr.ArgeoNames;
@@ -111,17 +109,8 @@ public class NewGroup extends AbstractHandler {
 			if (!canFinish())
 				return false;
 			String commonName = commonNameTxt.getText();
-
-			// Begin transaction if needed
-			try {
-				if (userTransaction.getStatus() == Status.STATUS_NO_TRANSACTION)
-					userTransaction.begin();
-			} catch (Exception e) {
-				throw new ArgeoException("Unable to start "
-						+ "transaction to create user " + commonName, e);
-			}
-
 			try {
+				UiAdminUtils.beginTransactionIfNeeded(userTransaction);
 				Group user = (Group) userAdmin.createRole(getDn(commonName),
 						Role.GROUP);
 				Dictionary props = user.getProperties();