You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by ag...@apache.org on 2007/06/04 03:16:18 UTC

svn commit: r544027 - in /roller/trunk: apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetManagerImpl.java tools/roller-planet/roller-planet-business.jar

Author: agilliland
Date: Sun Jun  3 18:16:17 2007
New Revision: 544027

URL: http://svn.apache.org/viewvc?view=rev&rev=544027
Log:
we can no longer use the same planet group uniqueness test via the saveGroup() method now that we are using generated ids because the group id is never null.


Modified:
    roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetManagerImpl.java
    roller/trunk/tools/roller-planet/roller-planet-business.jar

Modified: roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetManagerImpl.java
URL: http://svn.apache.org/viewvc/roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetManagerImpl.java?view=diff&rev=544027&r1=544026&r2=544027
==============================================================================
--- roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetManagerImpl.java (original)
+++ roller/trunk/apps/planet/src/java/org/apache/roller/planet/business/hibernate/HibernatePlanetManagerImpl.java Sun Jun  3 18:16:17 2007
@@ -101,13 +101,6 @@
     
     // save a Group
     public void saveGroup(PlanetGroupData group)  throws PlanetException {
-        // TODO: move this check outside this method?
-        if (group.getId() == null || getGroupById(group.getId()) == null) {
-            // If new group, make sure hadnle is unique within Planet
-            if (getGroup(group.getPlanet(), group.getHandle()) != null) {
-                throw new PlanetException("ERROR group handle already exists in Planet");
-            }
-        }
         strategy.store(group);
     }
         

Modified: roller/trunk/tools/roller-planet/roller-planet-business.jar
URL: http://svn.apache.org/viewvc/roller/trunk/tools/roller-planet/roller-planet-business.jar?view=diff&rev=544027&r1=544026&r2=544027
==============================================================================
Binary files - no diff available.