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