Can you send me this patch so I can apply it to our (my company's) local
JBoss-2.0-FINAL sources? We are facing this bug and cannot switch (yet)
to 2.1.
++Jeff
Lennart Petersson wrote:
> I have a patch for this one and including skipping any ASC/DESC from
> being copied to the select clause.
>
> BUT!!! I can not update CVS since there is something wrong with my new
> account (see other mail about that).
>
> Please!!! Can someone tell me what is wrong with my account so that i
> can check in my patches!
> <clip from WinCvs>
> NEW CVSROOT: :pserver:[EMAIL PROTECTED]:/products/cvs/ejboss
> (password authentication)
> cvs -q login
> (Logging in to [EMAIL PROTECTED])
>
> *****CVS exited normally with code 0*****
>
> cvs -q update JDBCDefinedFinderCommand.java (in directory
> D:\cvs-sources\jboss\src\main\org\jboss\ejb\plugins\jaws\jdbc\)
> cvs update: cannot find password
> cvs [update aborted]: use "cvs login" to log in first
>
> *****CVS exited normally with code 1*****
> <end clip>
>
> /Lennart
>
> Fulco Muriglio skrev:
>
>> Hallo
>> I've a CMP with 2 fields: id and code.
>> I've configured a custom finder that should find all
>> objects sorted, so my jaws.xml has:
>>
>> ...
>> <finder>
>> <name>findSorted</name>
>> <query></query>
>> <order>id</order>
>> </finder>
>> ...
>>
>> The SQL statement (traced enabling debug variable in JDBCCommand.java)
>> produced is (Oracle datasource):
>>
>> DEBUG - JAWS - findSorted command executing: SELECT id,id FROM mytable ORDER BY id
>> DEBUG - JAWS - java.sql.SQLException: ORA-00960: ambiguous column naming in select
>list
>>
>> If I write <order>to_char(id)</order> it works, no more ambiguity for
>> Oracle but the sorted access is less efficient.
>> In my opinion a test between fields in order clauses and field of the key
>> should be added or more drastically avoid to add order in field list
>> (are there dbms that need it??):
>> in cvs 2.1:
>>
>> JDBCDefinedFinderCommand.java, line 72 (with difference 77,84,88) :
>>
>> sql = "SELECT " + jawsEntity.getTableName()+"."+getPkColumnList() +
>> (f.getOrder() == null || f.getOrder().equals("") ? "" : ","+f.getOrder()) +
>> " FROM " + jawsEntity.getTableName() + " " + query;
>>
>> should be:
>>
>> sql = "SELECT " + jawsEntity.getTableName()+"."+getPkColumnList() +
>> " FROM " + jawsEntity.getTableName() + " " + query;
>>
>> Bye
>> Fulco
>>
>> --
>> --------------------------------------------------------------
>> To subscribe: [EMAIL PROTECTED]
>> To unsubscribe: [EMAIL PROTECTED]
>> List Help?: [EMAIL PROTECTED]
>
--
--------------------------------------------------------------
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
List Help?: [EMAIL PROTECTED]