Ok, two things today:

1. Preview implementation of just the "use sqlbuilder.Select for queryForSelect"
part is in trac #291, but trac is now down. I've attached the patch (against svn
r2158) to this message too.

2. Found a fun problem with sqlbuilder.SQLJoin. Since Alias's don't have a
__sqlrepr__, SQLJoin.__init__ manually tries to construct the string
representation as AliasField.tablesUsedImmediate does. This causes problems with
the _fixTablesForJoin / Select with join logic where it's looking at "if
j.table1 in tables", because SQLJoin always puts "x AS y" and the AliasField
implementation (that ends up populating tables) is always "x y" (per
AliasField.as_string).

Any opposition to adding a __sqlrepr__/tablesUsedImmediate to Alias/AliasTable,
and having SQLJoin just sqlrepr it's components as otherwise expected?


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
sqlobject-discuss mailing list
sqlobject-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss

Reply via email to