Fix: allow array|string|null for ValidationResult::$fix to prevent TypeError (librenms#18457)

nerdc0re-net · web-flow · commit 26f7108cf397 · 2025-10-30T22:07:49.000+01:00
diff --git a/LibreNMS/ValidationResult.php b/LibreNMS/ValidationResult.php
@@ -48,7 +48,7 @@ class ValidationResult
      * @param  int  $status  The status of this result FAILURE, WARNING, or SUCCESS
      * @param  string|null  $fix  a suggested fix to highlight for the user
      */
-    public function __construct(private string $message, private int $status, private ?string $fix = null)
+    public function __construct(private string $message, private int $status, private string|array|null $fix = null)
     {
     }
 

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ class ValidationResult`
`48`	`48`	`* @param int $status The status of this result FAILURE, WARNING, or SUCCESS`
`49`	`49`	`* @param string\|null $fix a suggested fix to highlight for the user`
`50`	`50`	`*/`
`51`		`- public function __construct(private string $message, private int $status, private ?string $fix = null)`
	`51`	`+ public function __construct(private string $message, private int $status, private string\|array\|null $fix = null)`
`52`	`52`	`{`
`53`	`53`	`}`
`54`	`54`