prevent article filters from modifying article GUID; add separate plugin_data field for this

This commit is contained in:
Andrew Dolgov 2013-02-21 22:24:06 +04:00
parent 7bfb3dabdd
commit b30abdadd2
6 changed files with 39 additions and 14 deletions

View File

@ -516,12 +516,23 @@
_debug("update_rss_feed: applying plugin filters.."); _debug("update_rss_feed: applying plugin filters..");
} }
// Todo unify with id checking below
$result = db_query($link, "SELECT plugin_data FROM ttrss_entries
WHERE guid = '".db_escape_string($entry_guid)."'");
if (db_num_rows($result) != 0) {
$entry_plugin_data = db_fetch_result($result, 0, "plugin_data");
} else {
$entry_plugin_data = "";
}
$article = array("owner_uid" => $owner_uid, // read only $article = array("owner_uid" => $owner_uid, // read only
"guid" => $entry_guid, "guid" => $entry_guid, // read only
"title" => $entry_title, "title" => $entry_title,
"content" => $entry_content, "content" => $entry_content,
"link" => $entry_link, "link" => $entry_link,
"tags" => $entry_tags, "tags" => $entry_tags,
"plugin_data" => $entry_plugin_data,
"author" => $entry_author); "author" => $entry_author);
foreach ($pluginhost->get_hooks($pluginhost::HOOK_ARTICLE_FILTER) as $plugin) { foreach ($pluginhost->get_hooks($pluginhost::HOOK_ARTICLE_FILTER) as $plugin) {
@ -529,11 +540,16 @@
} }
$entry_tags = $article["tags"]; $entry_tags = $article["tags"];
$entry_guid = db_escape_string($article["guid"]); $entry_guid = db_escape_string($entry_guid);
$entry_content = db_escape_string($article["content"], false); $entry_content = db_escape_string($article["content"], false);
$entry_title = db_escape_string($article["title"]); $entry_title = db_escape_string($article["title"]);
$entry_author = db_escape_string($article["author"]); $entry_author = db_escape_string($article["author"]);
$entry_link = db_escape_string($article["link"]); $entry_link = db_escape_string($article["link"]);
$entry_plugin_data = db_escape_string($article["plugin_data"]);
if ($debug_enabled) {
_debug("update_rss_feed: plugin data: $entry_plugin_data");
}
if ($cache_images && is_writable(CACHE_DIR . '/images')) if ($cache_images && is_writable(CACHE_DIR . '/images'))
$entry_content = cache_images($entry_content, $site_url, $debug_enabled); $entry_content = cache_images($entry_content, $site_url, $debug_enabled);
@ -567,6 +583,7 @@
date_entered, date_entered,
comments, comments,
num_comments, num_comments,
plugin_data,
author) author)
VALUES VALUES
('$entry_title', ('$entry_title',
@ -581,6 +598,7 @@
NOW(), NOW(),
'$entry_comments', '$entry_comments',
'$num_comments', '$num_comments',
'$entry_plugin_data',
'$entry_author')"); '$entry_author')");
$article_labels = array(); $article_labels = array();
@ -602,7 +620,7 @@
// now it should exist, if not - bad luck then // now it should exist, if not - bad luck then
$result = db_query($link, "SELECT $result = db_query($link, "SELECT
id,content_hash,no_orig_date,title, id,content_hash,no_orig_date,title,plugin_data,
".SUBSTRING_FOR_DATE."(date_updated,1,19) as date_updated, ".SUBSTRING_FOR_DATE."(date_updated,1,19) as date_updated,
".SUBSTRING_FOR_DATE."(updated,1,19) as updated, ".SUBSTRING_FOR_DATE."(updated,1,19) as updated,
num_comments num_comments
@ -625,6 +643,7 @@
$orig_num_comments = db_fetch_result($result, 0, "num_comments"); $orig_num_comments = db_fetch_result($result, 0, "num_comments");
$orig_date_updated = strtotime(db_fetch_result($result, $orig_date_updated = strtotime(db_fetch_result($result,
0, "date_updated")); 0, "date_updated"));
$orig_plugin_data = db_fetch_result($result, 0, "plugin_data");
$ref_id = db_fetch_result($result, 0, "id"); $ref_id = db_fetch_result($result, 0, "id");
$entry_ref_id = $ref_id; $entry_ref_id = $ref_id;
@ -764,6 +783,11 @@
$update_insignificant = true; $update_insignificant = true;
} }
if ($entry_plugin_data != $orig_plugin_data) {
$post_needs_update = true;
$update_insignificant = true;
}
if ($content_hash != $orig_content_hash) { if ($content_hash != $orig_content_hash) {
$post_needs_update = true; $post_needs_update = true;
$update_insignificant = false; $update_insignificant = false;
@ -788,7 +812,8 @@
SET title = '$entry_title', content = '$entry_content', SET title = '$entry_title', content = '$entry_content',
content_hash = '$content_hash', content_hash = '$content_hash',
updated = '$entry_timestamp_fmt', updated = '$entry_timestamp_fmt',
num_comments = '$num_comments' num_comments = '$num_comments',
plugin_data = '$entry_plugin_data'
WHERE id = '$ref_id'"); WHERE id = '$ref_id'");
if (!$update_insignificant) { if (!$update_insignificant) {

View File

@ -21,7 +21,7 @@ class Af_Buttersafe extends Plugin {
$owner_uid = $article["owner_uid"]; $owner_uid = $article["owner_uid"];
if (strpos($article["guid"], "buttersafe.com") !== FALSE && if (strpos($article["guid"], "buttersafe.com") !== FALSE &&
strpos($article["guid"], "buttersafe,$owner_uid:") === FALSE) { strpos($article["plugin_data"], "buttersafe,$owner_uid:") === FALSE) {
$doc = new DOMDocument(); $doc = new DOMDocument();
@$doc->loadHTML(fetch_file_contents($article["link"])); @$doc->loadHTML(fetch_file_contents($article["link"]));
@ -48,7 +48,7 @@ class Af_Buttersafe extends Plugin {
} }
} }
$article["guid"] = "buttersafe,$owner_uid:" . $article["guid"]; $article["plugin_data"] = "buttersafe,$owner_uid:" . $article["plugin_data"];
} }
return $article; return $article;

View File

@ -21,7 +21,7 @@ class Af_Explosm extends Plugin {
$owner_uid = $article["owner_uid"]; $owner_uid = $article["owner_uid"];
if (strpos($article["link"], "explosm.net/comics") !== FALSE && if (strpos($article["link"], "explosm.net/comics") !== FALSE &&
strpos($article["guid"], "explosm,$owner_uid:") === FALSE) { strpos($article["plugin_data"], "explosm,$owner_uid:") === FALSE) {
$doc = new DOMDocument(); $doc = new DOMDocument();
@$doc->loadHTML(fetch_file_contents($article["link"])); @$doc->loadHTML(fetch_file_contents($article["link"]));
@ -48,7 +48,7 @@ class Af_Explosm extends Plugin {
} }
} }
$article["guid"] = "explosm,$owner_uid:" . $article["guid"]; $article["plugin_data"] = "explosm,$owner_uid:" . $article["plugin_data"];
} }
return $article; return $article;

View File

@ -20,7 +20,7 @@ class Af_GoComics extends Plugin {
function hook_article_filter($article) { function hook_article_filter($article) {
$owner_uid = $article["owner_uid"]; $owner_uid = $article["owner_uid"];
if (strpos($article["guid"], "gocomics.com") !== FALSE && strpos($article["guid"], "gocomics,$owner_uid:") === FALSE) { if (strpos($article["guid"], "gocomics.com") !== FALSE && strpos($article["plugin_data"], "gocomics,$owner_uid:") === FALSE) {
$doc = new DOMDocument(); $doc = new DOMDocument();
@$doc->loadHTML(fetch_file_contents($article["link"])); @$doc->loadHTML(fetch_file_contents($article["link"]));
@ -47,7 +47,7 @@ class Af_GoComics extends Plugin {
} }
} }
$article["guid"] = "gocomics,$owner_uid:" . $article["guid"]; $article["plugin_data"] = "gocomics,$owner_uid:" . $article["plugin_data"];
} }
return $article; return $article;

View File

@ -21,7 +21,7 @@ class Af_PennyArcade extends Plugin {
$owner_uid = $article["owner_uid"]; $owner_uid = $article["owner_uid"];
if (strpos($article["link"], "penny-arcade.com") !== FALSE && strpos($article["title"], "Comic:") !== FALSE && if (strpos($article["link"], "penny-arcade.com") !== FALSE && strpos($article["title"], "Comic:") !== FALSE &&
strpos($article["guid"], "pennyarcade,$owner_uid:") === FALSE) { strpos($article["plugin_data"], "pennyarcade,$owner_uid:") === FALSE) {
$doc = new DOMDocument(); $doc = new DOMDocument();
@$doc->loadHTML(fetch_file_contents($article["link"])); @$doc->loadHTML(fetch_file_contents($article["link"]));
@ -48,7 +48,7 @@ class Af_PennyArcade extends Plugin {
} }
} }
$article["guid"] = "pennyarcade,$owner_uid:" . $article["guid"]; $article["plugin_data"] = "pennyarcade,$owner_uid:" . $article["plugin_data"];
} }
return $article; return $article;

View File

@ -26,7 +26,7 @@ class Af_Unburn extends Plugin {
if ((strpos($article["link"], "feedproxy.google.com") !== FALSE || if ((strpos($article["link"], "feedproxy.google.com") !== FALSE ||
strpos($article["link"], "/~r/") !== FALSE || strpos($article["link"], "/~r/") !== FALSE ||
strpos($article["link"], "feedsportal.com") !== FALSE) && strpos($article["link"], "feedsportal.com") !== FALSE) &&
strpos($article["guid"], "unburn,$owner_uid:") === FALSE) { strpos($article["plugin_data"], "unburn,$owner_uid:") === FALSE) {
$ch = curl_init($article["link"]); $ch = curl_init($article["link"]);
curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, 5); curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, 5);
@ -62,7 +62,7 @@ class Af_Unburn extends Plugin {
$real_url = preg_replace("/\?$/", "", $real_url); $real_url = preg_replace("/\?$/", "", $real_url);
$article["guid"] = "unburn,$owner_uid:" . $article["guid"]; $article["plugin_data"] = "unburn,$owner_uid:" . $article["plugin_data"];
$article["link"] = $real_url; $article["link"] = $real_url;
} }
} }