From 7f06157899073033a6c2f7bfb462a0c6849416a2 Mon Sep 17 00:00:00 2001
From: Michael M Slusarz <slusarz@curecanti.org>
Date: Tue, 2 Dec 2008 20:44:10 -0700
Subject: [PATCH] Fix some bugs in this new code.

---
 framework/Mime/lib/Horde/Mime.php | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/framework/Mime/lib/Horde/Mime.php b/framework/Mime/lib/Horde/Mime.php
index 253f90669..834daa01e 100644
--- a/framework/Mime/lib/Horde/Mime.php
+++ b/framework/Mime/lib/Horde/Mime.php
@@ -407,17 +407,17 @@ class Horde_Mime
         $ret = array('params' => array(), 'val' => '');
 
         /* Give $string a bogus body part or else decode() will complain. */
-        require 'Mail/mimeDecode.php';
+        require_once 'Mail/mimeDecode.php';
         $mime_decode = new Mail_mimeDecode($string . "\n\nA");
         $res = $mime_decode->decode();
 
         /* Are we dealing with content-type or content-disposition? */
-        if ($res->ctype_primary) {
-            $ret['val'] = $res->ctype_primary . '/' . $res->ctype_secondary;
-            $params = $res->ctype_parameters;
-        } elseif ($res->disposition) {
+        if (isset($res->disposition)) {
             $ret['val'] = $res->disposition;
-            $params = $res->d_parameters;
+            $params = isset($res->d_parameters) ? $res->d_parameters : array();
+        } elseif (isset($res->ctype_primary)) {
+            $ret['val'] = $res->ctype_primary . '/' . $res->ctype_secondary;
+            $params = isset($res->ctype_parameters) ? $res->ctype_parameters : array();
         } else {
             return $ret;
         }
@@ -447,7 +447,7 @@ class Horde_Mime
             }
         }
 
-        foreach ($convert as $name) {
+        foreach (array_keys($convert) as $name) {
             $val = $ret['params'][$name];
             $quote = strpos($val, "'");
             $orig_charset = substr($val, 0, $quote);
-- 
2.11.0