From: Michael M Slusarz <slusarz@curecanti.org>
Date: Wed, 3 Dec 2008 03:44:10 +0000 (-0700)
Subject: Fix some bugs in this new code.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=7f06157899073033a6c2f7bfb462a0c6849416a2;p=horde.git

Fix some bugs in this new code.
---

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);