Techsystech
/
server-tools
mirror of https://github.com/OCA/server-tools.git
3
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[FIX] Module 'auditlog' - When updating a resource, filter IDs on 2many fields to prevent a 'name_get()' call on potential deleted resources

pull/119/head
sebalix 2015-02-09 14:36:07 +01:00
parent 9734c08260
commit 39c83e67f5
1 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
auditlog/models/rule.py
View File

@ -351,8 +351,18 @@ class auditlog_rule(models.Model):
} }
# for *2many fields, log the name_get # for *2many fields, log the name_get
if field['relation'] and '2many' in field['ttype']: if field['relation'] and '2many' in field['ttype']:
old_value_text = self.env[field['relation']].browse( # Filter IDs to prevent a 'name_get()' call on deleted resources
vals['old_value']).name_get() existing_ids = self.env[field['relation']]._search(
[('id', 'in', vals['old_value'])])
old_value_text = []
if existing_ids:
existing_values = self.env[field['relation']].browse(
existing_ids).name_get()
old_value_text.extend(existing_values)
# Deleted resources will have a 'DELETED' text representation
deleted_ids = set(vals['old_value']) - set(existing_ids)
for deleted_id in deleted_ids:
old_value_text.append((deleted_id, 'DELETED'))
vals['old_value_text'] = old_value_text vals['old_value_text'] = old_value_text
new_value_text = self.env[field['relation']].browse( new_value_text = self.env[field['relation']].browse(
vals['new_value']).name_get() vals['new_value']).name_get()