In OFS.Findsupport, line 159, I found if hasattr(ob, 'aq_base'): bs=ob.aq_base if ( (not obj_ids or absattr(bs.id) in obj_ids) [...] That is a conditional assignment to bs, an bs is used later on in any case. Shouldn't there be an else-fork, like if hasattr(ob, 'aq_base'): bs=ob.aq_base else: bs=ob if the object is accessed without an aquisition wrapper? This lead to an error with one of my products, but propably my product is still faulty, and this only reveals the above problems. Stefan
Shouldn't there be an else-fork, like
if hasattr(ob, 'aq_base'): bs=ob.aq_base else: bs=ob
if the object is accessed without an aquisition wrapper? This lead to an
error with one of my products, but propably my product is still faulty, and this only reveals the above problems.
And two hours later Brian checks in the revision =) Quick work guys! -- Martijn Pieters, Web Developer | Antraciet http://www.antraciet.nl | Tel: +31-35-6254545 Fax: +31-35-6254555 | mailto:mj@antraciet.nl http://www.antraciet.nl/~mj | PGP: http://wwwkeys.nl.pgp.net:11371/pks/lookup?op=get&search=0xA8A32149 ------------------------------------------
participants (2)
-
Martijn Pieters -
Stefan Franke