got it. :-)
On May 27, 8:15 am, Nico de Groot <[email protected]> wrote: > Hi Massimo, > > Same problem exists in SQLserver (not surprising, uses the same code). > I think that arguably the best solution would be to change the current > JOIN X,Y ... WHERE [join condition] into X INNER JOIN Y ON [join > condition]. That would be a bigger patch... > > For now I've used the patch by Alexey successfully (stripped of the > full_table_name function, as suggested). I don't know if he has send > you the patch already but I'll send it by separate mail. I'll test it > further. > > Nico de Groot > > On 11 mei, 16:27, mdipierro <[email protected]> wrote: > > > Thank you. will look into it. > > > Massimo > > > On May 11, 6:31 am, Alexey Nezhdanov <[email protected]> wrote: > > > > Hi. > > > I've come across another subtle problem with web2py's DAL running over IBM > > > DB2 database. > > > DB2 requires thelefttable to immidiatedly preceed the 'leftouterjoin' > > > keyword. > > > IOW: > > > select * from a, bleftouterjoinc on c.a_id=a.id > > > is incorrect. Correct form is: > > > select * from b, aleftouterjoinc on c.a_id=a.id > > > it looks moronish enough, but alas, we have to live with it. > > > > I have modified my local copy of web2py to satisfy this requirement. > > > Please note that this sample can not be copypasted into web2py's sql.py as > > > is because of full_table_name wrapper that I added to solve another > > > problem > > > - schemas support. But replacing 'full_table_name(db, t)' with just 't' > > > should work just fine. > > > > join= attributes['left'] > > > command = self._db._translator['leftjoin'] > > > if not isinstance(join, (tuple, list)): > > > join= [join] > > > joint = [t._tablename for t injoinif not isinstance(t, > > > SQLJoin)] > > > joinon = [t for t injoinif isinstance(t, SQLJoin)] > > > tables_to_merge={} > > > > [tables_to_merge.update(dict.fromkeys(parse_tablenames(str(t.query)))) > > > for t > > > in joinon] > > > joinont = [t.table._tablename for t in joinon] > > > [tables_to_merge.pop(t) for t in joinont if t in > > > tables_to_merge] > > > important_tablenames = joint + joinont + > > > tables_to_merge.keys() > > > excluded = [t for t in tablenames if not t in > > > important_tablenames ] > > > sql_t = ', '.join([full_table_name(self._db, t) for t in > > > excluded + tables_to_merge.keys()]) > > > if joint: > > > sql_t += ' %s %s' % (command, ', > > > '.join([full_table_name(self._db, t) for t in joint])) > > > for t in joinon: > > > sql_t += ' %s %s' % (command, str(t)) > > > > Regards > > > Alexey- Tekst uit oorspronkelijk bericht niet weergeven - > > > - Tekst uit oorspronkelijk bericht weergeven -
