I'll try your suggestions. The db.commits() are necessary because DAL is used outside of the web2py web context (using dal in script).
if targ: ... hum... yeah, you're right its a useless thing (just removed that now, thanks :)) There is no other stack trace, but I suppose I can remove the try/ except bloc and let python spit out what it really thinks. Ok, yes, I'll do that too. Thanks for the input! Mart :) On Jul 12, 8:09 pm, pbreit <pbreitenb...@gmail.com> wrote: > Hmmm..hard to say. Do you have an error trace? > > Shouldn't matter but I would conside these changes: > > targets=None > to > targets=[] > > if targets is not None: > to > if targets: > > if targets:targetsList.append(targets) > to > targetsList.append(targets) > > If you are doing "if targ:stargetsList.append(targ)", isn't "if targ" always > going to be True? And the "else" clause will never run? > > Are the db.commits necessary?