You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by sn...@apache.org on 2006/11/16 19:40:12 UTC

svn commit: r475853 - /incubator/roller/trunk/sandbox/planetroller/src/org/apache/roller/planet/ui/forms/GroupForm.java

Author: snoopdave
Date: Thu Nov 16 10:40:11 2006
New Revision: 475853

URL: http://svn.apache.org/viewvc?view=rev&rev=475853
Log:
Disallow non-alphanumerics in handles

Modified:
    incubator/roller/trunk/sandbox/planetroller/src/org/apache/roller/planet/ui/forms/GroupForm.java

Modified: incubator/roller/trunk/sandbox/planetroller/src/org/apache/roller/planet/ui/forms/GroupForm.java
URL: http://svn.apache.org/viewvc/incubator/roller/trunk/sandbox/planetroller/src/org/apache/roller/planet/ui/forms/GroupForm.java?view=diff&rev=475853&r1=475852&r2=475853
==============================================================================
--- incubator/roller/trunk/sandbox/planetroller/src/org/apache/roller/planet/ui/forms/GroupForm.java (original)
+++ incubator/roller/trunk/sandbox/planetroller/src/org/apache/roller/planet/ui/forms/GroupForm.java Thu Nov 16 10:40:11 2006
@@ -30,13 +30,13 @@
 import org.apache.commons.lang.StringUtils;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
-import org.apache.roller.RollerException;
 import org.apache.roller.planet.business.Planet;
 import org.apache.roller.planet.business.PlanetFactory;
 import org.apache.roller.planet.business.PlanetManager;
 import org.apache.roller.planet.pojos.PlanetGroupData;
 import org.apache.roller.planet.pojos.PlanetSubscriptionData;
 import org.apache.roller.planet.ui.utils.LoadableForm;
+import org.apache.roller.util.Utilities;
 
 /**
  * UI bean for editing group data, designed for request scope.
@@ -78,7 +78,7 @@
         if (StringUtils.isNotEmpty(group.getId())) {
             PlanetGroupData dbgroup = planet.getPlanetManager().getGroupById(group.getId());
             dbgroup.setTitle(group.getTitle());
-            dbgroup.setHandle(group.getHandle());
+            dbgroup.setHandle(Utilities.replaceNonAlphanumeric(group.getHandle()));
             dbgroup.setDescription(group.getDescription());
             planet.getPlanetManager().saveGroup(dbgroup);
         } else {