Skip to content

Commit 523b9af

Browse files
committed
Fixes missing $html bug in ArticleExtractor.
1 parent cd6c723 commit 523b9af

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

src/ArticleExtractor.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -279,6 +279,7 @@ private function parseURLViaReadability($url) {
279279
$text = null;
280280
$title = null;
281281
$method = "readability";
282+
$html = null;
282283

283284
try {
284285
if($this->user_agent != null) {

tests/ExtractorTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
class ExtractorTest extends TestCase {
77

88
private $problem_sites = [
9+
'https://news.google.com/__i/rss/rd/articles/CBMihgFodHRwczovL29oaW9jYXBpdGFsam91cm5hbC5jb20vMjAyMS8wNy8yMS9zdHVkeS1jZW50cmFsLW9oaW8taG91c2luZy1zdGFiaWxpemF0aW9uLXByb2plY3QtaW1wcm92ZWQtaGVhbHRoLW91dGNvbWVzLWZvci13b21lbi1pbmZhbnRzL9IBAA?oc=5',
910
'https://www.google.com/url?rct=j&sa=t&url=https://medium.com/pcmag-access/majority-of-iot-traffic-on-corporate-networks-is-insecure-report-finds-fca274d7409a%3Fsource%3Dtopic_page---------0------------------1&ct=ga&cd=CAIyGjUwNDYzZmZmOTBiY2RhNTM6Y29tOmVuOlVT&usg=AFQjCNHdAURGnK0sMRnSgOckblLTRgxUAQ',
1011
'https://slashdot.org/story/18/07/19/2050244/microsofts-plan-to-try-to-win-back-consumers-with-modern-life-services',
1112
'http://feedproxy.google.com/~r/businessinsider/~3/EChmgXESt_4/wells-fargo-close-settlement-end-probes-sales-practices-federal-prosecutors-2020-2-1028927535', // Issue #26 multiple redirects occur when browser user-agent not set

0 commit comments

Comments
 (0)