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
1=== modified file 'openerp/osv/orm.py'
2--- openerp/osv/orm.py 2012-10-22 13:27:59 +0000
3+++ openerp/osv/orm.py 2012-10-24 15:20:26 +0000
4@@ -4224,6 +4224,7 @@
5 todo.append(id)
6 self.pool.get(object)._store_set_values(cr, user, todo, fields_to_recompute, context)
7
8+ self.check_access_rule(cr, user, ids, 'write', context=context)
9 self._workflow_trigger(cr, user, ids, 'trg_write', context=context)
10 return True
11