Hi,

Here is a better patch for this;

RCS file: 
/products/cvs/turbine/turbine/src/java/org/apache/turbine/om/user/TurbineUser.java,v
retrieving revision 1.3
diff -r1.3 TurbineUser.java
543a544,547
>     /**
>     * saves this object to the persistent storage - handles
>     * inserts and updates
>     */
547a552,557
>
>         // is this a newUser - default to false,
>       boolean newUser = false;
>       if (getId() == NEW_ID)
>           newUser = true; // need to do an insert
>
550,552c560,563
<             Hashtable permData = (Hashtable) getPermStorage().clone();
<             Criteria criteria = new Criteria();
<             criteria.add( TurbineUserPeer.VISITOR_ID, getId() );
---
>         Hashtable permData = (Hashtable) getPermStorage().clone();
>         Criteria criteria = new Criteria();
>
>         criteria.add( TurbineUserPeer.VISITOR_ID, getId() );
554c565,568
<             for (int i=1; i<TurbineUserPeer.columnNames.length; i++ )
---
>         for (int i=1; i<TurbineUserPeer.columnNames.length; i++ )
>         {
>           //System.err.println("["+(new java.util.Date())+"] i="+i+", 
>"+TurbineUserPeer.criteriaKeys[i]);
>             if ( permData.containsKey(TurbineUserPeer.criteriaKeys[i]) )
556,560c570,572
<                 if ( permData.containsKey(TurbineUserPeer.criteriaKeys[i]) )
<                 {
<                     criteria.add( TurbineUserPeer.criteriaKeys[i],
<                         permData.remove(TurbineUserPeer.criteriaKeys[i]) );
<                 }
---
>               //System.err.println("["+(new 
>java.util.Date())+"]"+TurbineUserPeer.criteriaKeys[i]+" = 
>"+permData.get(TurbineUserPeer.criteriaKeys[i]));
>                 criteria.add( TurbineUserPeer.criteriaKeys[i],
>                     permData.remove(TurbineUserPeer.criteriaKeys[i]) );
562c574,583
<             criteria.add( TurbineUserPeer.OBJECT_DATA, permData );
---
>         }
>         criteria.add( TurbineUserPeer.OBJECT_DATA, permData );
>
>       if (newUser)
>       {
>           //System.err.println(TurbineUserPeer.CREATED+" = "+getCreateDate());
>           criteria.add(TurbineUserPeer.CREATED,getCreateDate());
>           setId(TurbineUserPeer.doInsert(criteria));
>       }
>       else



Here's a comments patch

RCS file: 
/products/cvs/turbine/turbine/src/java/org/apache/turbine/om/user/peer/TurbineUserPeer.java,v
retrieving revision 1.16
diff -r1.16 TurbineUserPeer.java
173a174,176
>     /**
>         this method will insert a new visitor based on the supplied criteria
>     */
318c321,322
<         this method will delete a Visitor by VISITOR_ID
---
>         this method will update a Visitor by VISITOR_ID.
>       It returns the number of rows updated
---
"There is no spoon"
______________________________________________
FREE Personalized Email at Mail.com
Sign up at http://www.mail.com/?sr=signup



------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Problems?:           [EMAIL PROTECTED]

Reply via email to