cs-fixer

php-etl · Nov 20, 2023 · b39d262 · b39d262
1 parent 29e1b3f
commit b39d262
Show file tree

Hide file tree

Showing 28 changed files with 66 additions and 83 deletions.
diff --git a/src/Builder/Capacity/All.php b/src/Builder/Capacity/All.php
@@ -14,9 +14,7 @@ final class All implements Builder
     private null|Node\Expr $search = null;
     private null|Node\Expr $code = null;
 
-    public function __construct()
-    {
-    }
+    public function __construct() {}
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Create.php b/src/Builder/Capacity/Create.php
@@ -15,9 +15,7 @@ final class Create implements Builder
     private null|Node\Expr $code = null;
     private null|Node\Expr $data = null;
 
-    public function __construct()
-    {
-    }
+    public function __construct() {}
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/ListPerPage.php b/src/Builder/Capacity/ListPerPage.php
@@ -14,9 +14,7 @@ final class ListPerPage implements Builder
     private null|Node\Expr $search = null;
     private null|Node\Expr $code = null;
 
-    public function __construct()
-    {
-    }
+    public function __construct() {}
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Upsert.php b/src/Builder/Capacity/Upsert.php
@@ -15,9 +15,7 @@ final class Upsert implements Builder
     private null|Node\Expr $code = null;
     private null|Node\Expr $data = null;
 
-    public function __construct()
-    {
-    }
+    public function __construct() {}
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Client.php b/src/Builder/Client.php
@@ -27,9 +27,7 @@ final class Client implements Builder
     public const API_SHOP_KEY = 'shop';
     public const API_LEGACY_KEY = 'legacy';
 
-    public function __construct(private readonly Node\Expr $baseUrl, private readonly Node\Expr $clientId, private readonly Node\Expr $secret)
-    {
-    }
+    public function __construct(private readonly Node\Expr $baseUrl, private readonly Node\Expr $clientId, private readonly Node\Expr $secret) {}
 
     public function withToken(Node\Expr $token, Node\Expr $refreshToken): self
     {

diff --git a/src/Builder/Extractor.php b/src/Builder/Extractor.php
@@ -14,9 +14,7 @@ final class Extractor implements StepBuilderInterface
     private ?Node\Expr $client = null;
     private string $apiType;
 
-    public function __construct(private readonly Builder $capacity)
-    {
-    }
+    public function __construct(private readonly Builder $capacity) {}
 
     public function withClient(Node\Expr $client): self
     {
@@ -133,12 +131,12 @@ class: new Node\Stmt\Class_(
 
     public function getParamsNode(): array
     {
-
         $className = match ($this->apiType) {
             Client::API_ADMIN_KEY => \Diglin\Sylius\ApiClient\SyliusAdminClientInterface::class,
             Client::API_LEGACY_KEY => \Diglin\Sylius\ApiClient\SyliusLegacyClientInterface::class,
             Client::API_SHOP_KEY => \Diglin\Sylius\ApiClient\SyliusShopClientInterface::class,
         };
+
         return [
             new Node\Param(
                 var: new Node\Expr\Variable('client'),

diff --git a/src/Builder/Loader.php b/src/Builder/Loader.php
@@ -14,9 +14,7 @@ final class Loader implements StepBuilderInterface
     private ?Node\Expr $client = null;
     private string $apiType;
 
-    public function __construct(private readonly Builder $capacity)
-    {
-    }
+    public function __construct(private readonly Builder $capacity) {}
 
     public function withClient(Node\Expr $client): self
     {
@@ -139,12 +137,12 @@ class: new Node\Stmt\Class_(
 
     public function getParamsNode(): array
     {
-
         $className = match ($this->apiType) {
             Client::API_ADMIN_KEY => \Diglin\Sylius\ApiClient\SyliusAdminClientInterface::class,
             Client::API_LEGACY_KEY => \Diglin\Sylius\ApiClient\SyliusLegacyClientInterface::class,
             Client::API_SHOP_KEY => \Diglin\Sylius\ApiClient\SyliusShopClientInterface::class,
         };
+
         return [
             new Node\Param(
                 var: new Node\Expr\Variable('client'),

diff --git a/src/Capacity/All.php b/src/Capacity/All.php
@@ -115,16 +115,14 @@ final class All implements CapacityInterface
         'order',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter)
-    {
-    }
+    public function __construct(private readonly ExpressionLanguage $interpreter) {}
 
     public function applies(array $config): bool
     {
         if (!isset($config['api_type'])) {
             return false;
         }
-        switch($config['api_type']) {
+        switch ($config['api_type']) {
             case 'admin':
                 $endpoints = self::$endpointsAdmin;
                 $doubleEndpoints = self::$doubleEndpointsAdmin;
@@ -142,6 +140,7 @@ public function applies(array $config): bool
                 $doubleEndpoints = [];
                 break;
         }
+
         return isset($config['type'])
             && (\in_array($config['type'], $endpoints) || \in_array($config['type'], $doubleEndpoints))
             && isset($config['method'])

diff --git a/src/Capacity/Create.php b/src/Capacity/Create.php
@@ -81,11 +81,12 @@ public function applies(array $config): bool
         if (!isset($config['api_type'])) {
             return false;
         }
-        $endpoints = match($config['api_type']) {
+        $endpoints = match ($config['api_type']) {
             'admin' => self::$endpointsAdmin,
-            'shop' =>self::$endpointsShop,
+            'shop' => self::$endpointsShop,
             'legacy' => self::$endpointsLegacy,
         };
+
         return isset($config['type'])
             && \in_array($config['type'], $endpoints)
             && isset($config['method'])

diff --git a/src/Capacity/ListPerPage.php b/src/Capacity/ListPerPage.php
@@ -114,16 +114,14 @@ final class ListPerPage implements CapacityInterface
         'order',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter)
-    {
-    }
+    public function __construct(private readonly ExpressionLanguage $interpreter) {}
 
     public function applies(array $config): bool
     {
         if (!isset($config['api_type'])) {
             return false;
         }
-        switch($config['api_type']) {
+        switch ($config['api_type']) {
             case 'admin':
                 $endpoints = self::$endpointsAdmin;
                 $doubleEndpoints = self::$doubleEndpointsAdmin;
@@ -141,6 +139,7 @@ public function applies(array $config): bool
                 $doubleEndpoints = [];
                 break;
         }
+
         return isset($config['type'])
             && (\in_array($config['type'], $endpoints) || \in_array($config['type'], $doubleEndpoints))
             && isset($config['method'])

diff --git a/src/Capacity/Upsert.php b/src/Capacity/Upsert.php
@@ -73,11 +73,12 @@ public function applies(array $config): bool
         if (!isset($config['api_type'])) {
             return false;
         }
-        $endpoints = match($config['api_type']) {
+        $endpoints = match ($config['api_type']) {
             'admin' => self::$endpointsAdmin,
-            'shop' =>self::$endpointsShop,
+            'shop' => self::$endpointsShop,
             'legacy' => self::$endpointsLegacy,
         };
+
         return isset($config['type'])
             && \in_array($config['type'], $endpoints)
             && isset($config['method'])

diff --git a/src/Configuration.php b/src/Configuration.php
@@ -6,8 +6,6 @@
 
 use Kiboko\Contract\Configurator\PluginConfigurationInterface;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
-use function Kiboko\Component\SatelliteToolbox\Configuration\asExpression;
-use function Kiboko\Component\SatelliteToolbox\Configuration\isExpression;
 
 final class Configuration implements PluginConfigurationInterface
 {

diff --git a/src/Configuration/Extractor.php b/src/Configuration/Extractor.php
@@ -4,17 +4,14 @@
 
 namespace Kiboko\Plugin\Sylius\Configuration;
 
-use Kiboko\Plugin\Sylius\Validator\ApiType;
 use Kiboko\Plugin\Sylius\Validator\ExtractorConfigurationValidator;
-use Kiboko\Plugin\Sylius\Validator\LoaderConfigurationValidator;
 use Symfony\Component\Config;
 
 use function Kiboko\Component\SatelliteToolbox\Configuration\asExpression;
 use function Kiboko\Component\SatelliteToolbox\Configuration\isExpression;
 
 final class Extractor implements Config\Definition\ConfigurationInterface
 {
-
     public function getConfigTreeBuilder(): \Symfony\Component\Config\Definition\Builder\TreeBuilder
     {
         $filters = new Search();
@@ -24,34 +21,34 @@ public function getConfigTreeBuilder(): \Symfony\Component\Config\Definition\Bui
         /* @phpstan-ignore-next-line */
         $builder->getRootNode()
             ->validate()
-                ->always(fn(array $item) => ExtractorConfigurationValidator::validate($item)) //store the item value
+                ->always(fn (array $item) => ExtractorConfigurationValidator::validate($item)) // store the item value
             ->end()
             ->children()
                 ->scalarNode('api_type')
                     ->isRequired()
                     ->cannotBeEmpty()
                     ->validate()
-                        ->always(fn(string $item) => ExtractorConfigurationValidator::validateApiType($item)) //check index of the item value
+                        ->always(fn (string $item) => ExtractorConfigurationValidator::validateApiType($item)) // check index of the item value
                     ->end()
                 ->end()
                 ->scalarNode('type')
                     ->isRequired()
                     ->cannotBeEmpty()
                     ->validate()
-                        ->always(fn(string $item) => ExtractorConfigurationValidator::validateType($item)) //check index of the item value
+                        ->always(fn (string $item) => ExtractorConfigurationValidator::validateType($item)) // check index of the item value
                     ->end()
                 ->end()
                 ->scalarNode('method')
                     ->isRequired()
                     ->cannotBeEmpty()
                     ->validate()
-                        ->always(fn(string $item) => ExtractorConfigurationValidator::validateMethod($item)) //check index of the item value
+                        ->always(fn (string $item) => ExtractorConfigurationValidator::validateMethod($item)) // check index of the item value
                     ->end()
                 ->end()
                 ->scalarNode('code')
                     ->cannotBeEmpty()
                     ->validate()
-                        ->always(fn(string $item) => ExtractorConfigurationValidator::validateCode($item)) //check index of the item value
+                        ->always(fn (string $item) => ExtractorConfigurationValidator::validateCode($item)) // check index of the item value
                     ->end()
                     ->validate()
                         ->ifTrue(isExpression())

diff --git a/src/Configuration/Loader.php b/src/Configuration/Loader.php
@@ -4,13 +4,11 @@
 
 namespace Kiboko\Plugin\Sylius\Configuration;
 
-use Kiboko\Plugin\Sylius\Validator\ApiType;
 use Kiboko\Plugin\Sylius\Validator\LoaderConfigurationValidator;
 use Symfony\Component\Config;
 
 final class Loader implements Config\Definition\ConfigurationInterface
 {
-
     public function getConfigTreeBuilder(): \Symfony\Component\Config\Definition\Builder\TreeBuilder
     {
         $builder = new Config\Definition\Builder\TreeBuilder('loader');
@@ -22,21 +20,21 @@ public function getConfigTreeBuilder(): \Symfony\Component\Config\Definition\Bui
                     ->isRequired()
                     ->cannotBeEmpty()
                     ->validate()
-                        ->always(fn(string $item) => LoaderConfigurationValidator::validateApiType($item))
+                        ->always(fn (string $item) => LoaderConfigurationValidator::validateApiType($item))
                     ->end()
                 ->end()
                 ->scalarNode('type')
                     ->isRequired()
                     ->cannotBeEmpty()
                     ->validate()
-                        ->always(fn(string $item) => LoaderConfigurationValidator::validateType($item))
+                        ->always(fn (string $item) => LoaderConfigurationValidator::validateType($item))
                     ->end()
                 ->end()
                 ->scalarNode('method')
                     ->isRequired()
                     ->cannotBeEmpty()
                     ->validate()
-                        ->always(fn(string $item) => LoaderConfigurationValidator::validateMethod($item))
+                        ->always(fn (string $item) => LoaderConfigurationValidator::validateMethod($item))
                     ->end()
                 ->end()
             ->end()

diff --git a/src/Factory/Client.php b/src/Factory/Client.php
@@ -37,7 +37,7 @@ public function normalize(array $config): array
     {
         try {
             return $this->processor->processConfiguration($this->configuration, $config);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException($exception->getMessage(), 0, $exception);
         }
     }
@@ -110,7 +110,7 @@ public function compile(array $config): Repository\Client
             return new Repository\Client($clientBuilder);
         } catch (Sylius\MissingAuthenticationMethodException $exception) {
             throw new Configurator\InvalidConfigurationException(message: 'Your Sylius API configuration is missing an authentication method, you should either define "username" or "token" options.', previous: $exception);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException(message: $exception->getMessage(), previous: $exception);
         }
     }

diff --git a/src/Factory/Extractor.php b/src/Factory/Extractor.php
@@ -40,7 +40,7 @@ public function normalize(array $config): array
     {
         try {
             return $this->processor->processConfiguration($this->configuration, $config);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException($exception->getMessage(), 0, $exception);
         }
     }

diff --git a/src/Factory/Loader.php b/src/Factory/Loader.php
@@ -39,7 +39,7 @@ public function normalize(array $config): array
     {
         try {
             return $this->processor->processConfiguration($this->configuration, $config);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException($exception->getMessage(), 0, $exception);
         }
     }

diff --git a/src/Factory/NoApplicableCapacityException.php b/src/Factory/NoApplicableCapacityException.php
@@ -4,6 +4,4 @@
 
 namespace Kiboko\Plugin\Sylius\Factory;
 
-final class NoApplicableCapacityException extends \OutOfRangeException
-{
-}
+final class NoApplicableCapacityException extends \OutOfRangeException {}
diff --git a/src/Factory/Repository/RepositoryTrait.php b/src/Factory/Repository/RepositoryTrait.php
@@ -15,7 +15,7 @@ trait RepositoryTrait
     /** @var string[] */
     private array $packages;
 
-    public function addFiles(FileInterface|DirectoryInterface ...$files): Configurator\RepositoryInterface
+    public function addFiles(DirectoryInterface|FileInterface ...$files): Configurator\RepositoryInterface
     {
         array_push($this->files, ...$files);
 

diff --git a/src/Factory/Search.php b/src/Factory/Search.php
@@ -33,7 +33,7 @@ public function normalize(array $config): array
     {
         try {
             return $this->processor->processConfiguration($this->configuration, $config);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException($exception->getMessage(), 0, $exception);
         }
     }
@@ -59,7 +59,7 @@ public function compile(array $config): Repository\Search
             }
 
             return new Repository\Search($builder);
-        } catch (Symfony\InvalidTypeException|Symfony\InvalidConfigurationException $exception) {
+        } catch (Symfony\InvalidConfigurationException|Symfony\InvalidTypeException $exception) {
             throw new Configurator\InvalidConfigurationException(message: $exception->getMessage(), previous: $exception);
         }
     }

diff --git a/src/MissingAuthenticationMethodException.php b/src/MissingAuthenticationMethodException.php
@@ -4,6 +4,4 @@
 
 namespace Kiboko\Plugin\Sylius;
 
-final class MissingAuthenticationMethodException extends \RuntimeException
-{
-}
+final class MissingAuthenticationMethodException extends \RuntimeException {}
diff --git a/src/MissingEndpointException.php b/src/MissingEndpointException.php
@@ -4,6 +4,4 @@
 
 namespace Kiboko\Plugin\Sylius;
 
-final class MissingEndpointException extends \RuntimeException
-{
-}
+final class MissingEndpointException extends \RuntimeException {}
diff --git a/src/MissingParameterException.php b/src/MissingParameterException.php
@@ -4,6 +4,4 @@
 
 namespace Kiboko\Plugin\Sylius;
 
-final class MissingParameterException extends \UnexpectedValueException
-{
-}
+final class MissingParameterException extends \UnexpectedValueException {}