From b5b76d2ccb8e28183c078d242e1530508b2686ae Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Tue, 15 Jan 2013 12:18:11 -0500
Subject: [PATCH] Make sure mimetypes is an array not object in a better way

---
 program/steps/mail/show.inc |    2 +-
 program/js/app.js           |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/program/js/app.js b/program/js/app.js
index 6868686..7197609 100644
--- a/program/js/app.js
+++ b/program/js/app.js
@@ -820,7 +820,7 @@
         var qstring = '_mbox='+urlencode(this.env.mailbox)+'&_uid='+this.env.uid+'&_part='+props.part;
 
         // open attachment in frame if it's of a supported mimetype
-        if (this.env.uid && props.mimetype && this.env.mimetypes && $.inArray(props.mimetype, $.map(this.env.mimetypes, function(v,k){ return v })) >= 0) {
+        if (this.env.uid && props.mimetype && this.env.mimetypes && $.inArray(props.mimetype, this.env.mimetypes) >= 0) {
           if (props.mimetype == 'text/html')
             qstring += '&_safe=1';
           this.attachment_win = window.open(this.env.comm_path+'&_action=get&'+qstring+'&_frame=1', 'rcubemailattachment');
diff --git a/program/steps/mail/show.inc b/program/steps/mail/show.inc
index 64e6288..3495df9 100644
--- a/program/steps/mail/show.inc
+++ b/program/steps/mail/show.inc
@@ -79,7 +79,7 @@
     }
   }
 
-  $OUTPUT->set_env('mimetypes', $mimetypes);
+  $OUTPUT->set_env('mimetypes', array_values($mimetypes));
 
   if ($CONFIG['drafts_mbox'])
     $OUTPUT->set_env('drafts_mailbox', $CONFIG['drafts_mbox']);

--
Gitblit v1.9.1