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

nette · Apr 20, 2024 · bf86204 · bf86204
1 parent b46947b
commit bf86204
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 13 deletions.
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\\UI\\', '*\\', '**Presenter'],
 		'Nette' => ['NetteModule\\', '*\\', '*Presenter'],
 	];
 

diff --git a/tests/Application/PresenterFactory.formatPresenterClass.phpt b/tests/Application/PresenterFactory.formatPresenterClass.phpt
@@ -21,11 +21,11 @@ test('defined module', 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\UI\Foo\FooPresenter', $factory->formatPresenterClass('Foo'));
+	Assert::same('App\UI\Foo\Bar\BarPresenter', $factory->formatPresenterClass('Foo:Bar'));
+	Assert::same('App\UI\Foo\Bar\Baz\BazPresenter', $factory->formatPresenterClass('Foo:Bar:Baz'));
 
-	Assert::same('Foo2Presenter', $factory->formatPresenterClass('Foo2'));
+	Assert::same('App\UI\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('auto module', function () {
 		'Foo3' => 'My\App\*Presenter',
 	]);
 
-	Assert::same('Foo2Presenter', $factory->formatPresenterClass('Foo2'));
+	Assert::same('App\UI\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('location ** & defined module', 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\UI\Foo\FooPresenter', $factory->formatPresenterClass('Foo'));
+	Assert::same('App\UI\Foo\Bar\BarPresenter', $factory->formatPresenterClass('Foo:Bar'));
+	Assert::same('App\UI\Foo\Bar\Baz\BazPresenter', $factory->formatPresenterClass('Foo:Bar:Baz'));
 
-	Assert::same('Foo2Presenter', $factory->formatPresenterClass('Foo2'));
+	Assert::same('App\UI\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\UI\Homepage {
+
+	use Nette;
 
 	require __DIR__ . '/../bootstrap.php';
 
@@ -24,7 +26,7 @@ namespace {
 
 }
 
-namespace ModuleModule {
+namespace App\UI\Module\My {
 
 	use Nette;
 
@@ -77,7 +79,7 @@ namespace {
 	Assert::exception(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('', 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',