Looks good. Is setPrimaryKey overriden in User? Curious about the cast.
Sean Legassick <[EMAIL PROTECTED]> writes:
> 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]
>
>
--
Daniel Rall <[EMAIL PROTECTED]>
------------------------------------------------------------
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?: [EMAIL PROTECTED]
