Looks good; change matches my updates. Will reconfirm with the next build, thanks!
On Mar 30, 6:01 pm, Massimo Di Pierro <[email protected]> wrote: > This was fixed today in trunk. Can you please check it? > > On Mar 30, 4:58 pm, Chris <[email protected]> wrote: > > > > > > > > > Hey all, > > > I've been tearing my hair out over this one. I just updated from 1.91 > > to 1.94 and can't get virtual fields to work properly on a join/group > > select. > > > I found where the virtual field gets added in 1.91 - gluon/dal.py/ > > BaseAdapter.parse: 1265 > > for table, virtualfields in virtualtables: > > for item in virtualfields: > > try: > > rowsobj = rowsobj.setvirtualfields(**{table:item}) > > except KeyError: > > # to avoid breaking virtualfields when partial > > select > > pass > > > The problem with the new replacement iterator is that tablename > > doesn't iterate. dal.py:1342: > > for table in virtualtables: > > for item in db[tablename].virtualfields: > > try: > > rowsobj = rowsobj.setvirtualfields(**{table:item}) > > except KeyError: > > # to avoid breaking virtualfields when partial > > select > > pass > > > I've temporarily updated that locally; can that be patched? > > > Thanks, > > Chris
