Merge lp:~openerp-dev/openobject-addons/trunk-bug-1177840-dharmraj into lp:openobject-addons

Proposed by Dharmraj Zala(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-1177840-dharmraj
Merge into: lp:openobject-addons
Diff against target: 15 lines (+4/-1)
1 file modified
audittrail/audittrail.py (+4/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-1177840-dharmraj
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+168381@code.launchpad.net

Description of the change

Hello,

I have fixed the issue when audit rules applied on unlink method, it gives traceback.

Thanks,
Dharmraj.

To post a comment you must log in.

Unmerged revisions

8757. By Dharmraj Zala(OpenERP)

[FIX] improved the condition for audit-trail logs

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'audittrail/audittrail.py'
2--- audittrail/audittrail.py 2013-03-29 14:37:20 +0000
3+++ audittrail/audittrail.py 2013-06-10 08:41:33 +0000
4@@ -456,7 +456,10 @@
5
6 # if at least one modification has been found
7 for model_id, resource_id in lines:
8- name = pool[model.model].name_get(cr, uid, [resource_id])[0][1]
9+ if (model_id, resource_id) not in new_values and method not in ('copy', 'read'):
10+ name = old_values[(model_id, resource_id)]['value']['name']
11+ else:
12+ name = pool[model.model].name_get(cr, uid, [resource_id])[0][1]
13 vals = {
14 'method': method,
15 'object_id': model_id,

Subscribers

People subscribed via source and target branches

to all changes: