From de9342fac47c11489cab7dec1d0927a140c6e9ba Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=A9rome=20Maes?= Date: Thu, 25 Feb 2016 15:38:04 +0100 Subject: [PATCH] [FIX] ir_attachment : potential buggy comma Copy/paste mistake and the comma stays at the end of the line. This causes a wrong affectation : indeed, `index_content` gets a tuple with the desired value. This does not cause crashes, and the representation of the tuple is correctly stored in database, but this is not the expected behavior. --- openerp/addons/base/ir/ir_attachment.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openerp/addons/base/ir/ir_attachment.py b/openerp/addons/base/ir/ir_attachment.py index 18311e0ce53..2488726ff48 100644 --- a/openerp/addons/base/ir/ir_attachment.py +++ b/openerp/addons/base/ir/ir_attachment.py @@ -165,7 +165,7 @@ class ir_attachment(osv.osv): fname_to_delete = attach.store_fname location = self._storage(cr, uid, context) # compute the index_content field - vals['index_content'] = self._index(cr, SUPERUSER_ID, bin_data, attach.datas_fname, attach.mimetype), + vals['index_content'] = self._index(cr, SUPERUSER_ID, bin_data, attach.datas_fname, attach.mimetype) if location != 'db': # create the file fname = self._file_write(cr, uid, value, checksum)