diff --git a/composer.json b/composer.json
index 6eed577a52e3236af75eb34d15336c31cbf49ac1..c1ed28c719920175b8ce9cde522f951011b27d1f 100644
--- a/composer.json
+++ b/composer.json
@@ -309,6 +309,7 @@
"TYPO3Tests\\TestValidators\\": "typo3/sysext/extbase/Tests/Functional/Fixtures/Extensions/test_validators/Classes/",
"TYPO3Tests\\TestDatahandler\\": "typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_datahandler/Classes/",
"TYPO3Tests\\TestDataMapper\\": "typo3/sysext/extbase/Tests/Functional/Fixtures/Extensions/test_data_mapper/Classes/",
+ "TYPO3Tests\\TestDi\\": "typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Classes/",
"TYPO3Tests\\TestFluidTemplate\\": "typo3/sysext/frontend/Tests/Functional/Fixtures/Extensions/test_fluid_template/Classes/",
"TYPO3Tests\\TestIrreForeignfield\\": "typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_irre_foreignfield/Classes/",
"TYPO3Tests\\TestLogger\\": "typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_logger/Classes/",
diff --git a/typo3/sysext/core/Configuration/Services.php b/typo3/sysext/core/Configuration/Services.php
index c45e9d00a6c062f68047f79e857fc30852cd522b..b3bc50bf4664450bc17902968517212b90e2d383 100644
--- a/typo3/sysext/core/Configuration/Services.php
+++ b/typo3/sysext/core/Configuration/Services.php
@@ -43,11 +43,11 @@ return static function (ContainerConfigurator $container, ContainerBuilder $cont
$definition->addTag(
'console.command',
[
- 'command' => $attribute->name,
+ 'command' => ltrim($attribute->name, '|'),
'description' => $attribute->description,
- // `schedulable` and `hidden` flags are not configurable via symfony attribute parameters, use sane defaults
+ 'hidden' => str_starts_with($attribute->name, '|'),
+ // The `schedulable` flag is not configurable via symfony attribute parameters, use sane defaults
'schedulable' => true,
- 'hidden' => false,
]
);
}
diff --git a/typo3/sysext/core/Documentation/Changelog/12.4.x/Important-101567-UseSymfonyAttributeToAutoconfigureCliCommands.rst b/typo3/sysext/core/Documentation/Changelog/12.4.x/Important-101567-UseSymfonyAttributeToAutoconfigureCliCommands.rst
index 5560f34dc28a34dc247079c11c924c13969c2934..f9f588bb8d89467b36479642243a6d29956609f6 100644
--- a/typo3/sysext/core/Documentation/Changelog/12.4.x/Important-101567-UseSymfonyAttributeToAutoconfigureCliCommands.rst
+++ b/typo3/sysext/core/Documentation/Changelog/12.4.x/Important-101567-UseSymfonyAttributeToAutoconfigureCliCommands.rst
@@ -11,12 +11,12 @@ See :issue:`101567`
Description
===========
-The symfony PHP attribute :php:`\Symfony\Component\Console\Attribute\AsCommand` is now accepted to register
-console commands.
-This way CLI commands can be registered by setting the attribute on the command class.
-Only the parameters command, description are still viable.
-In order to overwrite the parameters schedulable and hidden use the old :file:`Services.yaml` way to
-register console commands. By default schedulable is true and hidden is false.
+The symfony PHP attribute :php:`\Symfony\Component\Console\Attribute\AsCommand`
+is now accepted to register console commands.
+This way CLI commands can be registered by setting the attribute on the command
+class. Only the parameters `command`, `description` and `hidden` are still viable. In order to
+overwrite the schedulable parameter use the old :file:`Services.yaml` way to
+register console commands. By default `schedulable` is true.
Before:
@@ -29,7 +29,6 @@ Before:
command: 'myprefix:dofoo'
description: 'My description'
schedulable: true
- hidden: false
After:
diff --git a/typo3/sysext/core/Tests/Functional/DependencyInjection/AsCommandAttributeTest.php b/typo3/sysext/core/Tests/Functional/DependencyInjection/AsCommandAttributeTest.php
new file mode 100644
index 0000000000000000000000000000000000000000..6bf5a6f4663a881af4d2820e8b49f4e717f50310
--- /dev/null
+++ b/typo3/sysext/core/Tests/Functional/DependencyInjection/AsCommandAttributeTest.php
@@ -0,0 +1,70 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * This file is part of the TYPO3 CMS project.
+ *
+ * It is free software; you can redistribute it and/or modify it under
+ * the terms of the GNU General Public License, either version 2
+ * of the License, or any later version.
+ *
+ * For the full copyright and license information, please read the
+ * LICENSE.txt file that was distributed with this source code.
+ *
+ * The TYPO3 project - inspiring people to share!
+ */
+
+namespace TYPO3\CMS\Core\Tests\Functional\DependencyInjection;
+
+use TYPO3\CMS\Core\Console\CommandRegistry;
+use TYPO3\TestingFramework\Core\Functional\FunctionalTestCase;
+use TYPO3Tests\TestDi\Command\HiddenTestCommand;
+use TYPO3Tests\TestDi\Command\VisibleTestCommand;
+
+final class AsCommandAttributeTest extends FunctionalTestCase
+{
+ protected array $testExtensionsToLoad = [
+ 'typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di',
+ ];
+
+ /**
+ * @test
+ */
+ public function asCommandRegisteredToCommandRegistry(): void
+ {
+ $commandRegistry = $this->get(CommandRegistry::class);
+
+ self::assertTrue($commandRegistry->has('testdi:ascommand:visible'));
+ self::assertInstanceOf(VisibleTestCommand::class, $commandRegistry->get('testdi:ascommand:visible'));
+
+ self::assertTrue($commandRegistry->has('testdi:ascommand:hidden'));
+ self::assertInstanceOf(HiddenTestCommand::class, $commandRegistry->get('testdi:ascommand:hidden'));
+ }
+
+ /**
+ * @test
+ */
+ public function asCommandHiddenAttributeIsRespected(): void
+ {
+ $commandRegistry = $this->get(CommandRegistry::class);
+ $visibleList = $commandRegistry->filter();
+
+ self::assertArrayHasKey('testdi:ascommand:visible', $visibleList);
+ self::assertArrayNotHasKey('testdi:ascommand:hidden', $visibleList);
+ }
+
+ /**
+ * @test
+ */
+ public function asCommandSetsDescription(): void
+ {
+ $commandRegistry = $this->get(CommandRegistry::class);
+
+ $visibleCommand = $commandRegistry->get('testdi:ascommand:visible');
+ $hiddenCommand = $commandRegistry->get('testdi:ascommand:hidden');
+
+ self::assertEquals('This is a visible command.', $visibleCommand->getDescription());
+ self::assertEquals('This is a hidden command.', $hiddenCommand->getDescription());
+ }
+}
diff --git a/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Classes/Command/HiddenTestCommand.php b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Classes/Command/HiddenTestCommand.php
new file mode 100644
index 0000000000000000000000000000000000000000..98f51e7fa7e747c032e0ca40114fef1ea4ed4e50
--- /dev/null
+++ b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Classes/Command/HiddenTestCommand.php
@@ -0,0 +1,32 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * This file is part of the TYPO3 CMS project.
+ *
+ * It is free software; you can redistribute it and/or modify it under
+ * the terms of the GNU General Public License, either version 2
+ * of the License, or any later version.
+ *
+ * For the full copyright and license information, please read the
+ * LICENSE.txt file that was distributed with this source code.
+ *
+ * The TYPO3 project - inspiring people to share!
+ */
+
+namespace TYPO3Tests\TestDi\Command;
+
+use Symfony\Component\Console\Attribute\AsCommand;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+
+#[AsCommand(name: 'testdi:ascommand:hidden', description: 'This is a hidden command.', hidden: true)]
+class HiddenTestCommand extends Command
+{
+ protected function execute(InputInterface $input, OutputInterface $output): int
+ {
+ return Command::SUCCESS;
+ }
+}
diff --git a/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Classes/Command/VisibleTestCommand.php b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Classes/Command/VisibleTestCommand.php
new file mode 100644
index 0000000000000000000000000000000000000000..c8556e2d857bd5f0c9ed908d8fa83ca4016835e6
--- /dev/null
+++ b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Classes/Command/VisibleTestCommand.php
@@ -0,0 +1,32 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * This file is part of the TYPO3 CMS project.
+ *
+ * It is free software; you can redistribute it and/or modify it under
+ * the terms of the GNU General Public License, either version 2
+ * of the License, or any later version.
+ *
+ * For the full copyright and license information, please read the
+ * LICENSE.txt file that was distributed with this source code.
+ *
+ * The TYPO3 project - inspiring people to share!
+ */
+
+namespace TYPO3Tests\TestDi\Command;
+
+use Symfony\Component\Console\Attribute\AsCommand;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+
+#[AsCommand(name: 'testdi:ascommand:visible', description: 'This is a visible command.')]
+class VisibleTestCommand extends Command
+{
+ protected function execute(InputInterface $input, OutputInterface $output): int
+ {
+ return Command::SUCCESS;
+ }
+}
diff --git a/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Configuration/Services.yaml b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Configuration/Services.yaml
new file mode 100644
index 0000000000000000000000000000000000000000..257a0b4ecd91f0fb8f67e3242c6fbc6d16a7f4a2
--- /dev/null
+++ b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/Configuration/Services.yaml
@@ -0,0 +1,8 @@
+services:
+ _defaults:
+ autowire: true
+ autoconfigure: true
+ public: false
+
+ TYPO3Tests\TestDi\:
+ resource: '../Classes/*'
diff --git a/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/composer.json b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/composer.json
new file mode 100644
index 0000000000000000000000000000000000000000..5fdaeac5a5c68f6e1b16a51f58ed87b8940ba339
--- /dev/null
+++ b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/composer.json
@@ -0,0 +1,19 @@
+{
+ "name": "typo3tests/test-di",
+ "type": "typo3-cms-extension",
+ "description": "This extension contains dependency injection fixtures.",
+ "license": "GPL-2.0-or-later",
+ "require": {
+ "typo3/cms-core": "13.0.*@dev"
+ },
+ "extra": {
+ "typo3/cms": {
+ "extension-key": "test_di"
+ }
+ },
+ "autoload": {
+ "psr-4": {
+ "TYPO3Tests\\TestDi\\": "Classes/"
+ }
+ }
+}
diff --git a/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/ext_emconf.php b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/ext_emconf.php
new file mode 100644
index 0000000000000000000000000000000000000000..c89d491b931018010bfeffb6281025220cae4b48
--- /dev/null
+++ b/typo3/sysext/core/Tests/Functional/Fixtures/Extensions/test_di/ext_emconf.php
@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+$EM_CONF[$_EXTKEY] = [
+ 'title' => 'This extension contains dependency injection fixtures.',
+ 'description' => 'This extension contains dependency injection fixture.',
+ 'category' => 'example',
+ 'version' => '13.0.0',
+ 'state' => 'beta',
+ 'author' => 'Benjamin Franzke',
+ 'author_email' => 'ben@bnf.dev',
+ 'author_company' => '',
+ 'constraints' => [
+ 'depends' => [
+ 'typo3' => '13.0.0',
+ ],
+ 'conflicts' => [],
+ 'suggests' => [],
+ ],
+];