From 52afa5e307bee1199252c4819b439ab58d8cd4e6 Mon Sep 17 00:00:00 2001
From: Christophe Simonis <chs@openerp.com>
Date: Fri, 6 Aug 2010 11:44:11 +0200
Subject: [PATCH] [FIX] email_template: avoid double extension for report
 attachement

bzr revid: chs@openerp.com-20100806094411-rg9vx3ysnfwaeyji
---
 addons/email_template/email_template.py | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/addons/email_template/email_template.py b/addons/email_template/email_template.py
index 5aedea7ed72..fc7d38fde90 100644
--- a/addons/email_template/email_template.py
+++ b/addons/email_template/email_template.py
@@ -468,18 +468,18 @@ class email_template(osv.osv):
                                           data,
                                           context)
         attachment_obj = self.pool.get('ir.attachment')
+
+        fname = tools.ustr(get_value(cursor, user, record_id,
+                                     template.file_name, template, context)
+                           or 'Report')
+        ext = '.' + format
+        if not fname.endswith(ext):
+            fname += ext
+
         new_att_vals = {
             'name':mail.subject + ' (Email Attachment)',
             'datas':base64.b64encode(result),
-            'datas_fname':tools.ustr(
-                             get_value(
-                                   cursor,
-                                   user,
-                                   record_id,
-                                   template.file_name,
-                                   template,
-                                   context
-                                   ) or 'Report') + "." + format,
+            'datas_fname': fname,
             'description':mail.subject or "No Description",
             'res_model':'email_template.mailbox',
             'res_id':mail.id