From b4a3ad24cadba858ab963148f34f04e15fdf9b98 Mon Sep 17 00:00:00 2001
From: Dinar <gabbasov@it-projects.info>
Date: Tue, 3 Jan 2017 12:12:41 +0500
Subject: [PATCH] [FIX] support all image formats in mail_attachment_popup
 (#71)

---
 .../static/src/xml/mail_attachment_popup.xml                  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/mail_attachment_popup/static/src/xml/mail_attachment_popup.xml b/mail_attachment_popup/static/src/xml/mail_attachment_popup.xml
index 714da19..7c5af6b 100644
--- a/mail_attachment_popup/static/src/xml/mail_attachment_popup.xml
+++ b/mail_attachment_popup/static/src/xml/mail_attachment_popup.xml
@@ -2,7 +2,7 @@
 <template>
     <t t-extend="mail.Attachment">
         <t t-jquery="div[t-att-title='attachment.name'] .o_image" t-operation="replace">
-            <t t-if="attachment.mimetype == 'image/png'">
+            <t t-if="attachment.mimetype and attachment.mimetype.search('image/') !== -1">
                 <span class="m-dotted" t-attf-onclick="$('#ImageModal{{ attachment.id }}').arcticmodal()">
                     <div class="o_image" target="_blank" t-att-data-mimetype="attachment.mimetype" t-attf-data-src="/web/image/#{attachment.id}/100x80">
                         <span class='o_attachment_name'><t t-esc='attachment.name'/></span>
@@ -19,7 +19,7 @@
                     </div>
                 </div>
             </t>
-            <t t-if="attachment.mimetype != 'image/png'">
+            <t t-if="! (attachment.mimetype and attachment.mimetype.search('image/') !== -1)">
                 <a class="o_image" t-att-href='attachment.url' target="_blank" t-att-data-mimetype="attachment.mimetype" t-attf-data-src="/web/image/#{attachment.id}/100x80">
                     <span class='o_attachment_name'><t t-esc='attachment.name'/></span>
                 </a>