Merge branch 'af-readability' into 'master'

Fix array checking method in 2 places, add array check in one location for af_readability

See: https://tt-rss.org/gitlab/fox/tt-rss/merge_requests/19

Update method of array checking to is_array() in 2 locations, add the same check in another location that could have compared a NULL.

See merge request !20
master
Andrew Dolgov 9 years ago
commit c4d0e08060

@ -33,7 +33,7 @@ class Af_Readability extends Plugin {
print_notice("Enable the plugin for specific feeds in the feed editor.");
$enabled_feeds = $this->host->get($this, "enabled_feeds");
if (!array($enabled_feeds)) $enabled_feeds = array();
if (!is_array($enabled_feeds)) $enabled_feeds = array();
$enabled_feeds = $this->filter_unknown_feeds($enabled_feeds);
$this->host->set($this, "enabled_feeds", $enabled_feeds);
@ -60,7 +60,7 @@ class Af_Readability extends Plugin {
print "<div class=\"dlgSecCont\">";
$enabled_feeds = $this->host->get($this, "enabled_feeds");
if (!array($enabled_feeds)) $enabled_feeds = array();
if (!is_array($enabled_feeds)) $enabled_feeds = array();
$key = array_search($feed_id, $enabled_feeds);
$checked = $key !== FALSE ? "checked" : "";
@ -169,6 +169,8 @@ class Af_Readability extends Plugin {
function hook_article_filter($article) {
$enabled_feeds = $this->host->get($this, "enabled_feeds");
if (!is_array($enabled_feeds)) return $article;
$key = array_search($article["feed"]["id"], $enabled_feeds);
if ($key === FALSE) return $article;

Loading…
Cancel
Save