SimpleRouter: uses loose comparison == for default values since PHP 8.0

xificurk · dg · commit bee5dc46b0a8 · 2025-08-01T04:14:03.000+02:00
diff --git a/src/Routing/SimpleRouter.php b/src/Routing/SimpleRouter.php
@@ -45,9 +45,7 @@ public function constructUrl(array $params, Nette\Http\UrlScript $refUrl): ?stri
 	{
 		// remove default values; null values are retain
 		foreach ($this->defaults as $key => $value) {
-			if (isset($params[$key])
-				&& (is_scalar($params[$key]) ? (string) $params[$key] : $params[$key]) === (is_scalar($value) ? (string) $value : $value)
-			) {
+			if (isset($params[$key]) && $params[$key] == $value) { // default value may be object, intentionally ==
 				unset($params[$key]);
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -45,9 +45,7 @@ public function constructUrl(array $params, Nette\Http\UrlScript $refUrl): ?stri`
`45`	`45`	`{`
`46`	`46`	`// remove default values; null values are retain`
`47`	`47`	`foreach ($this->defaults as $key => $value) {`
`48`		`- if (isset($params[$key])`
`49`		`- && (is_scalar($params[$key]) ? (string) $params[$key] : $params[$key]) === (is_scalar($value) ? (string) $value : $value)`
`50`		`- ) {`
	`48`	`+ if (isset($params[$key]) && $params[$key] == $value) { // default value may be object, intentionally ==`
`51`	`49`	`unset($params[$key]);`
`52`	`50`	`}`
`53`	`51`	`}`