Good point, code was wrong  

That is correct:


        if ( criteria.containsKey(key) )
        {
        Criteria.Criterion criterion =
(Criteria.Criterion)criteria.getCriterion(key);
            criterion.setDB(db);
            whereClause.add( criterion.toString() ); 
        }




-----Mensaje original-----
De: John McNally [mailto:[EMAIL PROTECTED]]
Enviado el: jueves 25 de abril de 2002 2:57
Para: Turbine Users List
Asunto: Re: Pacth for Delete in BasePeer.java



> in method doDelete 
> 
> 
>                 if ( criteria.containsKey(key) )
>                 {
>                     Criteria.Criterion criterion =
> (Criteria.Criterion)criteria.getCriterion(key);
>                     criterion.setDB(db);
>                     if ( criterion.getComparison().equals(Criteria.CUSTOM)
)
>                     {
>                         whereClause.add( criterion.toString() ); 
>                     }
>                     else
>                     {
>                         whereClause.add( criterion.toString() ); 
>                     }
>                 }
> 
>

why do you have a conditional with both blocks containing the same code?

john mcnally


--
To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to