Author: rwatler
Date: Tue Jun 29 19:42:43 2010
New Revision: 959079

URL: http://svn.apache.org/viewvc?rev=959079&view=rev
Log:
Jetspeed Migration: filter orphan web application security roles from registry 
on migration

Modified:
    
portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/tools/migration/JetspeedRegistryMigration.java

Modified: 
portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/tools/migration/JetspeedRegistryMigration.java
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/tools/migration/JetspeedRegistryMigration.java?rev=959079&r1=959078&r2=959079&view=diff
==============================================================================
--- 
portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/tools/migration/JetspeedRegistryMigration.java
 (original)
+++ 
portals/jetspeed-2/portal/trunk/components/jetspeed-registry/src/main/java/org/apache/jetspeed/tools/migration/JetspeedRegistryMigration.java
 Tue Jun 29 19:42:43 2010
@@ -588,36 +588,38 @@ public class JetspeedRegistryMigration i
                     {
                         contextPath = webApplicationResultSet.getString(1);
                     }
-                    else
-                    {
-                        throw new SQLException("Unable to find web application 
for id: "+webApplicationId);
-                    }
                     webApplicationResultSet.close();
                     
                     int applicationId = 0;
-                    applicationQueryStatement.setString(1, contextPath);
-                    ResultSet applicationResultSet = 
applicationQueryStatement.executeQuery();
-                    if (applicationResultSet.next())
+                    if (contextPath != null)
                     {
-                        applicationId = applicationResultSet.getInt(1);
+                        applicationQueryStatement.setString(1, contextPath);
+                        ResultSet applicationResultSet = 
applicationQueryStatement.executeQuery();
+                        if (applicationResultSet.next())
+                        {
+                            applicationId = applicationResultSet.getInt(1);
+                        }
+                        applicationResultSet.close();
                     }
-                    else
+     
+                    if ((contextPath != null) && (applicationId != 0))
                     {
-                        throw new SQLException("Unable to find portlet 
application for context: "+contextPath);
-                    }
-                    applicationResultSet.close();
-                    
-                    securityRoleInsertStatement.setInt(1, 
securityRoleResultSet.getInt(1));
-                    securityRoleInsertStatement.setInt(2, applicationId);
-                    securityRoleInsertStatement.setString(3, 
securityRoleResultSet.getString(3));
-                    securityRoleInsertStatement.executeUpdate();
-                    rowsMigrated++;
+                        securityRoleInsertStatement.setInt(1, 
securityRoleResultSet.getInt(1));
+                        securityRoleInsertStatement.setInt(2, applicationId);
+                        securityRoleInsertStatement.setString(3, 
securityRoleResultSet.getString(3));
+                        securityRoleInsertStatement.executeUpdate();
+                        rowsMigrated++;
 
-                    int securityRoleId = securityRoleResultSet.getInt(1);
-                    String description = securityRoleResultSet.getString(4);
-                    if (description != null)
+                        int securityRoleId = securityRoleResultSet.getInt(1);
+                        String description = 
securityRoleResultSet.getString(4);
+                        if (description != null)
+                        {
+                            localizedDescriptions.add(new 
LocalizedDescription(securityRoleId, 
"org.apache.jetspeed.om.portlet.impl.SecurityRoleImpl", description));
+                        }
+                    }
+                    else
                     {
-                        localizedDescriptions.add(new 
LocalizedDescription(securityRoleId, 
"org.apache.jetspeed.om.portlet.impl.SecurityRoleImpl", description));
+                        rowsDropped++;
                     }
                 }
                 applicationQueryStatement.close();



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to