diff options
author | Bernhard Posselt <dev@bernhard-posselt.com> | 2015-03-22 16:06:19 +0100 |
---|---|---|
committer | Bernhard Posselt <dev@bernhard-posselt.com> | 2015-03-22 16:06:19 +0100 |
commit | 8afd5adfcf6f7ec78eddb96b77efcd316a0a0293 (patch) | |
tree | ecd06129c779d170b083023db63604eceded641c /articleenhancer/xpatharticleenhancer.php | |
parent | 60ac229e36b1e2b07773dc93fc9bacdb2016df40 (diff) |
fix scrutinizer warnings
Diffstat (limited to 'articleenhancer/xpatharticleenhancer.php')
-rw-r--r-- | articleenhancer/xpatharticleenhancer.php | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/articleenhancer/xpatharticleenhancer.php b/articleenhancer/xpatharticleenhancer.php index 67fbf78d2..61bf230a0 100644 --- a/articleenhancer/xpatharticleenhancer.php +++ b/articleenhancer/xpatharticleenhancer.php @@ -25,7 +25,6 @@ use OCA\News\Db\Item; class XPathArticleEnhancer implements ArticleEnhancer { - private $maximumTimeout; private $clientFactory; private $regexXPathPair; @@ -72,7 +71,7 @@ class XPathArticleEnhancer implements ArticleEnhancer { } $dom = new DOMDocument(); - @$dom->loadHTML($body); + $isOk = @$dom->loadHTML($body); $xpath = new DOMXpath($dom); $xpathResult = $xpath->evaluate($search); @@ -90,7 +89,7 @@ class XPathArticleEnhancer implements ArticleEnhancer { $xpathResult, $item->getUrl() ); - if($xpathResult) { + if($isOk && $xpathResult !== false && $xpathResult !== '') { $item->setBody($xpathResult); } } @@ -122,7 +121,7 @@ class XPathArticleEnhancer implements ArticleEnhancer { $dom->preserveWhiteSpace = false; if($xmlString === '') { - return false; + return ''; } $xmlString = '<div>' . $xmlString . '</div>'; @@ -130,7 +129,7 @@ class XPathArticleEnhancer implements ArticleEnhancer { LIBXML_HTML_NODEFDTD); if(!$isOk) { - return false; + return ''; } foreach (['href', 'src'] as $attribute) { |