SV: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG????

2001-04-27 Thread Lennart Petersson
problem is that those days never comes... trust me :-( /L - Original Message - From: Vinay Menon <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Friday, April 27, 2001 1:15 PM Subject: Re: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG You bet. I've su

Re: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG????

2001-04-27 Thread Vinay Menon
pril 27, 2001 11:02 AM To: [EMAIL PROTECTED] Subject: Re: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG Match your bet, raise you one. :-)query.trim().charAt(0)I think we all need a holiday.___JBoss-user mailing list[EMAIL PROTECTED]http://li

Re: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG????

2001-04-27 Thread Peter Routtier-Wone
Match your bet, raise you one. :-) query.trim().charAt(0) I think we all need a holiday. ___ JBoss-user mailing list [EMAIL PROTECTED] http://lists.sourceforge.net/lists/listinfo/jboss-user

Re: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG????

2001-04-27 Thread Oleg Nitz
Hi, I love such discussions so much, that I can't keep from putting my 2 cents :-) I'd write this condition as if (query.charAt(0) == ',') { Oleg Peter Routtier-Wone wrote: > Commas aren't case sensitive. You can probably improve performance slightly > by changing this >> if (query.toL

Re: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG????

2001-04-27 Thread Peter Routtier-Wone
Commas aren't case sensitive. You can probably improve performance slightly by changing this > if (query.toLowerCase().startsWith(",")) { to this if (query.startsWith(",")) { And I'd be inclined to trim whitespace like this if (query.trim().startsWith(",")) { ___

Re: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG????

2001-04-26 Thread Toby Allsopp
ay Menon" <[EMAIL PROTECTED]> > To: <[EMAIL PROTECTED]> > Sent: Thursday, April 26, 2001 4:24 PM > Subject: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG > > > > Alright folks, > > Here is the modified code. It works a treat! __

Re: [JBoss-user] Followup - Joins - CMP Entity Beans - BUG????

2001-04-26 Thread Scott M Stark
Post your patch to http://sourceforge.net/tracker/?group_id=22866 under the Patches section. - Original Message - From: "Vinay Menon" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Thursday, April 26, 2001 4:24 PM Subject: [JBoss-user] Followup - Joins -

[JBoss-user] Followup - Joins - CMP Entity Beans - BUG????

2001-04-26 Thread Vinay Menon
Alright folks, Here is the modified code. It works a treat! if (query.toLowerCase().startsWith(",")) { //Modified by Vinay Menon StringBuffer sqlBuffer = new StringBuffer(); sqlBuffer.append("SELECT "); String primaryKeyList = getPkColumnList();

[JBoss-user] Followup - Joins - CMP Entity Beans - BUG????

2001-04-26 Thread Vinay Menon
Hi Again, Believe I have figured where the issue lies. The sql is constructed by org.jboss.ejb.plugins.jaws.jdbc.JDBCDefinedFinderCommand in the statement if (query.toLowerCase().startsWith(",")) { sql = "SELECT " + jawsEntity.getTableName()+"."+getPkColumnList() + strippedOr