Merge pull request #127 from andrewnicols/correctConstructor

andrewnicols · web-flow · commit 3b443cbe302b · 2024-08-20T10:43:29.000+08:00
Update constructor usage of call_user_func_array
diff --git a/src/Html2Text.php b/src/Html2Text.php
@@ -242,7 +242,9 @@ public function __construct($html = '', $options = array())
 
         // for backwards compatibility
         if (!is_array($options)) {
-            return call_user_func_array(array($this, 'legacyConstruct'), func_get_args());
+            // phpcs:ignore (PHPCompatibility.FunctionUse.ArgumentFunctionsReportCurrentValue.NeedsInspection
+            call_user_func_array(array($this, 'legacyConstruct'), func_get_args());
+            return;
         }
 
         $this->html = $html;

Original file line number	Diff line number	Diff line change
`@@ -242,7 +242,9 @@ public function __construct($html = '', $options = array())`
`242`	`242`
`243`	`243`	`// for backwards compatibility`
`244`	`244`	`if (!is_array($options)) {`
`245`		`- return call_user_func_array(array($this, 'legacyConstruct'), func_get_args());`
	`245`	`+ // phpcs:ignore (PHPCompatibility.FunctionUse.ArgumentFunctionsReportCurrentValue.NeedsInspection`
	`246`	`+ call_user_func_array(array($this, 'legacyConstruct'), func_get_args());`
	`247`	`+ return;`
`246`	`248`	`}`
`247`	`249`
`248`	`250`	`$this->html = $html;`