Split security UI to security admin ui
authorMathieu Baudier <mbaudier@argeo.org>
Thu, 17 Mar 2011 13:27:49 +0000 (13:27 +0000)
committerMathieu Baudier <mbaudier@argeo.org>
Thu, 17 Mar 2011 13:27:49 +0000 (13:27 +0000)
git-svn-id: https://svn.argeo.org/commons/trunk@4312 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc

32 files changed:
demo/argeo-node-ui.product
demo/argeo-node-web.properties
security/modules/org.argeo.security.manager.ldap/build.properties [new file with mode: 0644]
security/modules/org.argeo.security.services/META-INF/spring/osgi.xml
security/modules/org.argeo.security.services/build.properties [new file with mode: 0644]
security/plugins/org.argeo.security.ui.admin/META-INF/spring/commands.xml
security/plugins/org.argeo.security.ui.admin/META-INF/spring/editors.xml
security/plugins/org.argeo.security.ui.admin/META-INF/spring/osgi.xml
security/plugins/org.argeo.security.ui.admin/META-INF/spring/views.xml
security/plugins/org.argeo.security.ui.admin/build.properties [new file with mode: 0644]
security/plugins/org.argeo.security.ui.admin/plugin.xml
security/plugins/org.argeo.security.ui.admin/pom.xml
security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/SecurityAdminPerspective.java
security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/SecurityAdminPlugin.java
security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/AddRole.java
security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/OpenArgeoUserEditor.java
security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/commands/SaveArgeoUser.java
security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/editors/ArgeoUserEditor.java
security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/editors/DefaultUserMainPage.java
security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/views/RolesView.java
security/plugins/org.argeo.security.ui.admin/src/main/java/org/argeo/security/ui/admin/views/UsersView.java
security/plugins/org.argeo.security.ui.rap/build.properties [new file with mode: 0644]
security/plugins/org.argeo.security.ui/META-INF/spring/commands.xml
security/plugins/org.argeo.security.ui/META-INF/spring/osgi.xml
security/plugins/org.argeo.security.ui/pom.xml
security/plugins/org.argeo.security.ui/src/main/java/org/argeo/security/ui/commands/OpenChangePasswordDialog.java
security/plugins/org.argeo.security.ui/src/main/java/org/argeo/security/ui/dialogs/ChangePasswordDialog.java
security/plugins/pom.xml
security/runtime/org.argeo.security.activemq/build.properties
security/runtime/org.argeo.security.activemq/src/main/java/org/argeo/security/activemq/ActiveMqSecurityBrokerPlugin.java
server/plugins/org.argeo.jcr.ui.explorer/plugin.xml
server/runtime/org.argeo.server.jcr/src/main/java/org/argeo/jcr/BeanNodeMapper.java

index 9dba055366445de5672fd183d9ab50e65b80d7b7..820f999c779a30b9ca7feb67688f50fa097c6106 100644 (file)
@@ -8,7 +8,7 @@
 
    <launcherArgs>
       <programArgs>-console -clean</programArgs>
-      <vmArgs>-Dlog4j.configuration=&quot;file:${system_property:user.home}/dev/src/commons/security/demo/log4j.properties&quot;</vmArgs>
+      <vmArgs>-Dlog4j.configuration=&quot;file:${system_property:user.home}/dev/src/commons/demo/log4j.properties&quot;</vmArgs>
       <vmArgsMac>-XstartOnFirstThread -Dorg.eclipse.swt.internal.carbon.smallFonts</vmArgsMac>
    </launcherArgs>
 
@@ -92,7 +92,7 @@
       <plugin id="org.argeo.eclipse.ui"/>
       <plugin id="org.argeo.eclipse.ui.jcr"/>
       <plugin id="org.argeo.eclipse.ui.rcp"/>
-      <plugin id="org.argeo.infra.core"/>
+      <plugin id="org.argeo.infra.core" fragment=""/>
       <plugin id="org.argeo.infra.security.services" fragment="true"/>
       <plugin id="org.argeo.jcr.ui.explorer"/>
       <plugin id="org.argeo.node.repo.jackrabbit"/>
       <plugin id="org.argeo.node.repo.jackrabbit" autoStart="true" startLevel="0" />
       <plugin id="org.argeo.security.manager.ldap" autoStart="true" startLevel="0" />
       <plugin id="org.argeo.security.services" autoStart="true" startLevel="0" />
-      <plugin id="org.argeo.server.ads.server" autoStart="true" startLevel="0" />
+      <plugin id="org.argeo.server.ads.server" autoStart="false" startLevel="0" />
       <plugin id="org.springframework.osgi.extender" autoStart="true" startLevel="0" />
    </configurations>
 
index 5009da72a8b9fd7dcf1357936d2ff6ac36afa49b..50e0120547415e003a5e79ce02391456e54fb6df 100644 (file)
@@ -18,4 +18,7 @@ org.argeo.jcr.ui.explorer,\
 
 org.argeo.security.ui.initialPerspective=org.argeo.jcr.ui.explorer.perspective
 
+argeo.node.repo.home=~/.argeo/server_noderepo
+argeo.node.repo.dburl=jdbc:h2:file:~/.argeo/h2/server_noderepo_db
+
 log4j.configuration=file:../../log4j.properties
diff --git a/security/modules/org.argeo.security.manager.ldap/build.properties b/security/modules/org.argeo.security.manager.ldap/build.properties
new file mode 100644 (file)
index 0000000..5f22cdd
--- /dev/null
@@ -0,0 +1 @@
+bin.includes = META-INF/
index 92b4129fd8ba6f8800c94b19ce3fd369791feed6..6822ed70a76f7f05114d619ab469aa965368dfcd 100644 (file)
@@ -12,9 +12,6 @@
 \r
        <reference id="securityDao" interface="org.argeo.security.ArgeoSecurityDao" />\r
 \r
-       <!-- <reference id="authenticationProvider" -->\r
-       <!-- interface="org.springframework.security.providers.AuthenticationProvider" \r
-               /> -->\r
        <list id="authenticationProviders"\r
                interface="org.springframework.security.providers.AuthenticationProvider"\r
                cardinality="0..N">\r
@@ -23,7 +20,7 @@
        </list>\r
 \r
        <!-- SERVICES -->\r
-       <service ref="securityService" interface="org.argeo.security.ArgeoSecurityService" />\r
+<!--   <service ref="securityService" interface="org.argeo.security.ArgeoSecurityService" /> -->\r
        <service ref="securityService" interface="org.argeo.security.CurrentUserService" />\r
        <service ref="securityService" interface="org.argeo.security.UserAdminService" />\r
        <service ref="systemExecutionService" interface="org.argeo.security.SystemExecutionService" />\r
diff --git a/security/modules/org.argeo.security.services/build.properties b/security/modules/org.argeo.security.services/build.properties
new file mode 100644 (file)
index 0000000..5f22cdd
--- /dev/null
@@ -0,0 +1 @@
+bin.includes = META-INF/
index cc36ad9fbae4f300c1f4261fe2a23f4b06c7f5d1..044fabf582d39a3a8a516df80a54d3701fd5a8ad 100644 (file)
@@ -10,7 +10,7 @@
                scope="prototype" />
        <bean id="addRole" class="org.argeo.security.ui.admin.commands.AddRole"
                scope="prototype">
-               <property name="securityService" ref="securityService" />
+               <property name="userAdminService" ref="userAdminService" />
        </bean>
        <bean id="refreshUsersList" class="org.argeo.security.ui.admin.commands.RefreshUsersList"
                scope="prototype" />
index 8ff9cec583def5557e6b60793d8adca58fd689da..84399c26ec845744a4126a5cd64445634dc79e0d 100644 (file)
@@ -7,6 +7,6 @@
        <!-- Editors -->
        <bean id="adminArgeoUserEditor" class="org.argeo.security.ui.admin.editors.ArgeoUserEditor"
                scope="prototype">
-               <property name="securityService" ref="securityService" />
+               <property name="userAdminService" ref="userAdminService" />
        </bean>
 </beans>
index 6574cf125286ff0218e0ba898b69e8b05edf74d6..f77b5a029a7d68cdf0ed77771b1cad82ccc12a53 100644 (file)
@@ -8,5 +8,5 @@
        http://www.springframework.org/schema/beans/spring-beans-2.5.xsd"\r
        osgi:default-timeout="30000">\r
 \r
-       <reference id="securityService" interface="org.argeo.security.ArgeoSecurityService" />\r
+       <reference id="userAdminService" interface="org.argeo.security.UserAdminService" />\r
 </beans:beans>
\ No newline at end of file
index a94e3da3056eae5d0f43dc132314636976eefdd3..8cf9fd68c80358d115ba7cc1cc175243286523bf 100644 (file)
@@ -6,10 +6,10 @@
 
        <bean id="adminUsersView" class="org.argeo.security.ui.admin.views.UsersView"
                scope="prototype">
-               <property name="securityService" ref="securityService" />
+               <property name="userAdminService" ref="userAdminService" />
        </bean>
        <bean id="adminRolesView" class="org.argeo.security.ui.admin.views.RolesView"
                scope="prototype">
-               <property name="securityService" ref="securityService" />
+               <property name="userAdminService" ref="userAdminService" />
        </bean>
 </beans>
diff --git a/security/plugins/org.argeo.security.ui.admin/build.properties b/security/plugins/org.argeo.security.ui.admin/build.properties
new file mode 100644 (file)
index 0000000..b840a94
--- /dev/null
@@ -0,0 +1,4 @@
+bin.includes = plugin.xml,\
+               META-INF/
+source.. = src/main/java/
+output.. = target/classes/
index 74fe49297904313199b3f53e8aae2259a0ce602a..d5bc7d18d814282bda0e7671f3c8f19b8fc039c8 100644 (file)
@@ -4,7 +4,7 @@
    <extension
          point="org.eclipse.ui.perspectives">
       <perspective
-            class="org.argeo.security.ui.admin.SecurityPerspective"
+            class="org.argeo.security.ui.admin.SecurityAdminPerspective"
             icon="icons/security.gif"
             id="org.argeo.security.ui.admin.adminSecurityPerspective"
             name="Security">
index 83b0192c370bb9afc39d12d66f065ca9057dd003..807258b8232297546eb530a7d1c0055105ab407c 100644 (file)
                                <artifactId>maven-bundle-plugin</artifactId>
                                <version>${version.maven-bundle-plugin}</version>
                                <configuration>
-                                       <!-- <instructions> -->
-                                       <!-- <Bundle-ActivationPolicy>lazy</Bundle-ActivationPolicy> -->
-                                       <!-- <Bundle-Activator>org.argeo.security.ui.SecurityUiPlugin</Bundle-Activator> -->
-                                       <!-- <Require-Bundle>org.eclipse.ui;resolution:=optional,org.eclipse.rap.ui;resolution:=optional,org.eclipse.core.runtime</Require-Bundle> -->
-                                       <!-- <Import-Package> -->
-                                       <!-- org.argeo.eclipse.spring, -->
-                                       <!-- org.osgi.framework;version="0.0.0", -->
-                                       <!-- !org.eclipse.core.runtime, -->
-                                       <!-- !org.eclipse.core.commands, -->
-                                       <!-- !org.eclipse.ui.plugin, -->
-                                       <!-- !org.eclipse.ui, -->
-                                       <!-- !org.eclipse.ui.commands, -->
-                                       <!-- !org.eclipse.ui.handlers, -->
-                                       <!-- !org.eclipse.ui.part, -->
-                                       <!-- * -->
-                                       <!-- </Import-Package> -->
-                                       <!-- <Private-Package>icons</Private-Package> -->
-                                       <!-- <Export-Package>org.argeo.security.ui.*</Export-Package> -->
-                                       <!-- </instructions> -->
                                        <instructions>
                                                <Bundle-ActivationPolicy>lazy</Bundle-ActivationPolicy>
-                                               <Bundle-Activator>org.argeo.security.ui.SecurityUiPlugin</Bundle-Activator>
-                                               <Require-Bundle>org.eclipse.ui;resolution:=optional,org.eclipse.rap.ui;resolution:=optional,org.eclipse.core.runtime</Require-Bundle>
+                                               <Bundle-Activator>org.argeo.security.ui.admin.SecurityAdminPlugin
+                                               </Bundle-Activator>
+                                               <Require-Bundle>org.eclipse.ui;resolution:=optional,org.eclipse.rap.ui;resolution:=optional,org.eclipse.core.runtime
+                                               </Require-Bundle>
                                                <Import-Package>
                                                        org.argeo.eclipse.spring,
                                                        *
                                                </Import-Package>
-                                               <Export-Package>
-                                                       !org.argeo.security.ui.internal.*,
-                                                       org.argeo.security.ui.*
-                                               </Export-Package>
+                                               <Export-Package>org.argeo.security.ui.admin.*</Export-Package>
                                        </instructions>
                                </configuration>
                        </plugin>
                        <version>0.2.3-SNAPSHOT</version>
                </dependency>
 
-               <!-- Argeo Eclipse distribution (common dependencies for both RAP and RCP) -->
-               <dependency>
-                       <groupId>org.argeo.commons.eclipse</groupId>
-                       <artifactId>org.argeo.eclipse.dep.common</artifactId>
-                       <version>0.2.3-SNAPSHOT</version>
-               </dependency>
-
                <!-- RCP only dependency, needed at compile time -->
                <dependency>
                        <groupId>org.argeo.commons.eclipse</groupId>
                <!-- Others -->
                <dependency>
                        <groupId>org.slf4j</groupId>
-                       <artifactId>com.springsource.slf4j.org.apache.commons.logging</artifactId>
+                       <artifactId>com.springsource.slf4j.org.apache.commons.logging
+                       </artifactId>
                </dependency>
        </dependencies>
 </project>
index 49f537cd5ba1f8faa0a32394bffd3d9b664a1c19..4c17bbca6261eb14862a99a4073e6277ba7155bb 100644 (file)
@@ -1,5 +1,7 @@
 package org.argeo.security.ui.admin;
 
+import org.argeo.security.ui.admin.views.RolesView;
+import org.argeo.security.ui.admin.views.UsersView;
 import org.eclipse.ui.IFolderLayout;
 import org.eclipse.ui.IPageLayout;
 import org.eclipse.ui.IPerspectiveFactory;
@@ -12,8 +14,8 @@ public class SecurityAdminPerspective implements IPerspectiveFactory {
 
                IFolderLayout left = layout.createFolder("left", IPageLayout.LEFT,
                                0.4f, editorArea);
-               left.addView("org.argeo.security.ui.admin.adminUsersView");
-               left.addView("org.argeo.security.ui.admin.adminRolesView");
+               left.addView(UsersView.ID);
+               left.addView(RolesView.ID);
        }
 
 }
index cc5d42919bbeae2e641d763c111b2d3c73944e87..dfe767bf18fd9c81c4eb7ba9764c8a42224b3f91 100644 (file)
@@ -5,48 +5,22 @@ import org.eclipse.ui.plugin.AbstractUIPlugin;
 import org.osgi.framework.BundleContext;
 
 public class SecurityAdminPlugin extends AbstractUIPlugin {
-
-       // The plug-in ID
        public static final String PLUGIN_ID = "org.argeo.security.ui.admin"; //$NON-NLS-1$
-
-       // The shared instance
        private static SecurityAdminPlugin plugin;
 
-       /**
-        * The constructor
-        */
        public SecurityAdminPlugin() {
        }
 
-       /*
-        * (non-Javadoc)
-        * 
-        * @see
-        * org.eclipse.ui.plugin.AbstractUIPlugin#start(org.osgi.framework.BundleContext
-        * )
-        */
        public void start(BundleContext context) throws Exception {
                super.start(context);
                plugin = this;
        }
 
-       /*
-        * (non-Javadoc)
-        * 
-        * @see
-        * org.eclipse.ui.plugin.AbstractUIPlugin#stop(org.osgi.framework.BundleContext
-        * )
-        */
        public void stop(BundleContext context) throws Exception {
                plugin = null;
                super.stop(context);
        }
 
-       /**
-        * Returns the shared instance
-        * 
-        * @return the shared instance
-        */
        public static SecurityAdminPlugin getDefault() {
                return plugin;
        }
index e8825bce302c8e71b9f501cda3035a20b80ebbc6..047c167e07f34590e8f67c499de989276e712eea 100644 (file)
@@ -1,7 +1,7 @@
 package org.argeo.security.ui.admin.commands;
 
 import org.argeo.ArgeoException;
-import org.argeo.security.ArgeoSecurityService;
+import org.argeo.security.UserAdminService;
 import org.argeo.security.ui.admin.views.RolesView;
 import org.eclipse.core.commands.AbstractHandler;
 import org.eclipse.core.commands.ExecutionEvent;
@@ -10,8 +10,8 @@ import org.eclipse.ui.handlers.HandlerUtil;
 
 /** Add a new role. */
 public class AddRole extends AbstractHandler {
-       public final static String COMMAND_ID = "org.argeo.security.ui.addRole";
-       private ArgeoSecurityService securityService;
+       public final static String COMMAND_ID = "org.argeo.security.ui.admin.addRole";
+       private UserAdminService userAdminService;
        private String rolePrefix = "ROLE_";
 
        public Object execute(ExecutionEvent event) throws ExecutionException {
@@ -26,15 +26,15 @@ public class AddRole extends AbstractHandler {
                role = role.trim().toUpperCase();
                if (!role.startsWith(rolePrefix))
                        role = rolePrefix + role;
-               if (securityService.listEditableRoles().contains(role))
+               if (userAdminService.listEditableRoles().contains(role))
                        throw new ArgeoException("Role " + role + " already exists");
-               securityService.newRole(role);
+               userAdminService.newRole(role);
                rolesView.refresh();
                return null;
        }
 
-       public void setSecurityService(ArgeoSecurityService securityService) {
-               this.securityService = securityService;
+       public void setUserAdminService(UserAdminService userAdminService) {
+               this.userAdminService = userAdminService;
        }
 
 }
index 7cac7f4c4511514ea9e8ea79aad337c1ad351295..155e15560350b0ccfd684766485b54aaa063f63e 100644 (file)
@@ -10,8 +10,8 @@ import org.eclipse.ui.handlers.HandlerUtil;
 
 /** Command handler to set visible or open a Argeo user. */
 public class OpenArgeoUserEditor extends AbstractHandler {
-       public final static String COMMAND_ID = "org.argeo.security.ui.openArgeoUserEditor";
-       public final static String PARAM_USERNAME = "org.argeo.security.ui.username";
+       public final static String COMMAND_ID = "org.argeo.security.ui.admin.openArgeoUserEditor";
+       public final static String PARAM_USERNAME = "org.argeo.security.ui.admin.username";
 
        public Object execute(ExecutionEvent event) throws ExecutionException {
                try {
index 162451413e3fd8a3dc36a0c68846c5c4bdecb6eb..aed34747231592fd6633647eda688382ddeae7ca 100644 (file)
@@ -11,7 +11,7 @@ import org.eclipse.ui.handlers.HandlerUtil;
 
 /** Save the currently edited Argeo user. */
 public class SaveArgeoUser extends AbstractHandler {
-       public final static String COMMAND_ID = "org.argeo.security.ui.saveArgeoUser";
+       public final static String COMMAND_ID = "org.argeo.security.ui.admin.saveArgeoUser";
 
        public Object execute(ExecutionEvent event) throws ExecutionException {
                try {
index 41e6d4fe7251edc4e46587825b488ea71485a061..9b82e8783985325f4dfd3979e52125d035308a47 100644 (file)
@@ -1,9 +1,9 @@
 package org.argeo.security.ui.admin.editors;
 
 import org.argeo.ArgeoException;
-import org.argeo.security.ArgeoSecurityService;
 import org.argeo.security.ArgeoUser;
 import org.argeo.security.SimpleArgeoUser;
+import org.argeo.security.UserAdminService;
 import org.argeo.security.nature.SimpleUserNature;
 import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.ui.IEditorInput;
@@ -13,10 +13,10 @@ import org.eclipse.ui.forms.editor.FormEditor;
 
 /** Editor for an Argeo user. */
 public class ArgeoUserEditor extends FormEditor {
-       public final static String ID = "org.argeo.security.ui.adminArgeoUserEditor";
+       public final static String ID = "org.argeo.security.ui.admin.adminArgeoUserEditor";
 
        private ArgeoUser user;
-       private ArgeoSecurityService securityService;
+       private UserAdminService userAdminService;
 
        public void init(IEditorSite site, IEditorInput input)
                        throws PartInitException {
@@ -28,14 +28,14 @@ public class ArgeoUserEditor extends FormEditor {
                        user.getUserNatures().put(SimpleUserNature.TYPE,
                                        new SimpleUserNature());
                } else
-                       user = securityService.getUser(username);
+                       user = userAdminService.getUser(username);
                this.setPartProperty("name", username != null ? username : "<new user>");
                setPartName(username != null ? username : "<new user>");
        }
 
        protected void addPages() {
                try {
-                       addPage(new DefaultUserMainPage(this, securityService, user));
+                       addPage(new DefaultUserMainPage(this, userAdminService, user));
 
                } catch (PartInitException e) {
                        throw new ArgeoException("Not able to add page ", e);
@@ -48,10 +48,10 @@ public class ArgeoUserEditor extends FormEditor {
                // TODO: make it more generic
                findPage(DefaultUserMainPage.ID).doSave(monitor);
 
-               if (securityService.userExists(user.getUsername()))
-                       securityService.updateUser(user);
+               if (userAdminService.userExists(user.getUsername()))
+                       userAdminService.updateUser(user);
                else {
-                       securityService.newUser(user);
+                       userAdminService.newUser(user);
                        setPartName(user.getUsername());
                }
                firePropertyChange(PROP_DIRTY);
@@ -66,7 +66,7 @@ public class ArgeoUserEditor extends FormEditor {
                return false;
        }
 
-       public void setSecurityService(ArgeoSecurityService securityService) {
-               this.securityService = securityService;
+       public void setUserAdminService(UserAdminService userAdminService) {
+               this.userAdminService = userAdminService;
        }
 }
index e4761234ad7c7de8dd7abe658c3eabe169247647..ee36b573125f1d7246de9780d48b19e594b0438c 100644 (file)
@@ -2,9 +2,9 @@ package org.argeo.security.ui.admin.editors;
 
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
-import org.argeo.security.ArgeoSecurityService;
 import org.argeo.security.ArgeoUser;
 import org.argeo.security.SimpleArgeoUser;
+import org.argeo.security.UserAdminService;
 import org.argeo.security.nature.SimpleUserNature;
 import org.argeo.security.ui.admin.SecurityAdminPlugin;
 import org.eclipse.jface.viewers.CellEditor;
@@ -49,10 +49,10 @@ public class DefaultUserMainPage extends FormPage {
        private ArgeoUser user;
        private SimpleUserNature simpleNature;
        private String simpleNatureType;
-       private ArgeoSecurityService securityService;
+       private UserAdminService securityService;
 
        public DefaultUserMainPage(FormEditor editor,
-                       ArgeoSecurityService securityService, ArgeoUser user) {
+                       UserAdminService securityService, ArgeoUser user) {
                super(editor, ID, "Main");
                this.securityService = securityService;
                this.user = user;
index 8a7e5ec82ff7f1ddc34267e4a7d007113a82528e..e2779590a1856fbbffc1e8c087f42fd9a7cbb519 100644 (file)
@@ -3,8 +3,8 @@ package org.argeo.security.ui.admin.views;
 import java.util.ArrayList;
 
 import org.argeo.ArgeoException;
-import org.argeo.security.ArgeoSecurityService;
 import org.argeo.security.ArgeoUser;
+import org.argeo.security.UserAdminService;
 import org.argeo.security.ui.admin.SecurityAdminPlugin;
 import org.argeo.security.ui.admin.commands.AddRole;
 import org.argeo.security.ui.admin.commands.OpenArgeoUserEditor;
@@ -37,12 +37,12 @@ import org.eclipse.ui.part.ViewPart;
 
 /** List all roles. */
 public class RolesView extends ViewPart {
-       public final static String ID = "org.argeo.security.ui.rolesView";
+       public final static String ID = "org.argeo.security.ui.admin.adminRolesView";
 
        private Text newRole;
 
        private TableViewer viewer;
-       private ArgeoSecurityService securityService;
+       private UserAdminService userAdminService;
 
        private String addNewRoleText = "<add new role here>";
 
@@ -92,8 +92,8 @@ public class RolesView extends ViewPart {
                viewer.getTable().setFocus();
        }
 
-       public void setSecurityService(ArgeoSecurityService securityService) {
-               this.securityService = securityService;
+       public void setUserAdminService(UserAdminService userAdminService) {
+               this.userAdminService = userAdminService;
        }
 
        public String getAddNewRoleText() {
@@ -103,7 +103,7 @@ public class RolesView extends ViewPart {
        private class RolesContentProvider implements IStructuredContentProvider {
 
                public Object[] getElements(Object inputElement) {
-                       return securityService.listEditableRoles().toArray();
+                       return userAdminService.listEditableRoles().toArray();
                }
 
                public void dispose() {
index 9ef5afe2bb14ce2dee48964786f37710a6b5b2db..df909b6c761b3b020e047d39b04cf2aa983929c9 100644 (file)
@@ -3,8 +3,8 @@ package org.argeo.security.ui.admin.views;
 import java.util.ArrayList;
 
 import org.argeo.ArgeoException;
-import org.argeo.security.ArgeoSecurityService;
 import org.argeo.security.ArgeoUser;
+import org.argeo.security.UserAdminService;
 import org.argeo.security.nature.SimpleUserNature;
 import org.argeo.security.ui.admin.SecurityAdminPlugin;
 import org.argeo.security.ui.admin.commands.OpenArgeoUserEditor;
@@ -33,10 +33,10 @@ import org.eclipse.ui.part.ViewPart;
 
 /** List all users. */
 public class UsersView extends ViewPart {
-       public final static String ID = "org.argeo.security.ui.usersView";
+       public final static String ID = "org.argeo.security.ui.admin.adminUsersView";
 
        private TableViewer viewer;
-       private ArgeoSecurityService securityService;
+       private UserAdminService userAdminService;
 
        private String simpleNatureType = null;
 
@@ -73,8 +73,8 @@ public class UsersView extends ViewPart {
                viewer.getTable().setFocus();
        }
 
-       public void setSecurityService(ArgeoSecurityService securityService) {
-               this.securityService = securityService;
+       public void setUserAdminService(UserAdminService userAdminService) {
+               this.userAdminService = userAdminService;
        }
 
        public void setSimpleNatureType(String simpleNatureType) {
@@ -88,7 +88,7 @@ public class UsersView extends ViewPart {
        private class UsersContentProvider implements IStructuredContentProvider {
 
                public Object[] getElements(Object inputElement) {
-                       return securityService.listUsers().toArray();
+                       return userAdminService.listUsers().toArray();
                }
 
                public void dispose() {
@@ -102,7 +102,7 @@ public class UsersView extends ViewPart {
        private class UsersLabelProvider extends LabelProvider implements
                        ITableLabelProvider {
                public String getColumnText(Object element, int columnIndex) {
-                       //String currentUsername = CurrentUser.getUsername();
+                       // String currentUsername = CurrentUser.getUsername();
                        String currentUsername = "";
                        ArgeoUser user = (ArgeoUser) element;
                        SimpleUserNature simpleNature = SimpleUserNature
diff --git a/security/plugins/org.argeo.security.ui.rap/build.properties b/security/plugins/org.argeo.security.ui.rap/build.properties
new file mode 100644 (file)
index 0000000..b840a94
--- /dev/null
@@ -0,0 +1,4 @@
+bin.includes = plugin.xml,\
+               META-INF/
+source.. = src/main/java/
+output.. = target/classes/
index 5d25238502522e7de01c9f0f1d2271ad94a2ba43..c5fe0e337628132c9b0c84871fe2c714e1400063 100644 (file)
@@ -6,6 +6,6 @@
 
        <bean id="openChangePasswordDialog" class="org.argeo.security.ui.commands.OpenChangePasswordDialog"
                scope="prototype">
-               <property name="securityService" ref="securityService" />
+               <property name="currentUserService" ref="currentUserService" />
        </bean>
 </beans>
index 6574cf125286ff0218e0ba898b69e8b05edf74d6..2477c93a8d990ee0a62e0c35632f861f87ce65c2 100644 (file)
@@ -8,5 +8,5 @@
        http://www.springframework.org/schema/beans/spring-beans-2.5.xsd"\r
        osgi:default-timeout="30000">\r
 \r
-       <reference id="securityService" interface="org.argeo.security.ArgeoSecurityService" />\r
+       <reference id="currentUserService" interface="org.argeo.security.CurrentUserService" />\r
 </beans:beans>
\ No newline at end of file
index ab1ec19c1b9e618b195646cf412ce154992b7f7e..48427089c0dfd7ca096a4034ca704e46c157209d 100644 (file)
                                <artifactId>maven-bundle-plugin</artifactId>
                                <version>${version.maven-bundle-plugin}</version>
                                <configuration>
-                                       <!-- <instructions> -->
-                                       <!-- <Bundle-ActivationPolicy>lazy</Bundle-ActivationPolicy> -->
-                                       <!-- <Bundle-Activator>org.argeo.security.ui.SecurityUiPlugin</Bundle-Activator> -->
-                                       <!-- <Require-Bundle>org.eclipse.ui;resolution:=optional,org.eclipse.rap.ui;resolution:=optional,org.eclipse.core.runtime</Require-Bundle> -->
-                                       <!-- <Import-Package> -->
-                                       <!-- org.argeo.eclipse.spring, -->
-                                       <!-- org.osgi.framework;version="0.0.0", -->
-                                       <!-- !org.eclipse.core.runtime, -->
-                                       <!-- !org.eclipse.core.commands, -->
-                                       <!-- !org.eclipse.ui.plugin, -->
-                                       <!-- !org.eclipse.ui, -->
-                                       <!-- !org.eclipse.ui.commands, -->
-                                       <!-- !org.eclipse.ui.handlers, -->
-                                       <!-- !org.eclipse.ui.part, -->
-                                       <!-- * -->
-                                       <!-- </Import-Package> -->
-                                       <!-- <Private-Package>icons</Private-Package> -->
-                                       <!-- <Export-Package>org.argeo.security.ui.*</Export-Package> -->
-                                       <!-- </instructions> -->
                                        <instructions>
                                                <Bundle-ActivationPolicy>lazy</Bundle-ActivationPolicy>
                                                <Bundle-Activator>org.argeo.security.ui.SecurityUiPlugin</Bundle-Activator>
                        <version>0.2.3-SNAPSHOT</version>
                </dependency>
 
-               <!-- Argeo Eclipse distribution (common dependencies for both RAP and RCP) -->
-               <dependency>
-                       <groupId>org.argeo.commons.eclipse</groupId>
-                       <artifactId>org.argeo.eclipse.dep.common</artifactId>
-                       <version>0.2.3-SNAPSHOT</version>
-               </dependency>
-
                <!-- RCP only dependency, needed at compile time -->
                <dependency>
                        <groupId>org.argeo.commons.eclipse</groupId>
index 70471159df1903b0cc71cab107bb696ecb1b336f..bd6da36570d0e4fa1177bcca64bc242bef353000 100644 (file)
@@ -1,6 +1,6 @@
 package org.argeo.security.ui.commands;
 
-import org.argeo.security.ArgeoSecurityService;
+import org.argeo.security.CurrentUserService;
 import org.argeo.security.ui.dialogs.ChangePasswordDialog;
 import org.eclipse.core.commands.AbstractHandler;
 import org.eclipse.core.commands.ExecutionEvent;
@@ -9,17 +9,17 @@ import org.eclipse.ui.handlers.HandlerUtil;
 
 /** Opens the change password dialog. */
 public class OpenChangePasswordDialog extends AbstractHandler {
-       private ArgeoSecurityService securityService;
+       private CurrentUserService currentUserService;
 
        public Object execute(ExecutionEvent event) throws ExecutionException {
                ChangePasswordDialog dialog = new ChangePasswordDialog(
-                               HandlerUtil.getActiveShell(event), securityService);
+                               HandlerUtil.getActiveShell(event), currentUserService);
                dialog.open();
                return null;
        }
 
-       public void setSecurityService(ArgeoSecurityService securityService) {
-               this.securityService = securityService;
+       public void setCurrentUserService(CurrentUserService currentUserService) {
+               this.currentUserService = currentUserService;
        }
 
 }
index e2e78e8145dc0154ee1b3cd4d7da96d426a3ce65..21b37cb821fba4ae2a9f001e52a0a037d7f7bb43 100644 (file)
@@ -1,7 +1,7 @@
 package org.argeo.security.ui.dialogs;
 
 import org.argeo.ArgeoException;
-import org.argeo.security.ArgeoSecurityService;
+import org.argeo.security.CurrentUserService;
 import org.eclipse.jface.dialogs.IMessageProvider;
 import org.eclipse.jface.dialogs.TitleAreaDialog;
 import org.eclipse.swt.SWT;
@@ -17,10 +17,10 @@ import org.eclipse.swt.widgets.Text;
 /** Dialog to change the current user password */
 public class ChangePasswordDialog extends TitleAreaDialog {
        private Text currentPassword, newPassword1, newPassword2;
-       private ArgeoSecurityService securityService;
+       private CurrentUserService securityService;
 
        public ChangePasswordDialog(Shell parentShell,
-                       ArgeoSecurityService securityService) {
+                       CurrentUserService securityService) {
                super(parentShell);
                this.securityService = securityService;
        }
index b9a026858b7cc1861f7ac176b0651b7136a3ce75..4ea50012f8df618dbe92735e708de2a0a38f46da 100644 (file)
@@ -14,6 +14,7 @@
        <modules>
                <module>org.argeo.security.equinox</module>
                <module>org.argeo.security.ui</module>
+               <module>org.argeo.security.ui.admin</module>
                <module>org.argeo.security.ui.rcp</module>
                <module>org.argeo.security.ui.rap</module>
        </modules>
index 5fc538bc83f35ba123c9f84f2eb6124dd65c8789..2c65f866b46d53ef4bac707441e009f4215beee0 100644 (file)
@@ -1,4 +1,5 @@
-source.. = src/main/java/
+source.. = src/main/java/,\
+           src/main/resources/
 output.. = target/classes/
 bin.includes = META-INF/,\
                .
index f389145c864e20c5b76e8cd27dbfb296b1643e7d..1a5a6cd035d2b3200f66644ea48360dfddb7c31c 100644 (file)
@@ -27,6 +27,7 @@ import org.springframework.security.context.SecurityContext;
 import org.springframework.security.context.SecurityContextHolder;
 import org.springframework.security.providers.UsernamePasswordAuthenticationToken;
 
+@SuppressWarnings("unchecked")
 public class ActiveMqSecurityBrokerPlugin extends BrokerPluginSupport {
 //     private final static Log log = LogFactory
 //                     .getLog(ActiveMqSecurityBrokerPlugin.class);
index 8288ab5d725223cd07fe18d85778579a839ad50c..c1a1d6b154b99a2c7fccd601b51a8ab7f8f3abb9 100644 (file)
@@ -72,7 +72,6 @@
     </extension>
        <extension point="org.eclipse.ui.menus">
                <menuContribution
-                allPopups="false"
                 locationURI="menu:org.argeo.jcr.ui.explorer.browserView">
                 <command
                       commandId="org.argeo.jcr.ui.explorer.openGenericJcrQueryEditor"
@@ -86,7 +85,6 @@
                             </command>
           </menuContribution>
   <menuContribution
-        allPopups="false"
         locationURI="popup:org.argeo.jcr.ui.explorer.browserView">
      <command
            commandId="org.argeo.jcr.ui.explorer.refresh"
index 87eab775601068fc0007f27cf86bcb535e806f6e..63406999634b778bba41656317bd4bcda13792ba 100644 (file)
@@ -189,7 +189,7 @@ public class BeanNodeMapper implements NodeMapper {
 
                                PropertyDescriptor pd = beanWrapper.getPropertyDescriptor(prop
                                                .getName());
-                               Class propClass = pd.getPropertyType();
+                               Class<?> propClass = pd.getPropertyType();
 
                                if (log.isTraceEnabled())
                                        log.trace("Load " + prop + ", propClass=" + propClass
@@ -222,7 +222,7 @@ public class BeanNodeMapper implements NodeMapper {
                                        continue nodes;
 
                                PropertyDescriptor pd = beanWrapper.getPropertyDescriptor(name);
-                               Class propClass = pd.getPropertyType();
+                               Class<?> propClass = pd.getPropertyType();
 
                                // objects list
                                if (propClass != null && List.class.isAssignableFrom(propClass)) {
@@ -284,7 +284,7 @@ public class BeanNodeMapper implements NodeMapper {
                                                if (classProperty.equals(key))
                                                        continue keyProps;
 
-                                               Class keyPropClass = classFromProperty(keyProp);
+                                               Class<?> keyPropClass = classFromProperty(keyProp);
                                                if (keyPropClass != null) {
                                                        Object mapValue = asObject(keyProp.getValue(),
                                                                        keyPropClass);