OXDEV-9047 Remove static methods access

Author: tkcreateit

CHANGELOG-v10.md

@@ -6,8 +6,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [10.1.0] - unreleased
 
-## Changed
+### Changed
 - Update module to work with OXID eShop 7.3
+- Replaced `TokenFilterList::fromUserInput` and `TokenSorting::fromUserInput` with direct object instantiation
+- Removed static access for `Legacy::createUniqueIdentifier()` and made it an instance method
+- Updated `MissingSignatureKey` exception to use a constructor instead of a static factory method
+
+### Removed
+- `fromUserInput` factory methods from `TokenFilterList` and `TokenSorting`.
 
 ## [10.0.0] - 2024-11-27
 This is stable release for v10.0.0. No changes have been made since v10.0.0-rc.1.

src/Controller/Token.php

@@ -53,13 +53,11 @@ public function tokens(
         ?TokenSorting $sort = null
     ): array {
         return $this->tokenAdministration->tokens(
-            $filter ?? TokenFilterList::fromUserInput(
-                new IDFilter(
-                    $this->authentication->getUser()->id()
-                )
+            $filter ?? new TokenFilterList(
+                new IDFilter($this->authentication->getUser()->id())
             ),
             $pagination ?? new Pagination(),
-            $sort ?? TokenSorting::fromUserInput(Sorting::SORTING_ASC)
+            $sort ?? new TokenSorting(Sorting::SORTING_ASC),
         );
     }
 

src/DataType/Sorting/TokenSorting.php

@@ -9,20 +9,21 @@
 
 namespace OxidEsales\GraphQL\Base\DataType\Sorting;
 
-use TheCodingMachine\GraphQLite\Annotations\Factory;
+use TheCodingMachine\GraphQLite\Annotations\Field;
+use TheCodingMachine\GraphQLite\Annotations\Input;
 
+#[Input]
 final class TokenSorting extends Sorting
 {
     /**
-     * @Factory(name="TokenSorting",default=true)
-     *
      * Tokens will be sorted by their expiration date ('expires_at' column).
      */
-    public static function fromUserInput(
-        string $expiresAt = self::SORTING_ASC
-    ): self {
-        return new self([
-            'expires_at' => $expiresAt,
+    public function __construct(
+        #[Field]
+        private string $expiresAt = self::SORTING_ASC,
+    ) {
+        parent::__construct([
+            'expires_at' => $this->expiresAt,
         ]);
     }
 }

src/DataType/TokenFilterList.php

@@ -13,13 +13,18 @@
 use OxidEsales\GraphQL\Base\DataType\Filter\DateFilter;
 use OxidEsales\GraphQL\Base\DataType\Filter\FilterListInterface;
 use OxidEsales\GraphQL\Base\DataType\Filter\IDFilter;
-use TheCodingMachine\GraphQLite\Annotations\Factory;
+use TheCodingMachine\GraphQLite\Annotations\Field;
+use TheCodingMachine\GraphQLite\Annotations\Input;
 
+#[Input]
 final class TokenFilterList implements FilterListInterface
 {
     public function __construct(
+        #[Field]
         private readonly ?IDFilter $customerId = null,
+        #[Field]
         private readonly ?IDFilter $shopId = null,
+        #[Field]
         private readonly ?DateFilter $expiresAt = null
     ) {
     }
@@ -59,15 +64,4 @@ public function getFilters(): array
             'expires_at' => $this->expiresAt,
         ];
     }
-
-    /**
-     * @Factory(name="TokenFilterList",default=true)
-     */
-    public static function fromUserInput(
-        ?IDFilter $customerId,
-        ?IDFilter $shopId = null,
-        ?DateFilter $expiresAt = null
-    ): self {
-        return new self($customerId, $shopId, $expiresAt);
-    }
 }

src/Exception/MissingSignatureKey.php

@@ -13,8 +13,8 @@ class MissingSignatureKey extends Error
 {
     protected const WRONG_SIZE_MESSAGE = 'Signature key is too short';
 
-    public static function wrongSize(): self
+    public function __construct()
     {
-        return new self(self::WRONG_SIZE_MESSAGE);
+        parent::__construct(self::WRONG_SIZE_MESSAGE);
     }
 }

src/Infrastructure/Legacy.php

@@ -48,7 +48,7 @@ public function login(?string $username = null, ?string $password = null): UserI
             return new User($user, false);
         }
 
-        $user->setId(self::createUniqueIdentifier());
+        $user->setId($this->createUniqueIdentifier());
         return new User($user, true);
     }
 
@@ -126,7 +126,7 @@ public function getUserGroupIds(?string $userId): array
         return $userGroupIds;
     }
 
-    public static function createUniqueIdentifier(): string
+    public function createUniqueIdentifier(): string
     {
         $utils = Registry::getUtilsObject();
         return $utils->generateUId();

src/Service/ModuleConfiguration.php

@@ -77,7 +77,7 @@ public function getSignatureKey(): string
             ->toString();
 
         if (strlen($signature) < 64) {
-            throw MissingSignatureKey::wrongSize();
+            throw new MissingSignatureKey();
         }
 
         return $signature;

src/Service/Token.php

@@ -92,7 +92,7 @@ public function createTokenForUser(UserInterface $user): UnencryptedToken
             ->withClaim(self::CLAIM_USERNAME, $user->email())
             ->withClaim(self::CLAIM_USERID, $user->id()->val())
             ->withClaim(self::CLAIM_USER_ANONYMOUS, $user->isAnonymous())
-            ->withClaim(self::CLAIM_TOKENID, Legacy::createUniqueIdentifier());
+            ->withClaim(self::CLAIM_TOKENID, $this->legacyInfrastructure->createUniqueIdentifier());
 
         $event = new BeforeTokenCreation($builder, $user);
         $this->eventDispatcher->dispatch(

tests/PhpMd/phpmd.baseline.xml

@@ -1,13 +1,10 @@
 <?xml version="1.0"?>
 <phpmd-baseline>
   <violation rule="PHPMD\Rule\CleanCode\StaticAccess" file="src/Component/Widget/GraphQL.php"/>
-  <violation rule="PHPMD\Rule\CleanCode\StaticAccess" file="src/Controller/Token.php"/>
   <violation rule="PHPMD\Rule\Naming\ShortMethodName" file="src/DataType/RefreshTokenInterface.php" method="id"/>
   <violation rule="PHPMD\Rule\Naming\ShortMethodName" file="src/DataType/UserInterface.php" method="id"/>
   <violation rule="PHPMD\Rule\CleanCode\StaticAccess" file="src/Framework/GraphQLQueryHandler.php"/>
   <violation rule="PHPMD\Rule\CleanCode\StaticAccess" file="src/Framework/RequestReader.php"/>
   <violation rule="PHPMD\Rule\CleanCode\StaticAccess" file="src/Infrastructure/Legacy.php"/>
   <violation rule="PHPMD\Rule\CleanCode\StaticAccess" file="src/Service/JwtConfigurationBuilder.php"/>
-  <violation rule="PHPMD\Rule\CleanCode\StaticAccess" file="src/Service/ModuleConfiguration.php"/>
-  <violation rule="PHPMD\Rule\CleanCode\StaticAccess" file="src/Service/Token.php"/>
 </phpmd-baseline>

tests/Unit/Controller/LoginTest.php

@@ -72,6 +72,7 @@ public function testCreateTokenWithValidCredentials(): void
         $user = new User($userModelStub);
 
         $this->legacy->method('login')->with($username, $password)->willReturn($user);
+        $this->legacy->method('createUniqueIdentifier')->willReturn(uniqid());
 
         $loginController = new Login(
             $this->tokenService,