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.