X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.security.ui.admin%2Fsrc%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Finternal%2Fcommands%2FDeleteUsers.java;h=e583bef53bd610b5d99af00ab2266271536760d1;hb=c13da2a3a91edd21b54db7563727d009def37d3e;hp=87da43cdf14f7495c14ee3ba45837baf9cb014d3;hpb=0d843d64df3f588ad1429905cf5f6e29c65becd3;p=lgpl%2Fargeo-commons.git diff --git a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/DeleteUsers.java b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/DeleteUsers.java index 87da43cdf..e583bef53 100644 --- a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/DeleteUsers.java +++ b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/internal/commands/DeleteUsers.java @@ -81,7 +81,6 @@ public class DeleteUsers extends AbstractHandler { for (User user : users) { String userName = user.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 @@ -89,8 +88,11 @@ public class DeleteUsers extends AbstractHandler { IEditorPart part = iwp.findEditor(new UserEditorInput(userName)); if (part != null) iwp.closeEditor(part, false); - userAdmin.removeRole(userName); + } + userAdminWrapper.commitOrNotifyTransactionStateChange(); + + for (User user : users) { userAdminWrapper.notifyListeners(new UserAdminEvent(null, UserAdminEvent.ROLE_REMOVED, user)); }