forked from andreskrey/readability.php
-
Notifications
You must be signed in to change notification settings - Fork 37
Closed
Description
Hey,
a user of nextcloud/news reported an issue and I was able to identify a page that breaks readability.
<?php
require __DIR__ . '/vendor/autoload.php';
use fivefilters\Readability\Readability;
use fivefilters\Readability\Configuration;
$readability = new Readability(new Configuration());
$content = file_get_contents('https://taz.de/Krach-zwischen-Selenskyj-und-Trump/!6072970');
$success = $readability->parse($content);
if ($success) {
print("No DOM Error" . PHP_EOL);
}
exit($success ? 0 : 1);PHP Fatal error: Uncaught TypeError: fivefilters\Readability\Nodes\DOM\DOMNodeList::add(): Argument #1 ($node) must be of type fivefilters\Readability\Nodes\DOM\DOMNode|fivefilters\Readability\Nodes\DOM\DOMElement|fivefilters\Readability\Nodes\DOM\DOMText|fivefilters\Readability\Nodes\DOM\DOMComment|fivefilters\Readability\Nodes\DOM\DOMProcessingInstruction, fivefilters\Readability\Nodes\DOM\DOMCdataSection given, called in /home/grotax/git/readability-test/vendor/fivefilters/readability.php/src/Nodes/NodeUtility.php on line 162 and defined in /home/grotax/git/readability-test/vendor/fivefilters/readability.php/src/Nodes/DOM/DOMNodeList.php:45
Stack trace:
#0 /home/grotax/git/readability-test/vendor/fivefilters/readability.php/src/Nodes/NodeUtility.php(162): fivefilters\Readability\Nodes\DOM\DOMNodeList->add()
#1 /home/grotax/git/readability-test/vendor/fivefilters/readability.php/src/Nodes/NodeTrait.php(353): fivefilters\Readability\Nodes\NodeUtility::filterTextNodes()
#2 /home/grotax/git/readability-test/vendor/fivefilters/readability.php/src/Readability.php(917): fivefilters\Readability\Nodes\DOM\DOMElement->hasSingleTagInsideElement()
#3 /home/grotax/git/readability-test/vendor/fivefilters/readability.php/src/Readability.php(178): fivefilters\Readability\Readability->getNodes()
#4 /home/grotax/git/readability-test/test.php(10): fivefilters\Readability\Readability->parse()
#5 {main}
thrown in /home/grotax/git/readability-test/vendor/fivefilters/readability.php/src/Nodes/DOM/DOMNodeList.php on line 45Metadata
Metadata
Assignees
Labels
No labels