From 87b823056fade5df1376ad2fc2426b10e325cc6e Mon Sep 17 00:00:00 2001 From: "Naresh (OpenERP)" Date: Wed, 9 Nov 2011 11:39:30 +0530 Subject: [PATCH] [FIX]:remove log_id which does not have any lines to log bzr revid: nch@tinyerp.com-20111109060930-ti0gx3sed7a12329 --- addons/audittrail/audittrail.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/addons/audittrail/audittrail.py b/addons/audittrail/audittrail.py index 79277bdf0d5..49a9850a3dd 100644 --- a/addons/audittrail/audittrail.py +++ b/addons/audittrail/audittrail.py @@ -206,6 +206,7 @@ class audittrail_objects_proxy(object_proxy): model_pool = pool.get('ir.model') field_pool = pool.get('ir.model.fields') log_line_pool = pool.get('audittrail.log.line') + line_id = False for line in lines: field_obj = obj_pool._all_columns.get(line['name']) assert field_obj, _("'%s' field does not exist in '%s' model" %(line['name'], model.model)) @@ -233,6 +234,8 @@ class audittrail_objects_proxy(object_proxy): "field_description": field_obj.string } line_id = log_line_pool.create(cr, uid, vals) + if not line_id: + pool.get('audittrail.log').unlink(cr, uid, log_id) return True def start_log_process(self, cr, user_id, model, method, resource_data, pool, resource_pool):