Review: Needs Fixing

Hi,

There is a problem with that current implementation, following those steps:

1. create() with valid data (new id: 1)
2. create() with valid data (new id: 2)
3. write() with invalid data (raise Error + fill "self.for_rollback")
4. write() on id: 2 with correct data (update current data)
5. create() with invalid data

Current: on step 5, _validate() will rollback for data as they were on step 3. 
- so loosing step 4. modifications
Excepted: on step 5, _validate() will rollback to data as they were on step 4.
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/6.0-opw-355340-cpa/+merge/90831
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/6.0-opw-355340-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