From: justauser <justausr@hotmail.com>
Date: Fri, 28 Jun 2013 14:46:45 +0000 (-0400)
Subject: Changed the implementation of HOOK_QUERY_HEADLINES to deal with issue Fox raised
X-Git-Tag: 1.9~61^2~5
X-Git-Url: https://git.wh0rd.org/?a=commitdiff_plain;h=67631438353273ea260be518e627bc25500e01f2;p=tt-rss.git

Changed the implementation of HOOK_QUERY_HEADLINES to deal with issue Fox raised
---

diff --git a/classes/api.php b/classes/api.php
index 139a486e..aa092df7 100644
--- a/classes/api.php
+++ b/classes/api.php
@@ -638,7 +638,7 @@ class API extends Handler {
 			$headlines = array();
 			
 			while ($line = db_fetch_assoc($result)) {
-				
+				$line["content_preview"] = truncate_string(strip_tags($line["content_preview"]), 100);
 				foreach (PluginHost::getInstance()->get_hooks(PluginHost::HOOK_QUERY_HEADLINES) as $p) {
 					$line = $p->hook_query_headlines($line, 100, true);
 				}
@@ -669,13 +669,8 @@ class API extends Handler {
 						$headline_row['attachments'] = get_article_enclosures(
 							$line['id']);
 
-				if ($show_excerpt ){
-					if(isset($line["modified_preview"])) 
-						$excerpt = strip_tags($line["content_preview"]);
-					else
-						$excerpt = truncate_string(strip_tags($line["content_preview"]), 100);
-					$headline_row["excerpt"] = $excerpt;
-				}
+				if (!$show_excerpt )
+					$headline_row["excerpt"] = $ine["content_preview"];
 
 				if ($show_content) {
 
diff --git a/classes/feeds.php b/classes/feeds.php
index b0447fd2..44af9ae0 100644
--- a/classes/feeds.php
+++ b/classes/feeds.php
@@ -286,8 +286,9 @@ class Feeds extends Handler_Protected {
 			if ($_REQUEST["debug"]) $timing_info = print_checkpoint("PS", $timing_info);
 
 			$expand_cdm = get_pref('CDM_EXPANDED');
-
+				
 			while ($line = $this->dbh->fetch_assoc($result)) {
+				$line["content_preview"] =  "&mdash; " . truncate_string(strip_tags($line["content_preview"]),250);
 				foreach (PluginHost::getInstance()->get_hooks(PluginHost::HOOK_QUERY_HEADLINES) as $p) {
 					$line = $p->hook_query_headlines($line, 250);
 				}
@@ -364,11 +365,7 @@ class Feeds extends Handler_Protected {
 					make_local_datetime($line["date_entered"], false));
 
 				if (get_pref('SHOW_CONTENT_PREVIEW') ) {
-					if(isset($line["modified_preview"]))
-						$content_preview =  " &mdash; " .  strip_tags($line["content_preview"]);
-					else
-						$content_preview =  " &mdash; " . truncate_string(strip_tags($line["content_preview"]),
-							250);		
+						$content_preview =  $line["content_preview"];	
 				}
 
 				$score = $line["score"];
@@ -461,9 +458,7 @@ class Feeds extends Handler_Protected {
 						truncate_string($line["title"], 200);
 
 					if (get_pref('SHOW_CONTENT_PREVIEW')) {
-						if ($content_preview) {
-							$reply['content'] .= "<span class=\"contentPreview\">$content_preview</span>";
-						}
+							$reply['content'] .= "<span class=\"contentPreview\">" . $line["content_preview"] . "</span>";
 					}
 
 					$reply['content'] .= "</a></span>";
@@ -580,7 +575,7 @@ class Feeds extends Handler_Protected {
 					else
 						$excerpt_hidden = "style=\"display : none\"";
 
-					$reply['content'] .= "<span $excerpt_hidden id=\"CEXC-$id\" class=\"cdmExcerpt\">$content_preview</span>";
+					$reply['content'] .= "<span $excerpt_hidden id=\"CEXC-$id\" class=\"cdmExcerpt\">" . $line["content_preview"] . "</span>";
 
 					$reply['content'] .= "</span>";
 
diff --git a/classes/handler/public.php b/classes/handler/public.php
index 825ea5e2..08f35e21 100644
--- a/classes/handler/public.php
+++ b/classes/handler/public.php
@@ -85,7 +85,7 @@ class Handler_Public extends Handler {
 			}
 
 			$tpl->setVariable('SELF_URL', htmlspecialchars(get_self_url_prefix()), true);
-
+			$line["content_preview"] = truncate_string(strip_tags($line["content_preview"]), 100, '...');
 			while ($line = $this->dbh->fetch_assoc($result)) {
 				foreach (PluginHost::getInstance()->get_hooks(PluginHost::HOOK_QUERY_HEADLINES) as $p) {
 					$line = $p->hook_query_headlines($line);
@@ -94,11 +94,7 @@ class Handler_Public extends Handler {
 				$tpl->setVariable('ARTICLE_ID', htmlspecialchars($line['link']), true);
 				$tpl->setVariable('ARTICLE_LINK', htmlspecialchars($line['link']), true);
 				$tpl->setVariable('ARTICLE_TITLE', htmlspecialchars($line['title']), true);
-				if(isset($line["modified_preview"]))
-					$tpl->setVariable('ARTICLE_EXCERPT', strip_tags($line["content_preview"]), true);
-				else
-					$tpl->setVariable('ARTICLE_EXCERPT',
-						truncate_string(strip_tags($line["content_preview"]), 100, '...'), true);
+				$tpl->setVariable('ARTICLE_EXCERPT', $line["content_preview"], true);
 
 				$content = sanitize($line["content"], false, $owner_uid);
 
@@ -170,6 +166,7 @@ class Handler_Public extends Handler {
 			$feed['articles'] = array();
 
 			while ($line = $this->dbh->fetch_assoc($result)) {
+				$line["content_preview"]] = truncate_string(strip_tags($line["content_preview"]), 100, '...');
 				foreach (PluginHost::getInstance()->get_hooks(PluginHost::HOOK_QUERY_HEADLINES) as $p) {
 					$line = $p->hook_query_headlines($line, 100);
 				}
@@ -178,10 +175,7 @@ class Handler_Public extends Handler {
 				$article['id'] = $line['link'];
 				$article['link']	= $line['link'];
 				$article['title'] = $line['title'];
-				if(isset($line["modified_preview"]))
-					$article['excerpt'] = strip_tags($line["content_preview"]);
-				else
-					$article['excerpt'] = truncate_string(strip_tags($line["content_preview"]), 100, '...');
+				$article['excerpt'] = $line["content_preview"];
 				$article['content'] = sanitize($line["content"], false, $owner_uid);
 				$article['updated'] = date('c', strtotime($line["updated"]));
 
diff --git a/classes/pref/filters.php b/classes/pref/filters.php
index 63318033..5d19a555 100644
--- a/classes/pref/filters.php
+++ b/classes/pref/filters.php
@@ -95,7 +95,7 @@ class Pref_Filters extends Handler_Protected {
 
 		print "<div class=\"filterTestHolder\">";
 		print "<table width=\"100%\" cellspacing=\"0\" id=\"prefErrorFeedList\">";
-
+		$line["content_preview"] = strip_tags($line["content_preview"]), 100, '...');
 		while ($line = $this->dbh->fetch_assoc($result)) {
 			foreach (PluginHost::getInstance()->get_hooks(PluginHost::HOOK_QUERY_HEADLINES) as $p) {
 					$line = $p->hook_query_headlines($line, 100);
@@ -104,10 +104,7 @@ class Pref_Filters extends Handler_Protected {
 			$entry_timestamp = strtotime($line["updated"]);
 			$entry_tags = get_article_tags($line["id"], $_SESSION["uid"]);
 
-			if(isset($line["modified_preview"]))
-				$content_preview = strip_tags($line["content_preview"]);
-			else
-				$content_preview = truncate_string(strip_tags($line["content_preview"]), 100, '...');
+			$content_preview = $line["content_preview"];
 
 			if ($line["feed_title"])
 				$feed_title = $line["feed_title"];
@@ -123,7 +120,7 @@ class Pref_Filters extends Handler_Protected {
 			print "&nbsp;(";
 			print "<b>" . $feed_title . "</b>";
 			print "):&nbsp;";
-			print "<span class=\"insensitive\">" . $content_preview . "</span>";
+			print "<span class=\"insensitive\">" . $line["content_preview"] . "</span>";
 			print " " . mb_substr($line["date_entered"], 0, 16);
 
 			print "</td></tr>";