Review: Needs Fixing

This is not the right place to perform this operation, if it should be 
performed at all: at worst it should be performed in the editor's `cancel` 
method, but at best (and more importantly) wouldn't the actual problem more 
likely be in the way the O2M field (or its component One2ManyListView) checks 
for its own validity?
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/7.0-opw-584981-cpa/+merge/145602
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/7.0-opw-584981-cpa.

_______________________________________________
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

Reply via email to