Code review comment for lp:~openerp-dev/openerp-web/7.0-opw-584981-cpa
Revision history for this message
| Xavier (Open ERP) (xmo-deactivatedaccount) wrote | # |
review:
Needs Fixing
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
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?