svn commit: r574805 - in /maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web: check/RoleExistanceEnvironmentCheck.java startup/ConfigurationSynchronization.java

[brett]

Author: brett
Date: Tue Sep 11 23:56:33 2007
New Revision: 574805

URL: http://svn.apache.org/viewvc?rev=574805&view=rev
Log:
[MRM-462] some more clean up

Modified:
    
maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/check/RoleExistanceEnvironmentCheck.java
    
maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/ConfigurationSynchronization.java

Modified: 
maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/check/RoleExistanceEnvironmentCheck.java
URL: 
http://svn.apache.org/viewvc/maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/check/RoleExistanceEnvironmentCheck.java?rev=574805&r1=574804&r2=574805&view=diff
==============================================================================
--- 
maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/check/RoleExistanceEnvironmentCheck.java
 (original)
+++ 
maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/check/RoleExistanceEnvironmentCheck.java
 Tue Sep 11 23:56:33 2007
@@ -19,16 +19,15 @@
  * under the License.
  */
 
+import org.apache.maven.archiva.configuration.ArchivaConfiguration;
+import org.apache.maven.archiva.configuration.Configuration;
+import org.apache.maven.archiva.configuration.ManagedRepositoryConfiguration;
 import org.apache.maven.archiva.database.ArchivaDAO;
-import org.apache.maven.archiva.database.ArchivaDatabaseException;
-import org.apache.maven.archiva.database.ObjectNotFoundException;
-import org.apache.maven.archiva.model.ArchivaRepository;
 import org.codehaus.plexus.logging.AbstractLogEnabled;
 import org.codehaus.plexus.redback.role.RoleManager;
 import org.codehaus.plexus.redback.role.RoleManagerException;
 import org.codehaus.plexus.redback.system.check.EnvironmentCheck;
 
-import java.util.Iterator;
 import java.util.List;
 
 /**
@@ -57,6 +56,11 @@
      */
     private RoleManager roleManager;
 
+    /**
+     * @plexus.requirement
+     */
+    private ArchivaConfiguration configuration;
+
     private boolean checked;
 
     public void validateEnvironment( List list )
@@ -65,14 +69,9 @@
         {
             try
             {
-                List repos = dao.getRepositoryDAO().getRepositories();
-
-                // TODO! this be skipping non-managed repos
-                Iterator it = repos.iterator();
-                while ( it.hasNext() )
+                Configuration config = configuration.getConfiguration();
+                for ( ManagedRepositoryConfiguration repository : 
config.getManagedRepositoriesAsMap().values() )
                 {
-                    ArchivaRepository repository = (ArchivaRepository) 
it.next();
-
                     if ( !roleManager.templatedRoleExists( 
"archiva-repository-manager", repository.getId() ) )
                     {
                         roleManager.createTemplatedRole( 
"archiva-repository-manager", repository.getId() );
@@ -88,17 +87,6 @@
             {
                 list.add( this.getClass().getName() + "error initializing 
roles: " + rpe.getMessage() );
                 getLogger().info( "error initializing roles", rpe );
-            }
-            catch ( ObjectNotFoundException e )
-            {
-                list.add(
-                    this.getClass().getName() + "error initializing roles 
(repository not found): " + e.getMessage() );
-                getLogger().info( "error initializing roles", e );
-            }
-            catch ( ArchivaDatabaseException e )
-            {
-                list.add( this.getClass().getName() + "error initializing 
roles (database error): " + e.getMessage() );
-                getLogger().info( "error initializing roles", e );
             }
 
             checked = true;

Modified: 
maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/ConfigurationSynchronization.java
URL: 
http://svn.apache.org/viewvc/maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/ConfigurationSynchronization.java?rev=574805&r1=574804&r2=574805&view=diff
==============================================================================
--- 
maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/ConfigurationSynchronization.java
 (original)
+++ 
maven/archiva/branches/MRM-462/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/ConfigurationSynchronization.java
 Tue Sep 11 23:56:33 2007
@@ -46,6 +46,7 @@
  * @version $Id$
  * @plexus.component 
role="org.apache.maven.archiva.web.startup.ConfigurationSynchronization"
  * role-hint="default"
+ * @todo consider whether we really need these in the database or not
  */
 public class ConfigurationSynchronization
     extends AbstractLogEnabled
@@ -68,7 +69,6 @@
 
     public void afterConfigurationChange( Registry registry, String 
propertyName, Object propertyValue )
     {
-        // TODO! this used to store both types, but do we even need it?
         if ( ConfigurationNames.isManagedRepositories( propertyName ) )
         {
             synchConfiguration( 
archivaConfiguration.getConfiguration().getManagedRepositories() );