X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.security.ui.admin%2Fsrc%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Fcommands%2FAddRole.java;fp=org.argeo.security.ui.admin%2Fsrc%2Forg%2Fargeo%2Fsecurity%2Fui%2Fadmin%2Fcommands%2FAddRole.java;h=c91f81a3c455a4e39cf94592a5231db040e4c4ef;hb=f090d55a56532e0e6c2cde5760a451e56ef25f1c;hp=56946e76b48d191c4d94cd1fb046571216042d15;hpb=50911fdcc6df5cd35e71a0a4ecddf03f98f742a2;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/commands/AddRole.java b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/commands/AddRole.java
index 56946e76b..c91f81a3c 100644
--- a/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/commands/AddRole.java
+++ b/org.argeo.security.ui.admin/src/org/argeo/security/ui/admin/commands/AddRole.java
@@ -15,16 +15,10 @@
  */
 package org.argeo.security.ui.admin.commands;
 
-import org.argeo.ArgeoException;
 import org.argeo.security.UserAdminService;
-import org.argeo.security.ui.admin.editors.JcrArgeoUserEditor;
-import org.argeo.security.ui.admin.views.JcrRolesView;
 import org.eclipse.core.commands.AbstractHandler;
 import org.eclipse.core.commands.ExecutionEvent;
 import org.eclipse.core.commands.ExecutionException;
-import org.eclipse.ui.IEditorReference;
-import org.eclipse.ui.IWorkbenchPage;
-import org.eclipse.ui.handlers.HandlerUtil;
 
 /** Add a new role. */
 public class AddRole extends AbstractHandler {
@@ -33,37 +27,44 @@ public class AddRole extends AbstractHandler {
 	private String rolePrefix = "ROLE_";
 
 	public Object execute(ExecutionEvent event) throws ExecutionException {
-		JcrRolesView rolesView = (JcrRolesView) HandlerUtil
-				.getActiveWorkbenchWindow(event).getActivePage()
-				.findView(JcrRolesView.ID);
-		String role = rolesView.getNewRole();
-		if (role.trim().equals(""))
-			return null;
-		if (role.equals(rolesView.getAddNewRoleText()))
-			return null;
-		role = role.trim().toUpperCase();
-		if (!role.startsWith(rolePrefix))
-			role = rolePrefix + role;
-		if (userAdminService.listEditableRoles().contains(role))
-			throw new ArgeoException("Role " + role + " already exists");
-		userAdminService.newRole(role);
-		rolesView.refresh();
-
-		// refresh editors
-		IEditorReference[] refs = HandlerUtil.getActiveWorkbenchWindow(event)
-				.getActivePage()
-				.findEditors(null, JcrArgeoUserEditor.ID, IWorkbenchPage.MATCH_ID);
-		for (IEditorReference ref : refs) {
-			JcrArgeoUserEditor userEditor = (JcrArgeoUserEditor) ref.getEditor(false);
-			if (userEditor != null) {
-				userEditor.refresh();
-			}
-		}
 		return null;
 	}
 
 	public void setUserAdminService(UserAdminService userAdminService) {
 		this.userAdminService = userAdminService;
 	}
-
 }
+// JcrRolesView rolesView = (JcrRolesView) HandlerUtil
+// .getActiveWorkbenchWindow(event).getActivePage()
+// .findView(JcrRolesView.ID);
+// String role = rolesView.getNewRole();
+// if (role.trim().equals(""))
+// return null;
+// if (role.equals(rolesView.getAddNewRoleText()))
+// return null;
+// role = role.trim().toUpperCase();
+// if (!role.startsWith(rolePrefix))
+// role = rolePrefix + role;
+// if (userAdminService.listEditableRoles().contains(role))
+// throw new ArgeoException("Role " + role + " already exists");
+// userAdminService.newRole(role);
+// rolesView.refresh();
+//
+// // refresh editors
+// IEditorReference[] refs = HandlerUtil.getActiveWorkbenchWindow(event)
+// .getActivePage()
+// .findEditors(null, JcrArgeoUserEditor.ID, IWorkbenchPage.MATCH_ID);
+// for (IEditorReference ref : refs) {
+// JcrArgeoUserEditor userEditor = (JcrArgeoUserEditor) ref.getEditor(false);
+// if (userEditor != null) {
+// userEditor.refresh();
+// }
+// }
+// return null;
+// }
+//
+// public void setUserAdminService(UserAdminService userAdminService) {
+// this.userAdminService = userAdminService;
+// }
+//
+// }