From: Michael M Slusarz <slusarz@curecanti.org>
Date: Mon, 16 Aug 2010 08:20:23 +0000 (-0600)
Subject: Fix SMIL viewing
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=cda1ff2d153d1c511c8fc23ba13c5cba8cae5157;p=horde.git

Fix SMIL viewing
---

diff --git a/framework/Mime_Viewer/lib/Horde/Mime/Viewer/Smil.php b/framework/Mime_Viewer/lib/Horde/Mime/Viewer/Smil.php
index 542bc1c35..a3943b3c3 100644
--- a/framework/Mime_Viewer/lib/Horde/Mime/Viewer/Smil.php
+++ b/framework/Mime_Viewer/lib/Horde/Mime/Viewer/Smil.php
@@ -36,27 +36,26 @@ class Horde_Mime_Viewer_Smil extends Horde_Mime_Viewer_Base
     protected $_capability = array(
         'full' => true,
         'info' => false,
-        'inline' => false,
+        'inline' => true,
         'raw' => false
     );
 
     /**
-     * Metadata for the current viewer/data.
+     * Return the full rendered version of the Horde_Mime_Part object.
      *
-     * @var array
+     * @return array  See parent::render().
      */
-    protected $_metadata = array(
-        'compressed' => false,
-        'embedded' => false,
-        'forceinline' => true
-    );
+    protected function _render()
+    {
+        return $this->_renderFullReturn($this->_renderInline());
+    }
 
     /**
-     * Return the full rendered version of the Horde_Mime_Part object.
+     * Return the rendered inline version of the Horde_Mime_Part object.
      *
      * @return array  See parent::render().
      */
-    protected function _render()
+    protected function _renderInline()
     {
         $this->_content = '';
 
diff --git a/imp/lib/Mime/Viewer/Smil.php b/imp/lib/Mime/Viewer/Smil.php
index a7127fd8a..e58022863 100644
--- a/imp/lib/Mime/Viewer/Smil.php
+++ b/imp/lib/Mime/Viewer/Smil.php
@@ -28,7 +28,7 @@ class IMP_Mime_Viewer_Smil extends Horde_Mime_Viewer_Smil
         case 'IMG':
             if (isset($attrs['SRC']) &&
                 (($rp = $this->_getRelatedLink($attrs['SRC'])) !== false)) {
-                $this->_content .= '<img src="' . $this->getConfigParam('imp_contents')->urlView($rp, 'view_attach') . '" alt="" /><br />';
+                $this->_content .= '<img src="' . $this->getConfigParam('imp_contents')->urlView($rp, 'view_attach', array('params' => array('imp_img_view' => 'data'))) . '" /><br />';
             }
             break;
 
@@ -50,9 +50,11 @@ class IMP_Mime_Viewer_Smil extends Horde_Mime_Viewer_Smil
      */
     protected function _getRelatedLink($cid)
     {
-        if ($related_part = $this->findMimeType($this->_mimepart->getMimeId(), 'multipart/related') &&
-            (($key = array_search(trim($cid, '<>', $related_part->getMetadata('related_cids')))) !== false)) {
-            return $this->getConfigParam('imp_contents')->getMIMEPart($key);
+        if ($related_part = $this->getConfigParam('imp_contents')->findMimeType($this->_mimepart->getMimeId(), 'multipart/related')) {
+            $key = array_search('<' . trim($cid, '<>') . '>', $related_part->getMetadata('related_cids'));
+            if ($key !== false) {
+                return $this->getConfigParam('imp_contents')->getMIMEPart($key);
+            }
         }
 
         return false;