svn commit: r1412794 - in /archiva/redback/redback-core/trunk: redback-integrations/redback-common-integrations/src/main/java/org/apache/archiva/redback/integration/checks/security/ redback-users/redback-users-providers/redback-users-jdo/src/main/java/...

[olamy]

Author: olamy
Date: Fri Nov 23 09:29:18 2012
New Revision: 1412794

URL: http://svn.apache.org/viewvc?rev=1412794&view=rev
Log:
return updated user and un comment tested code

Modified:
    
archiva/redback/redback-core/trunk/redback-integrations/redback-common-integrations/src/main/java/org/apache/archiva/redback/integration/checks/security/LockedAdminEnvironmentCheck.java
    
archiva/redback/redback-core/trunk/redback-users/redback-users-providers/redback-users-jdo/src/main/java/org/apache/archiva/redback/users/jdo/JdoUserManager.java

Modified: 
archiva/redback/redback-core/trunk/redback-integrations/redback-common-integrations/src/main/java/org/apache/archiva/redback/integration/checks/security/LockedAdminEnvironmentCheck.java
URL: 
http://svn.apache.org/viewvc/archiva/redback/redback-core/trunk/redback-integrations/redback-common-integrations/src/main/java/org/apache/archiva/redback/integration/checks/security/LockedAdminEnvironmentCheck.java?rev=1412794&r1=1412793&r2=1412794&view=diff
==============================================================================
--- 
archiva/redback/redback-core/trunk/redback-integrations/redback-common-integrations/src/main/java/org/apache/archiva/redback/integration/checks/security/LockedAdminEnvironmentCheck.java
 (original)
+++ 
archiva/redback/redback-core/trunk/redback-integrations/redback-common-integrations/src/main/java/org/apache/archiva/redback/integration/checks/security/LockedAdminEnvironmentCheck.java
 Fri Nov 23 09:29:18 2012
@@ -71,7 +71,7 @@ public class LockedAdminEnvironmentCheck
      */
     public void validateEnvironment( List<String> violations )
     {
-        if ( !checked ) //&& !userManager.isReadOnly() )
+        if ( !checked && !userManager.isReadOnly() )
         {
             List<String> roles = new ArrayList<String>();
             roles.add( RedbackRoleConstants.SYSTEM_ADMINISTRATOR_ROLE );

Modified: 
archiva/redback/redback-core/trunk/redback-users/redback-users-providers/redback-users-jdo/src/main/java/org/apache/archiva/redback/users/jdo/JdoUserManager.java
URL: 
http://svn.apache.org/viewvc/archiva/redback/redback-core/trunk/redback-users/redback-users-providers/redback-users-jdo/src/main/java/org/apache/archiva/redback/users/jdo/JdoUserManager.java?rev=1412794&r1=1412793&r2=1412794&view=diff
==============================================================================
--- 
archiva/redback/redback-core/trunk/redback-users/redback-users-providers/redback-users-jdo/src/main/java/org/apache/archiva/redback/users/jdo/JdoUserManager.java
 (original)
+++ 
archiva/redback/redback-core/trunk/redback-users/redback-users-providers/redback-users-jdo/src/main/java/org/apache/archiva/redback/users/jdo/JdoUserManager.java
 Fri Nov 23 09:29:18 2012
@@ -21,6 +21,8 @@ package org.apache.archiva.redback.users
 
 import org.apache.archiva.redback.components.jdo.JdoFactory;
 import org.apache.archiva.redback.components.jdo.PlexusJdoUtils;
+import org.apache.archiva.redback.components.jdo.PlexusObjectNotFoundException;
+import org.apache.archiva.redback.components.jdo.PlexusStoreException;
 import org.apache.archiva.redback.policy.UserSecurityPolicy;
 import org.apache.archiva.redback.users.AbstractUserManager;
 import org.apache.archiva.redback.users.PermanentUserException;
@@ -28,8 +30,6 @@ import org.apache.archiva.redback.users.
 import org.apache.archiva.redback.users.UserManagerException;
 import org.apache.archiva.redback.users.UserNotFoundException;
 import org.apache.archiva.redback.users.UserQuery;
-import org.apache.archiva.redback.components.jdo.PlexusObjectNotFoundException;
-import org.apache.archiva.redback.components.jdo.PlexusStoreException;
 import org.codehaus.plexus.util.StringUtils;
 import org.jpox.JDOClassLoaderResolver;
 import org.springframework.stereotype.Service;
@@ -49,14 +49,13 @@ import java.util.List;
  * JdoUserManager
  *
  * @author <a href="mailto:joa...@erdfelt.com";>Joakim Erdfelt</a>
- *
  */
-@Service( "userManager#jdo" )
+@Service ("userManager#jdo")
 public class JdoUserManager
     extends AbstractUserManager
 {
     @Inject
-    @Named( value = "jdoFactory#users" )
+    @Named (value = "jdoFactory#users")
     private JdoFactory jdoFactory;
 
     @Inject
@@ -105,7 +104,7 @@ public class JdoUserManager
         return getAllObjectsDetached( ordering );
     }
 
-    @SuppressWarnings( "unchecked" )
+    @SuppressWarnings ("unchecked")
     private List<User> getAllObjectsDetached( String ordering )
     {
         return PlexusJdoUtils.getAllObjectsDetached( getPersistenceManager(), 
JdoUser.class, ordering, (String) null );
@@ -126,7 +125,7 @@ public class JdoUserManager
         return findUsers( "email", emailKey, orderAscending );
     }
 
-    @SuppressWarnings( "unchecked" )
+    @SuppressWarnings ("unchecked")
     public List<User> findUsersByQuery( UserQuery userQuery )
     {
         JdoUserQuery uq = (JdoUserQuery) userQuery;
@@ -170,7 +169,7 @@ public class JdoUserManager
         }
     }
 
-    @SuppressWarnings( "unchecked" )
+    @SuppressWarnings ("unchecked")
     private List<User> findUsers( String searchField, String searchKey, 
boolean ascendingUsername )
     {
         PersistenceManager pm = getPersistenceManager();
@@ -377,7 +376,7 @@ public class JdoUserManager
             userSecurityPolicy.extensionChangePassword( user, 
passwordChangeRequired );
         }
 
-        updateObject( user );
+        user = (User) updateObject( user );
 
         fireUserManagerUserUpdated( user );