Odoo Server (MOVED TO GITHUB)

Merge lp:~therp-nl/openobject-server/trunk-lp1070884-apply_rule_after_write into lp:openobject-server

  1. trunk-lp1070884-apply_rule_after_write
  2. Merge into trunk
Proposed by Stefan Rijnhart (Opener)
Status: Work in progress
Proposed branch: lp:~therp-nl/openobject-server/trunk-lp1070884-apply_rule_after_write
Merge into: lp:openobject-server
Diff against target: 10 lines (+1/-0)
1 file modified
openerp/osv/orm.py (+1/-0)
To merge this branch: bzr merge lp:~therp-nl/openobject-server/trunk-lp1070884-apply_rule_after_write
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+131210@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

The fix appears simple enough, but it will prevent some use cases that we want to allow, as discussed on the bug report.
Setting MP status to to work in progress until the discussion is settled.
Thanks!

Reply

Unmerged revisions

4505. By Stefan Rijnhart (Opener)

[FIX] Apply record rule to result of write

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py 2012-10-22 13:27:59 +0000
+++ openerp/osv/orm.py 2012-10-24 15:20:26 +0000
@@ -4224,6 +4224,7 @@
4224 todo.append(id)4224 todo.append(id)
4225 self.pool.get(object)._store_set_values(cr, user, todo, fields_to_recompute, context)4225 self.pool.get(object)._store_set_values(cr, user, todo, fields_to_recompute, context)
42264226
4227 self.check_access_rule(cr, user, ids, 'write', context=context)
4227 self._workflow_trigger(cr, user, ids, 'trg_write', context=context)4228 self._workflow_trigger(cr, user, ids, 'trg_write', context=context)
4228 return True4229 return True
42294230