Author: dward
Date: 2008-02-08 18:16:00 +0100 (Fri, 08 Feb 2008)
New Revision: 7407

Modified:
   
xwiki-products/curriki/trunk/plugins/currikispacemanager/src/main/java/org/curriki/plugin/spacemanager/impl/CurrikiSpaceManagerExtension.java
Log:
- CURRIKI-1566
 - getRoleGroupName now expects role page name, not a pretty name (consistent 
with how spaceName works, we shouldn't be using pretty names).

Modified: 
xwiki-products/curriki/trunk/plugins/currikispacemanager/src/main/java/org/curriki/plugin/spacemanager/impl/CurrikiSpaceManagerExtension.java
===================================================================
--- 
xwiki-products/curriki/trunk/plugins/currikispacemanager/src/main/java/org/curriki/plugin/spacemanager/impl/CurrikiSpaceManagerExtension.java
       2008-02-08 16:55:31 UTC (rev 7406)
+++ 
xwiki-products/curriki/trunk/plugins/currikispacemanager/src/main/java/org/curriki/plugin/spacemanager/impl/CurrikiSpaceManagerExtension.java
       2008-02-08 17:16:00 UTC (rev 7407)
@@ -97,4 +97,17 @@
                        throw new SpaceManagerException(e);
                }
        }
+
+    /**
+     * Gets the full page name of the role
+     *
+     * @param spaceName The space name that the role is in
+     * @param role The page name of the role we want to use
+     * @return The full page name of the role
+     *
+     * TODO: This should probably check to see if the role is valid
+     */
+    public String getRoleGroupName(String spaceName, String role) {
+        return (role.startsWith(spaceName+".")?role:spaceName+"."+role);
+    }
 }

_______________________________________________
notifications mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/notifications

Reply via email to