The `not(... or ...)` is not equivalent to the previous condition (i.e. it will crash if objmodel is None). It should have been `not (... and ...)`.
The test for AbstractModel is better than nothing (i.e. to not discard report which have _auto to False but still exist in database) but still not enough; we need something more accurate. As we have still the merge prop. from Guewen I will reject this one and work on the other one. -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-1029344-aja/+merge/128691 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-bug-1029344-aja. _______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp
