From 51682b23305e9e6c613c74b4b2bcbcd24a8e906f Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@bah.org.ru>
Date: Tue, 6 Oct 2009 12:58:57 +0400
Subject: [PATCH] move inline player generation code into separate function;
 disable support for native html5 player due to unsolved usability reasons
 (closes #246)

---
 functions.php | 78 ++++++++++++++++++++++++++-------------------------
 viewfeed.js   |  2 +-
 2 files changed, 41 insertions(+), 39 deletions(-)

diff --git a/functions.php b/functions.php
index b0904c1ff..32a5b2ec3 100644
--- a/functions.php
+++ b/functions.php
@@ -4519,6 +4519,42 @@
 		return vsprintf(__(array_shift($args)), $args);
 	}
 
+	function format_inline_player($link, $url, $ctype) {
+
+		$entry = "";
+
+		if (($ctype == __("audio/mpeg")) &&  (get_pref($link, "ENABLE_FLASH_PLAYER")) ) {  
+                     
+			$entry .= "<object type=\"application/x-shockwave-flash\" 
+				data=\"extras/button/musicplayer.swf?song_url=$url\" 
+				width=\"17\" height=\"17\"> 
+					<param name=\"movie\" value=\"extras/button/musicplayer.swf?song_url=$url\" /> </object>";  
+		}
+
+		/*
+
+		if (substr($ctype,0,6)=="audio/" || $ctype=="application/ogg" || $ctype=="application/x-ogg") {
+			$entry .= "<audio controls=\"controls\"><source src=\"$url\" type=\"$ctype\" />";
+			if (($ctype == __("audio/mpeg")) && 
+				(get_pref($link, "ENABLE_FLASH_PLAYER")) ) { 
+				$entry .= "<span><object type=\"application/x-shockwave-flash\" data=\"extras/button/musicplayer.swf?song_url=$url\" width=\"17\" height=\"17\"> <param name=\"movie\" value=\"extras/button/musicplayer.swf?song_url=$url\" /> </object></span>";
+			}
+			$entry .= "</audio> ";
+			if (($ctype == __("audio/mpeg")) && 
+				(get_pref($link, "ENABLE_FLASH_PLAYER")) ) {
+				$entry .= "<a id='switchToFlashLink' href='#' onclick='return switchToFlash(this)'>".__('Switch to Flash Player')."</a>";	
+				$entry .= "<script type='text/javascript'>html5AudioOrFlash('$ctype');</script>"; 
+			}
+		} elseif (substr($ctype,0,6)=="video/") {
+			$entry .= "<video controls=\"controls\"><source src=\"$url\" type=\"$ctype\" />";
+			$entry .= "</video>";
+		} */
+
+
+
+		return $entry;
+	}
+
 	function outputArticleXML($link, $id, $feed_id, $mark_as_read = true,
 		$zoom_mode = false) {
 
@@ -4706,26 +4742,9 @@
 
 					$filename = substr($url, strrpos($url, "/")+1);
 
-					$entry = ""; 
-					
-					if (substr($ctype,0,6)=="audio/" || $ctype=="application/ogg" || $ctype=="application/x-ogg") {
-						$entry .= "<audio controls=\"controls\"><source src=\"$url\" type=\"$ctype\" />";
-						if (($ctype == __("audio/mpeg")) && 
-							(get_pref($link, "ENABLE_FLASH_PLAYER")) ) { 
-							$entry .= "<span><object type=\"application/x-shockwave-flash\" data=\"extras/button/musicplayer.swf?song_url=$url\" width=\"17\" height=\"17\"> <param name=\"movie\" value=\"extras/button/musicplayer.swf?song_url=$url\" /> </object></span>";
-						}
-						$entry .= "</audio> ";
-						if (($ctype == __("audio/mpeg")) && 
-							(get_pref($link, "ENABLE_FLASH_PLAYER")) ) {
-							$entry .= "<a id='switchToFlashLink' href='#' onclick='return switchToFlash(this)'>".__('Switch to Flash Player')."</a>";	
-							$entry .= "<script type='text/javascript'>html5AudioOrFlash('$ctype');</script>"; 
-						}
-					} elseif (substr($ctype,0,6)=="video/") {
-						$entry .= "<video controls=\"controls\"><source src=\"$url\" type=\"$ctype\" />";
-						$entry .= "</video>";
-					}
+					$entry = format_inline_player($link, $url, $ctype);
 
-					$entry .= "<a target=\"_blank\" href=\"" . htmlspecialchars($url) . "\">" .
+					$entry .= " <a target=\"_blank\" href=\"" . htmlspecialchars($url) . "\">" .
 						$filename . " (" . $ctype . ")" . "</a>";
 
 					array_push($entries_html, $entry);
@@ -5275,26 +5294,9 @@
 
 					$filename = substr($url, strrpos($url, "/")+1);
 
-					$entry = ""; 
-					
-					if (substr($ctype,0,6)=="audio/" || $ctype=="application/ogg" || $ctype=="application/x-ogg") {
-						$entry .= "<audio controls=\"controls\"><source src=\"$url\" type=\"$ctype\" />";
-						if (($ctype == __("audio/mpeg")) && 
-							(get_pref($link, "ENABLE_FLASH_PLAYER")) ) { 
-							$entry .= "<span><object type=\"application/x-shockwave-flash\" data=\"extras/button/musicplayer.swf?song_url=$url\" width=\"17\" height=\"17\"> <param name=\"movie\" value=\"extras/button/musicplayer.swf?song_url=$url\" /> </object></span>";
-						}
-						$entry .= "</audio> ";
-						if (($ctype == __("audio/mpeg")) && 
-							(get_pref($link, "ENABLE_FLASH_PLAYER")) ) {
-							$entry .= "<a id='switchToFlashLink' href='#' onclick='return switchToFlash(this)'>".__('Switch to Flash Player')."</a>";	
-							$entry .= "<script type='text/javascript'>html5AudioOrFlash('$ctype');</script>"; 
-						}
-					} elseif (substr($ctype,0,6)=="video/") {
-						$entry .= "<video controls=\"controls\"><source src=\"$url\" type=\"$ctype\" />";
-						$entry .= "</video>";
-					}
+					$entry = format_inline_player($link, $url, $ctype);
 
-					$entry .= "<a target=\"_blank\" href=\"" . htmlspecialchars($url) . "\">" .
+					$entry .= " <a target=\"_blank\" href=\"" . htmlspecialchars($url) . "\">" .
 						$filename . " (" . $ctype . ")" . "</a>";
 
 					array_push($entries_html, $entry);
diff --git a/viewfeed.js b/viewfeed.js
index 429412ba4..227aa87a5 100644
--- a/viewfeed.js
+++ b/viewfeed.js
@@ -278,7 +278,7 @@ function render_article(article) {
 		
 		fi.innerHTML = article;
 		
-		article.evalScripts();		
+//		article.evalScripts();		
 
 	} catch (e) {
 		exception_error("render_article", e);