code format commit

php-etl · May 28, 2024 · 4686622 · 4686622
1 parent 235e8d7
commit 4686622
Show file tree

Hide file tree

Showing 33 changed files with 120 additions and 70 deletions.
diff --git a/src/Builder/AlternativeLookup.php b/src/Builder/AlternativeLookup.php
@@ -14,7 +14,8 @@ final class AlternativeLookup implements Builder
 
     public function __construct(
         private readonly Builder $capacity,
-    ) {}
+    ) {
+    }
 
     public function withMerge(Builder $merge): self
     {

diff --git a/src/Builder/Capacity/Extractor/All.php b/src/Builder/Capacity/Extractor/All.php
@@ -10,11 +10,11 @@
 
 final class All implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $search = null;
-    private null|Node\Expr $code = null;
-    private null|Node\Expr $referenceEntity = null;
-    private null|Node\Expr $referenceEntityAttributeCode = null;
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $search = null;
+    private ?Node\Expr $code = null;
+    private ?Node\Expr $referenceEntity = null;
+    private ?Node\Expr $referenceEntityAttributeCode = null;
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Extractor/Get.php b/src/Builder/Capacity/Extractor/Get.php
@@ -10,10 +10,12 @@
 
 final class Get implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $identifier = null;
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $identifier = null;
 
-    public function __construct() {}
+    public function __construct()
+    {
+    }
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Extractor/ListPerPage.php b/src/Builder/Capacity/Extractor/ListPerPage.php
@@ -10,11 +10,13 @@
 
 final class ListPerPage implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $search = null;
-    private null|Node\Expr $code = null;
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $search = null;
+    private ?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/Loader/Create.php b/src/Builder/Capacity/Loader/Create.php
@@ -11,11 +11,11 @@
 
 final class Create implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $code = null;
-    private null|Node\Expr $data = null;
-    private null|Node\Expr $referenceEntity = null;
-    private null|Node\Expr $referenceEntityAttribute = null;
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $code = null;
+    private ?Node\Expr $data = null;
+    private ?Node\Expr $referenceEntity = null;
+    private ?Node\Expr $referenceEntityAttribute = null;
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Loader/Upsert.php b/src/Builder/Capacity/Loader/Upsert.php
@@ -11,11 +11,11 @@
 
 final class Upsert implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $code = null;
-    private null|Node\Expr $data = null;
-    private null|Node\Expr $referenceEntity = null;
-    private null|Node\Expr $referenceEntityAttribute = null;
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $code = null;
+    private ?Node\Expr $data = null;
+    private ?Node\Expr $referenceEntity = null;
+    private ?Node\Expr $referenceEntityAttribute = null;
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Loader/UpsertList.php b/src/Builder/Capacity/Loader/UpsertList.php
@@ -11,10 +11,10 @@
 
 final class UpsertList implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $data = null;
-    private null|Node\Expr $referenceEntity = null;
-    private null|Node\Expr $attributeCode = null;
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $data = null;
+    private ?Node\Expr $referenceEntity = null;
+    private ?Node\Expr $attributeCode = null;
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Lookup/All.php b/src/Builder/Capacity/Lookup/All.php
@@ -10,12 +10,14 @@
 
 final class All implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $search = null;
-    private null|Node\Expr $code = null;
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $search = null;
+    private ?Node\Expr $code = null;
     private string $type = '';
 
-    public function __construct() {}
+    public function __construct()
+    {
+    }
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Lookup/Download.php b/src/Builder/Capacity/Lookup/Download.php
@@ -11,10 +11,12 @@
 
 final class Download implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $file = null;
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $file = null;
 
-    public function __construct() {}
+    public function __construct()
+    {
+    }
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Lookup/Get.php b/src/Builder/Capacity/Lookup/Get.php
@@ -10,12 +10,14 @@
 
 final class Get implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $identifier = null;
-    private null|Node\Expr $code = null;
-    private null|string $type = '';
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $identifier = null;
+    private ?Node\Expr $code = null;
+    private ?string $type = '';
 
-    public function __construct() {}
+    public function __construct()
+    {
+    }
 
     public function withEndpoint(Node\Expr|Node\Identifier $endpoint): self
     {

diff --git a/src/Builder/Capacity/Lookup/ListPerPage.php b/src/Builder/Capacity/Lookup/ListPerPage.php
@@ -10,11 +10,13 @@
 
 final class ListPerPage implements Builder
 {
-    private null|Node\Expr|Node\Identifier $endpoint = null;
-    private null|Node\Expr $search = null;
-    private null|Node\Expr $code = null;
+    private Node\Expr|Node\Identifier|null $endpoint = null;
+    private ?Node\Expr $search = null;
+    private ?Node\Expr $code = 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
@@ -19,7 +19,9 @@ final class Client implements Builder
     private ?Node\Expr $httpStreamFactory = null;
     private ?Node\Expr $fileSystem = null;
 
-    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/ConditionalLookup.php b/src/Builder/ConditionalLookup.php
@@ -14,7 +14,9 @@ final class ConditionalLookup implements StepBuilderInterface
     private iterable $alternatives = [];
     private ?Node\Expr $client = null;
 
-    public function __construct() {}
+    public function __construct()
+    {
+    }
 
     public function withClient(Node\Expr $client): self
     {

diff --git a/src/Builder/Extractor.php b/src/Builder/Extractor.php
@@ -15,7 +15,8 @@ final class Extractor implements StepBuilderInterface
 
     public function __construct(
         private readonly Builder $capacity,
-    ) {}
+    ) {
+    }
 
     public function withClient(Node\Expr $client): self
     {

diff --git a/src/Builder/Inline.php b/src/Builder/Inline.php
@@ -13,7 +13,9 @@
 
 final readonly class Inline implements Builder
 {
-    public function __construct(private ArrayBuilderInterface|ObjectBuilderInterface $mapper) {}
+    public function __construct(private ArrayBuilderInterface|ObjectBuilderInterface $mapper)
+    {
+    }
 
     public function getNode(): Node
     {

diff --git a/src/Builder/Loader.php b/src/Builder/Loader.php
@@ -15,7 +15,8 @@ final class Loader implements StepBuilderInterface
 
     public function __construct(
         private readonly Builder $capacity,
-    ) {}
+    ) {
+    }
 
     public function withClient(Node\Expr $client): self
     {

diff --git a/src/Builder/Lookup.php b/src/Builder/Lookup.php
@@ -12,7 +12,9 @@ final class Lookup implements StepBuilderInterface
     private ?Node\Expr $logger = null;
     private ?Node\Expr $client = null;
 
-    public function __construct(private readonly AlternativeLookup $alternative) {}
+    public function __construct(private readonly AlternativeLookup $alternative)
+    {
+    }
 
     public function withClient(Node\Expr $client): self
     {

diff --git a/src/Builder/Search.php b/src/Builder/Search.php
@@ -11,14 +11,15 @@ final class Search implements Builder
 {
     public function __construct(
         private array $filters = []
-    ) {}
+    ) {
+    }
 
     public function addFilter(
         Node\Expr $field,
         Node\Expr $operator,
-        Node\Expr $value = null,
-        Node\Expr $scope = null,
-        Node\Expr $locale = null
+        ?Node\Expr $value = null,
+        ?Node\Expr $scope = null,
+        ?Node\Expr $locale = null
     ): self {
         $arguments = [
             new Node\Arg(

diff --git a/src/Capacity/Extractor/All.php b/src/Capacity/Extractor/All.php
@@ -55,7 +55,9 @@ final class All implements Akeneo\Capacity\CapacityInterface
         'UNCLASSIFIED',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter) {}
+    public function __construct(private readonly ExpressionLanguage $interpreter)
+    {
+    }
 
     public function applies(array $config): bool
     {

diff --git a/src/Capacity/Extractor/Get.php b/src/Capacity/Extractor/Get.php
@@ -43,7 +43,9 @@ final class Get implements Akeneo\Capacity\CapacityInterface
         'assetManager',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter) {}
+    public function __construct(private readonly ExpressionLanguage $interpreter)
+    {
+    }
 
     public function applies(array $config): bool
     {

diff --git a/src/Capacity/Extractor/ListPerPage.php b/src/Capacity/Extractor/ListPerPage.php
@@ -53,7 +53,9 @@ final class ListPerPage implements Akeneo\Capacity\CapacityInterface
         'UNCLASSIFIED',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter) {}
+    public function __construct(private readonly ExpressionLanguage $interpreter)
+    {
+    }
 
     public function applies(array $config): bool
     {

diff --git a/src/Capacity/Loader/Create.php b/src/Capacity/Loader/Create.php
@@ -8,6 +8,7 @@
 use PhpParser\Builder;
 use PhpParser\Node;
 use Symfony\Component\ExpressionLanguage\ExpressionLanguage;
+
 use function Kiboko\Component\SatelliteToolbox\Configuration\compileValueWhenExpression;
 
 final class Create implements Akeneo\Capacity\CapacityInterface
@@ -16,7 +17,9 @@ final class Create implements Akeneo\Capacity\CapacityInterface
         'productMediaFile',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter) {}
+    public function __construct(private readonly ExpressionLanguage $interpreter)
+    {
+    }
 
     public function applies(array $config): bool
     {
@@ -28,11 +31,10 @@ public function applies(array $config): bool
 
     public function getBuilder(array $config): Builder
     {
-        $builder = (new Akeneo\Builder\Capacity\Loader\Create())
+        return (new Akeneo\Builder\Capacity\Loader\Create())
             ->withEndpoint(endpoint: new Node\Identifier(sprintf('get%sApi', ucfirst((string) $config['type']))))
             ->withCode(code: compileValueWhenExpression($this->interpreter, $config['code'], 'line'))
             ->withData(line: new Node\Expr\Variable('line["product"]'))
         ;
-        return $builder;
     }
 }
diff --git a/src/Capacity/Loader/Upsert.php b/src/Capacity/Loader/Upsert.php
@@ -45,7 +45,9 @@ final class Upsert implements Akeneo\Capacity\CapacityInterface
         'referenceEntity',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter) {}
+    public function __construct(private readonly ExpressionLanguage $interpreter)
+    {
+    }
 
     public function applies(array $config): bool
     {

diff --git a/src/Capacity/Lookup/All.php b/src/Capacity/Lookup/All.php
@@ -54,7 +54,9 @@ final class All implements Akeneo\Capacity\CapacityInterface
         'UNCLASSIFIED',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter) {}
+    public function __construct(private readonly ExpressionLanguage $interpreter)
+    {
+    }
 
     public function applies(array $config): bool
     {

diff --git a/src/Capacity/Lookup/Download.php b/src/Capacity/Lookup/Download.php
@@ -22,7 +22,9 @@ final class Download implements Akeneo\Capacity\CapacityInterface
         'assetMediaFile',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter) {}
+    public function __construct(private readonly ExpressionLanguage $interpreter)
+    {
+    }
 
     public function applies(array $config): bool
     {

diff --git a/src/Capacity/Lookup/Get.php b/src/Capacity/Lookup/Get.php
@@ -43,7 +43,9 @@ final class Get implements Akeneo\Capacity\CapacityInterface
         'assetManager',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter) {}
+    public function __construct(private readonly ExpressionLanguage $interpreter)
+    {
+    }
 
     public function applies(array $config): bool
     {

diff --git a/src/Capacity/Lookup/ListPerPage.php b/src/Capacity/Lookup/ListPerPage.php
@@ -53,7 +53,9 @@ final class ListPerPage implements Akeneo\Capacity\CapacityInterface
         'UNCLASSIFIED',
     ];
 
-    public function __construct(private readonly ExpressionLanguage $interpreter) {}
+    public function __construct(private readonly ExpressionLanguage $interpreter)
+    {
+    }
 
     public function applies(array $config): bool
     {

diff --git a/src/Configuration/Loader.php b/src/Configuration/Loader.php
@@ -175,7 +175,7 @@ final class Loader implements PluginConfigurationInterface
         ],
     ];
 
-    public function getConfigTreeBuilder(): \Symfony\Component\Config\Definition\Builder\TreeBuilder
+    public function getConfigTreeBuilder(): Config\Definition\Builder\TreeBuilder
     {
         $builder = new Config\Definition\Builder\TreeBuilder('loader');