send info on db, version, and driver. -----Original Message----- From: Jakob Braeuchi [mailto:[EMAIL PROTECTED]] Sent: Friday, November 22, 2002 8:50 AM To: OJB Users List Subject: Re: [SQLGenerator::appendOrderByClause()] 0.9.7 Generates 'ORDER BY <field-number1>, <field-number2>, ...'
hi corey, does your database support the numbers ? jakob ----- Original Message ----- From: "Corey Klaasmeyer" <[EMAIL PROTECTED]> To: "'OJB Users List'" <[EMAIL PROTECTED]> Sent: Friday, November 22, 2002 5:29 PM Subject: [SQLGenerator::appendOrderByClause()] 0.9.7 Generates 'ORDER BY <field-number1>, <field-number2>, ...' > I believe these should be field names *not* field numbers -- at least, field > numbers do not work with my driver. Here is the code: > > for (int i = 0; i < orderByColumnNumbers.length; i++) > { > cf = (Criteria.Field) orderByFields.get(i); > if (i > 0) > { > buf.append(","); > } > > this --> buf.append(orderByColumnNumbers[i]); > > should be --> FieldDescriptor fd = > getRoot().cld.getFieldDescriptorByName(cf.name); > buf.append(fd.getColumnName()); > > if (!cf.isAscending) > { > buf.append(" DESC"); > } > } > > This works for me, but I'm not sure whether this will scope column names > properly in the case of joins. > > Am I missing something here? Is this is just a deficiency in my Driver? > > Corey > > -- > 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]> -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>