PresenterFactory: default mapping is App\Presentation\*\**Presenter (BC break)

dg · dg · commit 3463b9a2159b · 2025-06-19T03:33:58.000+02:00
diff --git a/src/Application/PresenterFactory.php b/src/Application/PresenterFactory.php
@@ -19,7 +19,7 @@ class PresenterFactory implements IPresenterFactory
 {
 	/** @var array[] of module => splited mask */
 	private array $mapping = [
-		'*' => ['', '*Module\\', '*Presenter'],
+		'*' => ['App\Presentation\\', '*\\', '**Presenter'],
 		'Nette' => ['NetteModule\\', '*\\', '*Presenter'],
 	];
 
diff --git a/tests/Application/PresenterFactory.formatPresenterClass.phpt b/tests/Application/PresenterFactory.formatPresenterClass.phpt
@@ -21,11 +21,11 @@ test('complex mapping with multiple wildcards', function () {
 		'Foo3' => 'My\App\*Mod\*Presenter',
 	]);
 
-	Assert::same('FooPresenter', $factory->formatPresenterClass('Foo'));
-	Assert::same('FooModule\BarPresenter', $factory->formatPresenterClass('Foo:Bar'));
-	Assert::same('FooModule\BarModule\BazPresenter', $factory->formatPresenterClass('Foo:Bar:Baz'));
+	Assert::same('App\Presentation\Foo\FooPresenter', $factory->formatPresenterClass('Foo'));
+	Assert::same('App\Presentation\Foo\Bar\BarPresenter', $factory->formatPresenterClass('Foo:Bar'));
+	Assert::same('App\Presentation\Foo\Bar\Baz\BazPresenter', $factory->formatPresenterClass('Foo:Bar:Baz'));
 
-	Assert::same('Foo2Presenter', $factory->formatPresenterClass('Foo2'));
+	Assert::same('App\Presentation\Foo2\Foo2Presenter', $factory->formatPresenterClass('Foo2'));
 	Assert::same('App2\BarPresenter', $factory->formatPresenterClass('Foo2:Bar'));
 	Assert::same('App2\Bar\BazPresenter', $factory->formatPresenterClass('Foo2:Bar:Baz'));
 
@@ -44,7 +44,7 @@ test('simple wildcard mapping', function () {
 		'Foo3' => 'My\App\*Presenter',
 	]);
 
-	Assert::same('Foo2Presenter', $factory->formatPresenterClass('Foo2'));
+	Assert::same('App\Presentation\Foo2\Foo2Presenter', $factory->formatPresenterClass('Foo2'));
 	Assert::same('App2\BarPresenter', $factory->formatPresenterClass('Foo2:Bar'));
 	Assert::same('App2\BarModule\BazPresenter', $factory->formatPresenterClass('Foo2:Bar:Baz'));
 
@@ -61,11 +61,11 @@ test('multi-segment wildcard expansion', function () {
 		'Foo3' => 'My\App\*Mod\**Presenter',
 	]);
 
-	Assert::same('FooPresenter', $factory->formatPresenterClass('Foo'));
-	Assert::same('FooModule\BarPresenter', $factory->formatPresenterClass('Foo:Bar'));
-	Assert::same('FooModule\BarModule\BazPresenter', $factory->formatPresenterClass('Foo:Bar:Baz'));
+	Assert::same('App\Presentation\Foo\FooPresenter', $factory->formatPresenterClass('Foo'));
+	Assert::same('App\Presentation\Foo\Bar\BarPresenter', $factory->formatPresenterClass('Foo:Bar'));
+	Assert::same('App\Presentation\Foo\Bar\Baz\BazPresenter', $factory->formatPresenterClass('Foo:Bar:Baz'));
 
-	Assert::same('Foo2Presenter', $factory->formatPresenterClass('Foo2'));
+	Assert::same('App\Presentation\Foo2\Foo2Presenter', $factory->formatPresenterClass('Foo2'));
 	Assert::same('App2\Bar\BarPresenter', $factory->formatPresenterClass('Foo2:Bar'));
 	Assert::same('App2\Bar\Baz\BazPresenter', $factory->formatPresenterClass('Foo2:Bar:Baz'));
 
diff --git a/tests/Routers/LinkGenerator.phpt b/tests/Routers/LinkGenerator.phpt
@@ -6,7 +6,9 @@
 
 declare(strict_types=1);
 
-namespace {
+namespace App\Presentation\Homepage {
+
+	use Nette;
 
 	require __DIR__ . '/../bootstrap.php';
 
@@ -24,7 +26,7 @@ namespace {
 
 }
 
-namespace ModuleModule {
+namespace App\Presentation\Module\My {
 
 	use Nette;
 
@@ -77,7 +79,7 @@ namespace {
 	testException('invalid action parameter propagation', function () use ($pf) {
 		$generator = new LinkGenerator(new Routers\Route('/', 'Homepage:'), new Http\UrlScript('http://nette.org/en/'), $pf);
 		$generator->link('Homepage:missing', [10]);
-	}, Nette\Application\UI\InvalidLinkException::class, "Unable to pass parameters to action 'Homepage:missing', missing corresponding method HomepagePresenter::renderMissing().");
+	}, Nette\Application\UI\InvalidLinkException::class, "Unable to pass parameters to action 'Homepage:missing', missing corresponding method App\\UI\\Homepage\\HomepagePresenter::renderMissing().");
 
 
 	test('URL generation without PresenterFactory', function () {
diff --git a/tests/Routers/link-aliases.phpt b/tests/Routers/link-aliases.phpt
@@ -20,6 +20,7 @@ class TestPresenter extends Application\UI\Presenter
 
 
 $factory = new PresenterFactory;
+$factory->setMapping(['*' => '*Presenter']);
 $factory->setAliases([
 	'a' => 'Test:a',
 	'b' => 'Test:b',

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ class PresenterFactory implements IPresenterFactory`
`19`	`19`	`{`
`20`	`20`	`/** @var array[] of module => splited mask */`
`21`	`21`	`private array $mapping = [`
`22`		`- '' => ['', 'Module\\', '*Presenter'],`
	`22`	`+ '' => ['App\Presentation\\', '\\', '**Presenter'],`
`23`	`23`	`'Nette' => ['NetteModule\\', '\\', 'Presenter'],`
`24`	`24`	`];`
`25`	`25`