I'd quite like DBUserManager to set the PrimaryKey of (BaseObject)user
from the return value of TurbineUserPeer.doInsert. It seems to me that
DBUserManager can validly assume that user is castable to BaseObject -
would anyone disagree?


Index: DBUserManager.java
===================================================================
RCS file: 
/products/cvs/turbine/turbine/src/java/org/apache/turbine/services/security/DBUserManager.java,v
retrieving revision 1.7
diff -u -r1.7 DBUserManager.java
--- DBUserManager.java  2001/01/05 17:08:20     1.7
+++ DBUserManager.java  2001/02/07 17:08:57
@@ -369,7 +369,7 @@
         Criteria criteria = TurbineUserPeer.buildCriteria(user);
         try
         {
-            TurbineUserPeer.doInsert(criteria);
+            ((BaseObject)user).setPrimaryKey(TurbineUserPeer.doInsert(criteria));
         }
         catch(Exception e)
         {

-- 
Sean Legassick
[EMAIL PROTECTED]
      Als Mensch kann mir nichts menschliches Fremd sein  
      
      


------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

Reply via email to