Fix #5 : Specify all arguments for Document ctor

unsaturated · unsaturated · commit 828ef4f7d455 · 2018-04-01T19:54:24.000-04:00
- Add type check to ensure $page-&gt;content() is a string
 - Add all arguments to DiDom\Document constructor call
diff --git a/lazy-image.php b/lazy-image.php
@@ -103,7 +103,7 @@ protected function isValidClass($c = "")
 
     protected function updateImages($content)
     {
-        if(trim($content) === '') 
+        if(!is_string($content) || trim($content) === '')
         {
             return;
         }
@@ -112,7 +112,7 @@ protected function updateImages($content)
 
         if($this->isValidClass($class_for_images))
         {
-            $document = new Document($content);
+            $document = new Document($content, false, 'UTF-8', 'html');
             if (count($images = $document->find("img.".$class_for_images)) > 0) {
                 foreach ($images as $image) {
                     if(!$image->getAttribute("data-src")) {

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ protected function isValidClass($c = "")`
`103`	`103`
`104`	`104`	`protected function updateImages($content)`
`105`	`105`	`{`
`106`		`- if(trim($content) === '')`
	`106`	`+ if(!is_string($content) \|\| trim($content) === '')`
`107`	`107`	`{`
`108`	`108`	`return;`
`109`	`109`	`}`
`@@ -112,7 +112,7 @@ protected function updateImages($content)`
`112`	`112`
`113`	`113`	`if($this->isValidClass($class_for_images))`
`114`	`114`	`{`
`115`		`- $document = new Document($content);`
	`115`	`+ $document = new Document($content, false, 'UTF-8', 'html');`
`116`	`116`	`if (count($images = $document->find("img.".$class_for_images)) > 0) {`
`117`	`117`	`foreach ($images as $image) {`
`118`	`118`	`if(!$image->getAttribute("data-src")) {`