John,
I have a tested patch, how I submit it?
----------------------------------------------------------------------------
Edgar Gonzalez Gonzalez
VALHALLA Project, s.a.
Chief Technology Officer
Web: www.valhallaproject.com
E-mail: [EMAIL PROTECTED]
Phone: +58-212-242.4379 / 6662 / 4055 / 6475
Fax: +58-212-242.6809
"The limits of my language mean the limits of my world."
Ludwig Wittgenstein
----------------------------------------------------------------------------
----- Original Message -----
From: "John McNally" <[EMAIL PROTECTED]>
To: "Turbine Users List" <[EMAIL PROTECTED]>
Sent: Wednesday, January 09, 2002 7:51 PM
Subject: Re: REPOST: Is this bug in BasePeer.createQueryString()
> Please submit a patch that you have tested.
>
> john mcnally
>
> Edgar Gonzalez wrote:
> >
> > Hi,
> >
> > I think there is a bug in BasePeer.createQueryString when it parses the
Criterions and the ignoreCase property is set.
> >
> > In line 997 the code is:
> >
>
> --------------------------------------------------------------------------
-------------------------------------
> > boolean ignorCase = (criteria.isIgnoreCase() &&
> > (dbMap.getTable(table).getColumn(
> > someCriteria[i].getColumn()).getType() instanceof
String));
> >
> > someCriteria[i].setIgnoreCase(ignorCase);
>
> --------------------------------------------------------------------------
-------------------------------------
> >
> > This code ignore any previous value of someCriteria[i].isIgnoreCase(),
and try to apply to all the Criterions the criteria.isIgnoreCase() value.
> >
> > Suppose that I want a WHERE CLAUSE like this:
> >
> > UPPER( TABLE_A.COLUMN_1 ) = UPPER('XXXX') AND TABLE_A.COLUMN_2 = 'yyyy'
> >
> > Using criterions to do this with the current code it's impossible,
because the criteria.isIgnoreCase() apply to both Criterions.
> >
> > I think one possible workaround is to consider the previous value of
someCriteria[i].isIgnoreCase(), something like:
> >
> > boolean ignorCase = (someCriteria[i].isIgnoreCase() &&
> >
> > any comment or suggestion?
> >
> > Thanks in advance
> >
> > Edgar
>
> --
> 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]>
