Review: Needs Fixing
- Notice how, in the many2one case, there are three cases: string, id, ids. 
Please correct your code to allow the same things,
- Same for the one2many case.
- Please add tests to openerp/addons/base/test/test_osv_expression.yml
(You can use the Customer/Segmentation/{Bad,Important} categories).

If you notice any duplication of code, please refactor it as a function/method.
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-740799-nch/+merge/61392
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/trunk-bug-740799-nch.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-web
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-web
More help   : https://help.launchpad.net/ListHelp

Reply via email to