Author: brushed
Date: Sat Aug 23 13:16:03 2008
New Revision: 688407

URL: http://svn.apache.org/viewvc?rev=688407&view=rev
Log:
v2.7.0-alpha-26 [JSPWIKI-333] Side-impact GroupManager

Modified:
    incubator/jspwiki/trunk/ChangeLog
    incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/Release.java
    
incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/auth/authorize/GroupManager.java

Modified: incubator/jspwiki/trunk/ChangeLog
URL: 
http://svn.apache.org/viewvc/incubator/jspwiki/trunk/ChangeLog?rev=688407&r1=688406&r2=688407&view=diff
==============================================================================
--- incubator/jspwiki/trunk/ChangeLog (original)
+++ incubator/jspwiki/trunk/ChangeLog Sat Aug 23 13:16:03 2008
@@ -1,5 +1,13 @@
 2008-08-23 Dirk Frederickx <[EMAIL PROTECTED]>
 
+        * 2.7.0-alpha-26 
+
+        * [JSPWIKI-333] Fixed side-impact in GroupManager due to new 
+        InputValidator constructor.  
+        
+
+2008-08-23 Dirk Frederickx <[EMAIL PROTECTED]>
+
         * 2.7.0-alpha-25  Template fixes (mainly section editing)
 
         * The Section-Editing user preference can now also be set via the

Modified: incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/Release.java
URL: 
http://svn.apache.org/viewvc/incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/Release.java?rev=688407&r1=688406&r2=688407&view=diff
==============================================================================
--- incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/Release.java (original)
+++ incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/Release.java Sat Aug 23 
13:16:03 2008
@@ -77,7 +77,7 @@
      *  <p>
      *  If the build identifier is empty, it is not added.
      */
-    public static final String     BUILD         = "25";
+    public static final String     BUILD         = "26";
     
     /**
      *  This is the generic version string you should use

Modified: 
incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/auth/authorize/GroupManager.java
URL: 
http://svn.apache.org/viewvc/incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/auth/authorize/GroupManager.java?rev=688407&r1=688406&r2=688407&view=diff
==============================================================================
--- 
incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/auth/authorize/GroupManager.java 
(original)
+++ 
incubator/jspwiki/trunk/src/com/ecyrd/jspwiki/auth/authorize/GroupManager.java 
Sat Aug 23 13:16:03 2008
@@ -579,12 +579,12 @@
     public final void validateGroup( WikiContext context, Group group )
     {
         WikiSession session = context.getWikiSession();
-        InputValidator validator = new InputValidator( MESSAGES_KEY, session );
+        InputValidator validator = new InputValidator( MESSAGES_KEY, context );
 
         // Name cannot be null or one of the restricted names
         try
         {
-            checkGroupName( session, group.getName() );
+            checkGroupName( context, group.getName() );
         }
         catch( WikiSecurityException e )
         {
@@ -631,17 +631,12 @@
      * <code>null</code> or the Group name is illegal
      * @see Group#RESTRICTED_GROUPNAMES
      */
-    protected final void checkGroupName( WikiSession session, String name ) 
throws WikiSecurityException
+    protected final void checkGroupName( WikiContext context, String name ) 
throws WikiSecurityException
     {
         //TODO: groups cannot have the same name as a user
 
-        if( session == null )
-        {
-            throw new WikiSecurityException( "Session cannot be null." );
-        }
-
         // Name cannot be null
-        InputValidator validator = new InputValidator( MESSAGES_KEY, session );
+        InputValidator validator = new InputValidator( MESSAGES_KEY, context );
         validator.validateNotNull( name, "Group name" );
 
         // Name cannot be one of the restricted names either


Reply via email to