← Back to branch summary

~certify-web-dev/storm/certify-staging

« back to all changes in this revision

Viewing changes to storm/references.py

  • Committer: Gustavo Niemeyer
  • Date: 2008-01-10 20:46:22 UTC
  • mfrom: (203.1.2 162528-proxy-joins)
  • Revision ID: gustavo@niemeyer.net-20080110204622-nqp6vdpsm2hphk5l
Merged 162528-proxy-joins branch [r=jkakar,radix]

Specify both of the joining tables explicitly when compiling Proxy,
so that it doesn't break due to incorrect references in the ON clause
when multiple tables are used (reported in by S3ym0ur and Hamilton Tran)

Show diffs side-by-side

added added

removed removed

Lines of Context:
storm/references.py
383
383
 
384
384
    # Inject the join between the table of the class holding the proxy
385
385
    # and the table of the class which is the target of the reference.
386
 
    left_join = LeftJoin(proxy._remote_prop.table,
 
386
    left_join = LeftJoin(proxy._reference._relation.local_cls,
 
387
                         proxy._remote_prop.table,
387
388
                         proxy._reference._relation.get_where_for_join())
388
389
    state.auto_tables.append(left_join)
389
390