On Sun, Aug 14, 2005 at 08:31:49PM +0200, Ola Lundqvist wrote:
> On Sun, Aug 14, 2005 at 05:55:24PM +0200, Laurent Fousse wrote:

>> There are some more occurrences of the problem in the source code. On
>> our system one user reported seeing the warning for
>> lib/MIME/Viewer.php, line 64. Adjusting priority as this no longer
>> prevents a successful login.

> Ok. I'll see what I can do. Patch for this is of course welcome.

Here is a patch, applying the same fix as for the other
occurrences. As I don't know in what situation the user saw that
warning, I cannot formally test it.

-- 
Lionel
--- Viewer.php.deb      2005-08-15 13:49:32.987860274 +0200
+++ Viewer.php  2005-08-15 13:51:23.370048759 +0200
@@ -61,7 +61,8 @@
         @include_once MIME_Viewer::resolveDriver($driver, $app);
         $class = (($app == 'horde') ? '' : $app . '_') . 'MIME_Viewer_' . 
$driver;
         if (class_exists($class)) {
-            return new $class($mime_part, $mime_drivers[$app][$driver]);
+            $result = new $class($mime_part, $mime_drivers[$app][$driver]);
+            return $result;
         } else {
             return false;
         }

Reply via email to