From 666cd333608d5fcd40f67fc6235e47a557aafc9c Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Fri, 29 Jan 2016 15:38:05 +0300 Subject: [PATCH] enable support for readability (if af_readability is enabled) in shareanything bookmarklet --- classes/article.php | 14 +++++++ plugins/af_readability/init.php | 69 +++++++++++++++++++++++++++------ 2 files changed, 71 insertions(+), 12 deletions(-) diff --git a/classes/article.php b/classes/article.php index 01f6b5126..e47da8617 100644 --- a/classes/article.php +++ b/classes/article.php @@ -88,6 +88,20 @@ class Article extends Handler_Protected { $owner_uid) { $guid = 'SHA1:' . sha1("ttshared:" . $url . $owner_uid); // include owner_uid to prevent global GUID clash + + if (!$content) { + $pluginhost = new PluginHost(); + $pluginhost->load_all(PluginHost::KIND_ALL, $owner_uid); + + $af_readability = $pluginhost->get_plugin("Af_Readability"); + + if ($af_readability) { + $extracted_content = $af_readability->extract_content($url); + + if ($extracted_content) $content = db_escape_string($extracted_content); + } + } + $content_hash = sha1($content); if ($labels_str != "") { diff --git a/plugins/af_readability/init.php b/plugins/af_readability/init.php index a52b745b8..f5b764d9c 100755 --- a/plugins/af_readability/init.php +++ b/plugins/af_readability/init.php @@ -14,7 +14,11 @@ class Af_Readability extends Plugin { } function save() { - // + $enable_share_anything = checkbox_to_sql_bool($_POST["enable_share_anything"]) == "true"; + + $this->host->set($this, "enable_share_anything", $enable_share_anything); + + echo __("Data saved."); } function init($host) @@ -36,6 +40,38 @@ class Af_Readability extends Plugin { print_notice("Enable the plugin for specific feeds in the feed editor."); + print "
"; + + print ""; + + print ""; + print ""; + print ""; + + $enable_share_anything = $this->host->get($this, "enable_share_anything"); + $enable_share_anything_checked = $enable_share_anything ? "checked" : ""; + + print " + "; + + print "

"; + + print "

"; + $enabled_feeds = $this->host->get($this, "enabled_feeds"); if (!is_array($enabled_feeds)) $enabled_feeds = array(); @@ -100,13 +136,12 @@ class Af_Readability extends Plugin { return $this->process_article($article); } - function process_article($article) { - + public function extract_content($url) { if (!class_exists("Readability")) require_once(dirname(dirname(__DIR__)). "/lib/readability/Readability.php"); if (!defined('NO_CURL') && function_exists('curl_init') && !ini_get("open_basedir")) { - $ch = curl_init($article["link"]); + $ch = curl_init($url); curl_setopt($ch, CURLOPT_TIMEOUT, 5); curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1); @@ -119,16 +154,16 @@ class Af_Readability extends Plugin { $content_type = curl_getinfo($ch, CURLINFO_CONTENT_TYPE); if (strpos($content_type, "text/html") === FALSE) - return $article; + return false; } - $tmp = fetch_file_contents($article["link"]); + $tmp = fetch_file_contents($url); if ($tmp && mb_strlen($tmp) < 65535 * 4) { $tmpdoc = new DOMDocument("1.0", "UTF-8"); if (!$tmpdoc->loadHTML($tmp)) - return $article; + return false; if (strtolower($tmpdoc->encoding) != 'utf-8') { $tmpxpath = new DOMXPath($tmpdoc); @@ -140,10 +175,9 @@ class Af_Readability extends Plugin { $tmp = $tmpdoc->saveHTML(); } - $r = new Readability($tmp, $article["link"]); + $r = new Readability($tmp, $url); if ($r->init()) { - $tmpxpath = new DOMXPath($r->dom); $entries = $tmpxpath->query('(//a[@href]|//img[@src])'); @@ -151,22 +185,33 @@ class Af_Readability extends Plugin { foreach ($entries as $entry) { if ($entry->hasAttribute("href")) { $entry->setAttribute("href", - rewrite_relative_url($article["link"], $entry->getAttribute("href"))); + rewrite_relative_url($url, $entry->getAttribute("href"))); } if ($entry->hasAttribute("src")) { $entry->setAttribute("src", - rewrite_relative_url($article["link"], $entry->getAttribute("src"))); + rewrite_relative_url($url, $entry->getAttribute("src"))); } } - $article["content"] = $r->articleContent->innerHTML; + return $r->articleContent->innerHTML; } } + return false; + } + + function process_article($article) { + + $extracted_content = $this->extract_content($article["link"]); + + if ($extracted_content) { + $article["content"] = $extracted_content; + } + return $article; }