MDL-8992 Now only appearing once per page, and XHTML compliant. Also removed an ugly...

diff --git a/filter/mediaplugin/filter.php b/filter/mediaplugin/filter.php
index c9708acbc2f9ed29f2bea3839f3dfdd873186b46..ac925d48b376eed06ec947893aaf3d7fb4bd26ba 100644 (file)
--- a/filter/mediaplugin/filter.php
+++ b/filter/mediaplugin/filter.php
@@ -20,6 +20,7 @@ require_once($CFG->libdir.'/filelib.php');
 
 function mediaplugin_filter($courseid, $text) {
     global $CFG;
+    static $eolas_fix_applied = false;
 
     include 'defaultsettings.php';
 
@@ -86,7 +87,10 @@ function mediaplugin_filter($courseid, $text) {
         $newtext = $text;
     }
     
-    $newtext .= '<script defer="defer" src="' . $CFG->wwwroot . '/filter/mediaplugin/eolas_fix.js" type="text/javascript"></script>';
+    if (!$eolas_fix_applied) {
+        $newtext .= '<script defer="defer" src="' . $CFG->wwwroot . '/filter/mediaplugin/eolas_fix.js" type="text/javascript">// <![CDATA[ ]]></script>';
+        $eolas_fix_applied = true;
+    }
 
     return $newtext;
 }

diff --git a/mod/resource/type/file/resource.class.php b/mod/resource/type/file/resource.class.php
index 12fcd92ddbaf3f19f33b73eaa54a85c53ace4f03..3e523224af85dabb255b32d7cbc28d19ffab9c40 100644 (file)
--- a/mod/resource/type/file/resource.class.php
+++ b/mod/resource/type/file/resource.class.php
@@ -545,15 +545,6 @@ class resource_file extends resource_base {
                 echo '</div>';
 
             } else if ($resourcetype == "quicktime") {
-                echo '<style type="text/css">';
-                echo '/* class to hide nested objects in IE */';
-                echo '/* hides the second object from all versions of IE */';
-                echo '* html object.hiddenObjectForIE { display: none; }';
-                echo '/* display the second object only for IE5 Mac */';
-                echo '/* IE Mac \*//*/';
-                echo '* html object.hiddenObjectForIE { display: inline; }';
-                echo '/**/';
-                echo '</style>';
                 echo '<div class="resourcecontent resourceqt">';
 
                 echo '<object classid="clsid:02BF25D5-8C17-4B23-BC80-D3488ABDDC6B"';
@@ -564,13 +555,17 @@ class resource_file extends resource_base {
                 echo '<param name="controller" value="true" />';
                 echo '<param name="scale" value="aspect" />';
 
-                echo "<object class=\"hiddenObjectForIE\" type=\"video/quicktime\" data=\"$fullurl\">";
+                echo '<!--[if !IE]>-->';
+                echo "<object type=\"video/quicktime\" data=\"$fullurl\">";
                 echo '<param name="controller" value="true" />';
                 echo '<param name="autoplay" value="true" />';
                 echo '<param name="loop" value="true" />';
                 echo '<param name="scale" value="aspect" />';
-                echo '</object>';
+                echo '<!--<![endif]-->';
                 echo '<a href="' . $fullurl . '">' . $fullurl . '</a>';
+                echo '<!--[if !IE]>-->';
+                echo '</object>';
+                echo '<!--<![endif]-->';
                 echo '</object>';
                 echo '</div>';
             }  else if ($resourcetype == "flash") {