svn commit: r1429817 - in /archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo: JdoRbacManager.java JdoTool.java

[olamy]

Author: olamy
Date: Mon Jan  7 14:40:32 2013
New Revision: 1429817

URL: http://svn.apache.org/viewvc?rev=1429817&view=rev
Log:
add more generics

Modified:
    
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoRbacManager.java
    
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoTool.java

Modified: 
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoRbacManager.java
URL: 
http://svn.apache.org/viewvc/archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoRbacManager.java?rev=1429817&r1=1429816&r2=1429817&view=diff
==============================================================================
--- 
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoRbacManager.java
 (original)
+++ 
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoRbacManager.java
 Mon Jan  7 14:40:32 2013
@@ -132,7 +132,7 @@ public class JdoRbacManager
     public Role getRole( String roleName )
         throws RbacObjectNotFoundException, RbacManagerException
     {
-        return (Role) jdo.getObjectById( JdoRole.class, roleName, ROLE_DETAIL 
);
+        return jdo.getObjectById( JdoRole.class, roleName, ROLE_DETAIL );
     }
 
     /**
@@ -309,7 +309,7 @@ public class JdoRbacManager
     public Permission getPermission( String permissionName )
         throws RbacObjectNotFoundException, RbacManagerException
     {
-        return (Permission) jdo.getObjectById( JdoPermission.class, 
permissionName, null );
+        return jdo.getObjectById( JdoPermission.class, permissionName, null );
     }
 
     @SuppressWarnings( "unchecked" )
@@ -392,7 +392,7 @@ public class JdoRbacManager
     public Operation getOperation( String operationName )
         throws RbacObjectNotFoundException, RbacManagerException
     {
-        return (Operation) jdo.getObjectById( JdoOperation.class, 
operationName, null );
+        return jdo.getObjectById( JdoOperation.class, operationName, null );
     }
 
     @SuppressWarnings( "unchecked" )
@@ -477,7 +477,7 @@ public class JdoRbacManager
     public Resource getResource( String resourceIdentifier )
         throws RbacObjectNotFoundException, RbacManagerException
     {
-        return (Resource) jdo.getObjectById( JdoResource.class, 
resourceIdentifier, null );
+        return jdo.getObjectById( JdoResource.class, resourceIdentifier, null 
);
     }
 
     @SuppressWarnings( "unchecked" )
@@ -568,7 +568,7 @@ public class JdoRbacManager
     public UserAssignment getUserAssignment( String principal )
         throws RbacObjectNotFoundException, RbacManagerException
     {
-        return (UserAssignment) jdo.getObjectById( JdoUserAssignment.class, 
principal, ROLE_DETAIL );
+        return jdo.getObjectById( JdoUserAssignment.class, principal, 
ROLE_DETAIL );
     }
 
     /**

Modified: 
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoTool.java
URL: 
http://svn.apache.org/viewvc/archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoTool.java?rev=1429817&r1=1429816&r2=1429817&view=diff
==============================================================================
--- 
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoTool.java
 (original)
+++ 
archiva/redback/redback-core/trunk/redback-rbac/redback-rbac-providers/redback-rbac-jdo/src/main/java/org/apache/archiva/redback/rbac/jdo/JdoTool.java
 Mon Jan  7 14:40:32 2013
@@ -149,12 +149,12 @@ public class JdoTool
         cache.pinAll( clazz, false ); // Pin all objects of type clazz from 
now on
     }
 
-    public Object saveObject( Object object )
+    public <T>T saveObject( T object )
     {
-        return saveObject( object, null );
+        return (T) saveObject( object, null );
     }
 
-    public Object saveObject( Object object, String[] fetchGroups )
+    public <T>T saveObject( T object, String[] fetchGroups )
     {
         PersistenceManager pm = getPersistenceManager();
         Transaction tx = pm.currentTransaction();
@@ -180,7 +180,7 @@ public class JdoTool
 
             pm.makePersistent( object );
 
-            object = pm.detachCopy( object );
+            object = (T) pm.detachCopy( object );
 
             tx.commit();
 
@@ -289,7 +289,7 @@ public class JdoTool
         }
     }
 
-    public Object getObjectById( Class<?> clazz, String id, String fetchGroup )
+    public <T>T getObjectById( Class<T> clazz, String id, String fetchGroup )
         throws RbacObjectNotFoundException, RbacManagerException
     {
         if ( StringUtils.isEmpty( id ) )
@@ -318,7 +318,7 @@ public class JdoTool
 
             tx.commit();
 
-            return object;
+            return (T) object;
         }
         catch ( JDOObjectNotFoundException e )
         {
@@ -355,7 +355,7 @@ public class JdoTool
         }
     }
 
-    public Object removeObject( Object o )
+    public <T>T removeObject( T o )
         throws RbacManagerException
     {
         if ( o == null )
@@ -370,7 +370,7 @@ public class JdoTool
         {
             tx.begin();
 
-            o = pm.getObjectById( pm.getObjectId( o ) );
+            o = (T) pm.getObjectById( pm.getObjectId( o ) );
 
             pm.deletePersistent( o );