Issue #394: Service refactoring

Signed-off-by: alexmerlin <[email protected]>

Author: alexmerlin

composer.json

@@ -97,6 +97,7 @@
             "Api\\User\\": "src/User/src/",
             "Core\\Admin\\": "src/Core/src/Admin/src/",
             "Core\\App\\": "src/Core/src/App/src/",
+            "Core\\Security\\": "src/Core/src/Security/src/",
             "Core\\User\\": "src/Core/src/User/src/"
         }
     },

config/autoload/cors.local.php.dist

@@ -11,7 +11,7 @@ return [
              * Leaving this line here makes your application accessible by any origin.
              *
              * To restrict, replace this line with a list of origins that should have access to your application.
-             * Example: "domain1.com", "domain2.com"
+             * Example: 'domain1.com', 'domain2.com'
              */
             ConfigurationInterface::ANY_ORIGIN,
         ],

config/autoload/dependencies.global.php

@@ -3,66 +3,53 @@
 declare(strict_types=1);
 
 use Api\App\Factory\ErrorResponseGeneratorFactory;
-use Api\App\Factory\OAuthAccessTokenRepositoryFactory;
-use Api\App\Factory\OAuthAuthCodeRepositoryFactory;
-use Api\App\Factory\OAuthClientRepositoryFactory;
-use Api\App\Factory\OAuthRefreshTokenRepositoryFactory;
-use Api\App\Factory\OAuthScopeRepositoryFactory;
 use Api\App\Factory\UserIdentityFactory;
-use Api\App\Factory\UserRepositoryFactory;
-use Core\App\Repository\OAuthAccessTokenRepository;
-use Core\App\Repository\OAuthAuthCodeRepository;
-use Core\App\Repository\OAuthClientRepository;
-use Core\App\Repository\OAuthRefreshTokenRepository;
-use Core\App\Repository\OAuthScopeRepository;
+use Core\Security\Repository\OAuthAccessTokenRepository;
+use Core\Security\Repository\OAuthAuthCodeRepository;
+use Core\Security\Repository\OAuthClientRepository;
+use Core\Security\Repository\OAuthRefreshTokenRepository;
+use Core\Security\Repository\OAuthScopeRepository;
 use Core\User\Repository\UserRepository;
 use Core\User\UserIdentity;
 use Doctrine\Migrations\Tools\Console\Command\ExecuteCommand;
-use Dot\ErrorHandler\ErrorHandlerInterface;
-use Dot\ErrorHandler\LogErrorHandler;
 use League\OAuth2\Server\Repositories\AccessTokenRepositoryInterface;
 use League\OAuth2\Server\Repositories\AuthCodeRepositoryInterface;
 use League\OAuth2\Server\Repositories\ClientRepositoryInterface;
 use League\OAuth2\Server\Repositories\RefreshTokenRepositoryInterface;
 use League\OAuth2\Server\Repositories\ScopeRepositoryInterface;
 use League\OAuth2\Server\Repositories\UserRepositoryInterface;
+use Mezzio\Authentication\UserInterface;
+use Mezzio\Authorization\AuthorizationInterface;
+use Mezzio\Authorization\Rbac\LaminasRbac;
+use Mezzio\Middleware\ErrorResponseGenerator;
 use Roave\PsrContainerDoctrine\Migrations\CommandFactory;
 
 return [
     // Provides application-wide services.
-    // We recommend using fully-qualified class names whenever possible as
-    // service names.
+    // We recommend using fully-qualified class names whenever possible as service names.
     'dependencies' => [
-        // Use 'aliases' to alias a service name to another service. The
-        // key is the alias name, the value is the service to which it points.
+        // Use 'aliases' to alias a service name to another service.
+        // The key is the alias name, the value is the service to which it points.
         'aliases' => [
-            AccessTokenRepositoryInterface::class              => OAuthAccessTokenRepository::class,
-            AuthCodeRepositoryInterface::class                 => OAuthAuthCodeRepository::class,
-            ClientRepositoryInterface::class                   => OAuthClientRepository::class,
-            RefreshTokenRepositoryInterface::class             => OAuthRefreshTokenRepository::class,
-            ScopeRepositoryInterface::class                    => OAuthScopeRepository::class,
-            Mezzio\Authentication\UserInterface::class         => UserIdentity::class,
-            ErrorHandlerInterface::class                       => LogErrorHandler::class,
-            Mezzio\Authorization\AuthorizationInterface::class => Mezzio\Authorization\Rbac\LaminasRbac::class,
-            UserRepositoryInterface::class                     => UserRepository::class,
+            AccessTokenRepositoryInterface::class  => OAuthAccessTokenRepository::class,
+            AuthCodeRepositoryInterface::class     => OAuthAuthCodeRepository::class,
+            ClientRepositoryInterface::class       => OAuthClientRepository::class,
+            UserInterface::class                   => UserIdentity::class,
+            AuthorizationInterface::class          => LaminasRbac::class,
+            RefreshTokenRepositoryInterface::class => OAuthRefreshTokenRepository::class,
+            ScopeRepositoryInterface::class        => OAuthScopeRepository::class,
+            UserRepositoryInterface::class         => UserRepository::class,
         ],
-        // Use 'invokables' for constructor-less services, or services that do
-        // not require arguments to the constructor. Map a service name to the
-        // class name.
+        // Use 'invokables' for constructor-less services, or services that do not require arguments to the constructor.
+        // Map a service name to the class name.
         'invokables' => [
             // Fully\Qualified\InterfaceName::class => Fully\Qualified\ClassName::class,
         ],
         // Use 'factories' for services provided by callbacks/factory classes.
         'factories' => [
-            ExecuteCommand::class                           => CommandFactory::class,
-            Mezzio\Middleware\ErrorResponseGenerator::class => ErrorResponseGeneratorFactory::class,
-            OAuthAccessTokenRepository::class               => OAuthAccessTokenRepositoryFactory::class,
-            OAuthAuthCodeRepository::class                  => OAuthAuthCodeRepositoryFactory::class,
-            OAuthClientRepository::class                    => OAuthClientRepositoryFactory::class,
-            OAuthRefreshTokenRepository::class              => OAuthRefreshTokenRepositoryFactory::class,
-            OAuthScopeRepository::class                     => OAuthScopeRepositoryFactory::class,
-            UserRepository::class                           => UserRepositoryFactory::class,
-            UserIdentity::class                             => UserIdentityFactory::class,
+            ExecuteCommand::class         => CommandFactory::class,
+            ErrorResponseGenerator::class => ErrorResponseGeneratorFactory::class,
+            UserIdentity::class           => UserIdentityFactory::class,
         ],
     ],
 ];

config/autoload/mezzio-tooling-factories.global.php

@@ -17,6 +17,6 @@
         'globals'         => [],
         'optimizations'   => -1,
         'runtime_loaders' => [],
-//        'timezone'        => '',
+        'timezone'        => 'UTC',
     ],
 ];

config/autoload/templates.global.php

@@ -51,6 +51,7 @@ class_exists(Mezzio\Tooling\ConfigProvider::class)
     // Default App module config
     Core\Admin\ConfigProvider::class,
     Core\App\ConfigProvider::class,
+    Core\Security\ConfigProvider::class,
     Core\User\ConfigProvider::class,
     Api\Admin\ConfigProvider::class,
     Api\App\ConfigProvider::class,

config/config.php

@@ -23,9 +23,9 @@ after replacing:
 
 If the submitted data is valid, the outputted response is:
 
-```text
-Admin account has been created.
-```
+>
+> [INFO] Admin account has been created.
+>
 
 The new admin account is ready to use.
 

documentation/command/admin-create.md

@@ -36,6 +36,10 @@ The output should look similar to this:
 Error reporting token:
 
     0123456789abcdef0123456789abcdef01234567
+
+* copy the generated token
+* open config/autoload/error-handling.global.php
+* paste the copied string inside the tokens array found under the ErrorReportServiceInterface::class key.
 ```
 
 Copy the generated token.

documentation/command/token-generate.md

@@ -4,6 +4,9 @@ includes:
 parameters:
     level: 5
     paths:
+        - bin
+        - config
+        - public
         - src
         - test
     treatPhpDocTypesAsCertain: false

phpstan.neon

@@ -5,18 +5,20 @@
 namespace Api\Admin\Command;
 
 use Api\Admin\InputFilter\CreateAdminInputFilter;
-use Api\Admin\Service\AdminRoleService;
-use Api\Admin\Service\AdminService;
-use Api\App\Exception\BadRequestException;
-use Api\App\Exception\ConflictException;
-use Api\App\Exception\NotFoundException;
 use Core\Admin\Entity\AdminRole;
+use Core\Admin\Service\AdminRoleServiceInterface;
+use Core\Admin\Service\AdminServiceInterface;
+use Core\App\Exception\BadRequestException;
+use Core\App\Exception\ConflictException;
+use Core\App\Exception\NotFoundException;
 use Core\App\Message;
+use Dot\DependencyInjection\Attribute\Inject;
 use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Console\Style\SymfonyStyle;
 
 use function implode;
 use function sprintf;
@@ -32,9 +34,13 @@ class AdminCreateCommand extends Command
     /** @var string $defaultName */
     protected static $defaultName = 'admin:create';
 
+    #[Inject(
+        AdminServiceInterface::class,
+        AdminRoleServiceInterface::class,
+    )]
     public function __construct(
-        protected AdminService $adminService,
-        protected AdminRoleService $adminRoleService,
+        protected AdminServiceInterface $adminService,
+        protected AdminRoleServiceInterface $adminRoleService,
     ) {
         parent::__construct(self::$defaultName);
     }
@@ -74,7 +80,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $this->adminService->createAdmin($inputFilter->getValues());
 
-        $output->writeln(Message::ADMIN_CREATED);
+        (new SymfonyStyle($input, $output))->info(Message::ADMIN_CREATED);
 
         return Command::SUCCESS;
     }
@@ -84,7 +90,10 @@ protected function execute(InputInterface $input, OutputInterface $output): int
      */
     private function getData(InputInterface $input): array
     {
-        $role = $this->adminRoleService->findOneBy(['name' => AdminRole::ROLE_ADMIN]);
+        $adminRole = $this->adminRoleService->getAdminRoleRepository()->findOneBy(['name' => AdminRole::ROLE_ADMIN]);
+        if (! $adminRole instanceof AdminRole) {
+            throw new NotFoundException(Message::ROLE_NOT_FOUND);
+        }
 
         return [
             'identity'        => $input->getOption('identity'),
@@ -93,7 +102,7 @@ private function getData(InputInterface $input): array
             'firstName'       => $input->getOption('firstName'),
             'lastName'        => $input->getOption('lastName'),
             'roles'           => [
-                ['uuid' => $role->getUuid()->toString()],
+                ['uuid' => $adminRole->getUuid()->toString()],
             ],
         ];
     }