Route: uses loose comparison == for default values [Closes #8]

Author: dg

src/Routing/Route.php

@@ -319,7 +319,8 @@ private function preprocessParams(array &$params): bool
 			}
 
 			if ($fixity !== null) {
-				if ($params[$name] === $meta[self::Value]) { // remove default values; null values are retain
+				if ($params[$name] == $meta[self::Value]) { // default value may be object, intentionally ==
+					// remove default values; null values are retain
 					unset($params[$name]);
 					continue;
 

tests/Route/objectParameter.phpt

@@ -0,0 +1,32 @@
+<?php
+
+/**
+ * Test: Nette\Routing\Route object parameter default value.
+ */
+
+declare(strict_types=1);
+
+use Nette\Routing\Route;
+use Tester\Assert;
+
+
+require __DIR__ . '/../bootstrap.php';
+
+
+test('', function () {
+	$object = new stdClass;
+
+	$route = new Route('', [
+		'param' => $object,
+	]);
+
+	Assert::same(
+		'http://example.com/',
+		testRouteOut($route, ['param' => $object]),
+	);
+
+	Assert::same(
+		'http://example.com/',
+		testRouteOut($route, ['param' => new stdClass]),
+	);
+});

tests/Route/scalarParams.phpt

@@ -245,5 +245,8 @@ test('', function () {
 		testRouteOut($route, ['presenter' => 'homepage', 'param' => null]),
 	);
 
-	Assert::null(testRouteOut($route, ['presenter' => 'homepage', 'param' => '']));
+	Assert::same(
+		'http://example.com/homepage/',
+		testRouteOut($route, ['presenter' => 'homepage', 'param' => '']),
+	);
 });