X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.security.ui.admin%2Fsrc%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Finternal%2Fcommands%2FDeleteGroups.java;h=df5d430abd1fad48dee70330d558819dec39c415;hb=8260f4470f514ea347ca53f5b4dfc632c4a4de66;hp=868aa0fc7f6a3239ad5591f1b2cb315d63f4b94b;hpb=89ad04528a66488eb3ad6f51c198d5df3c81b04c;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/DeleteGroups.java b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/DeleteGroups.java
index 868aa0fc7..df5d430ab 100644
--- a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/DeleteGroups.java
+++ b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/DeleteGroups.java
@@ -71,10 +71,8 @@ public class DeleteGroups extends AbstractHandler {
 		UserAdmin userAdmin = userAdminWrapper.getUserAdmin();
 		IWorkbenchPage iwp = HandlerUtil.getActiveWorkbenchWindow(event)
 				.getActivePage();
-
 		for (Group group : groups) {
 			String groupName = group.getName();
-
 			// TODO find a way to close the editor cleanly if opened. Cannot be
 			// done through the UserAdminListeners, it causes a
 			// java.util.ConcurrentModificationException because disposing the
@@ -82,11 +80,16 @@ public class DeleteGroups extends AbstractHandler {
 			IEditorPart part = iwp.findEditor(new UserEditorInput(groupName));
 			if (part != null)
 				iwp.closeEditor(part, false);
-
 			userAdmin.removeRole(groupName);
+		}
+		userAdminWrapper.commitOrNotifyTransactionStateChange();
+
+		// Update the view
+		for (Group group : groups) {
 			userAdminWrapper.notifyListeners(new UserAdminEvent(null,
 					UserAdminEvent.ROLE_REMOVED, group));
 		}
+
 		return null;
 	}