Merge pull request #4566 from apphp/master

PHP 8 compatibility: Fix passing null to string parameter parameter
yiisoft · Sep 30, 2024 · f0ea6bd · f0ea6bd
2 parents 9d0b1ff + bc0f4d0
commit f0ea6bd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -7,6 +7,7 @@ Version 1.1.30 under development
 - Bug #4547: PHP 8 compatibility: Fix deprecation in CCaptcha when using Imagick extension (apphp)
 - Bug #4541: PHP 8 compatibility: Fix deprecation in MarkdownParser (mdeweerd)
 - Bug #4544: PHP 8 compatibility: Fix deprecation in CLocale (apphp)
+- Bug #4566: PHP 8 compatibility: Fix passing null to string parameter parameter (apphp)
 - Enh #4552: Added support for PHP 8.3 (sandippingle, ThePepster, marcovtwout)
 
 Version 1.1.29 November 14, 2023

diff --git a/framework/console/CConsoleCommand.php b/framework/console/CConsoleCommand.php
@@ -229,6 +229,9 @@ protected function resolveRequest($args)
 		$params=array();	// unnamed parameters
 		foreach($args as $arg)
 		{
+			if (is_null($arg))
+				continue;
+
 			if(preg_match('/^--(\w+)(=(.*))?$/',$arg,$matches))  // an option
 			{
 				$name=$matches[1];