ECS fix

src/Controller/CustomerCreationAction.php

@@ -20,7 +20,7 @@ final class CustomerCreationAction
 
     public function __construct(
         RouterInterface $router,
-        CustomerProviderInterface $customerProvider
+        CustomerProviderInterface $customerProvider,
     ) {
         $this->router = $router;
         $this->customerProvider = $customerProvider;
@@ -38,8 +38,8 @@ public function __invoke(Request $request): Response
                 [
                     'customerId' => $customer->getId(),
                     'channelCode' => $request->attributes->get('channelCode'),
-                ]
-            )
+                ],
+            ),
         );
     }
 }

src/Controller/OrderCreateAction.php

@@ -25,7 +25,7 @@ final class OrderCreateAction
     public function __construct(
         OrderFactoryInterface $orderFactory,
         FormFactoryInterface $formFactory,
-        Environment $twig
+        Environment $twig,
     ) {
         $this->orderFactory = $orderFactory;
         $this->formFactory = $formFactory;

src/Controller/OrderPreviewAction.php

@@ -31,7 +31,7 @@ public function __construct(
         OrderFactoryInterface $orderFactory,
         FormFactoryInterface $formFactory,
         OrderProcessorInterface $orderProcessor,
-        Environment $twig
+        Environment $twig,
     ) {
         $this->orderFactory = $orderFactory;
         $this->formFactory = $formFactory;

src/Controller/ProvideAvailableShippingMethodsAction.php

@@ -20,7 +20,7 @@ final class ProvideAvailableShippingMethodsAction
 
     public function __construct(
         OrderPreparatorInterface $orderPreparator,
-        AvailableShippingMethodsListProvider $availableShippingMethodsListProvider
+        AvailableShippingMethodsListProvider $availableShippingMethodsListProvider,
     ) {
         $this->orderPreparator = $orderPreparator;
         $this->availableShippingMethodsListProvider = $availableShippingMethodsListProvider;

src/DependencyInjection/Compiler/RegisterReorderProcessorsPass.php

@@ -14,7 +14,7 @@ public function __construct()
             'Sylius\AdminOrderCreationPlugin\ReorderProcessing\CompositeReorderProcessor',
             'Sylius\AdminOrderCreationPlugin\ReorderProcessing\CompositeReorderProcessor',
             'sylius_admin_order_creation.reorder_processor',
-            'addProcessor'
+            'addProcessor',
         );
     }
 }

src/EventListener/PaymentLinkCreationListener.php

@@ -28,7 +28,7 @@ final class PaymentLinkCreationListener
     public function __construct(
         PaymentTokenProviderInterface $paymentTokenProvider,
         ObjectManager $orderManager,
-        OrderPaymentLinkSenderInterface $orderPaymentLinkSender
+        OrderPaymentLinkSenderInterface $orderPaymentLinkSender,
     ) {
         $this->paymentTokenProvider = $paymentTokenProvider;
         $this->orderManager = $orderManager;

src/Factory/OrderFactory.php

@@ -33,7 +33,7 @@ public function __construct(
         FactoryInterface $baseOrderFactory,
         CustomerRepositoryInterface $customerRepository,
         ChannelRepositoryInterface $channelRepository,
-        ReorderProcessor $reorderProcessor
+        ReorderProcessor $reorderProcessor,
     ) {
         $this->baseOrderFactory = $baseOrderFactory;
         $this->customerRepository = $customerRepository;

src/Form/Type/CurrencyCodeChoiceType.php

@@ -24,7 +24,7 @@ public function __construct(RepositoryInterface $currencyRepository)
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
         $builder->addModelTransformer(
-            new ReversedTransformer(new ResourceToIdentifierTransformer($this->currencyRepository, 'code'))
+            new ReversedTransformer(new ResourceToIdentifierTransformer($this->currencyRepository, 'code')),
         );
     }
 

src/Form/Type/CustomerAutocompleteChoiceType.php

@@ -23,7 +23,7 @@ public function configureOptions(OptionsResolver $resolver): void
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function buildView(FormView $view, FormInterface $form, array $options): void
     {
@@ -32,15 +32,15 @@ public function buildView(FormView $view, FormInterface $form, array $options):
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function getBlockPrefix(): string
     {
         return 'sylius_customer_autocomplete_choice';
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function getParent(): string
     {

src/Form/Type/LocaleCodeChoiceType.php

@@ -22,23 +22,23 @@ public function __construct(RepositoryInterface $localeRepository)
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
         $builder->addModelTransformer(new ReversedTransformer(new ResourceToIdentifierTransformer($this->localeRepository, 'code')));
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function getParent(): string
     {
         return LocaleChoiceType::class;
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function getBlockPrefix(): string
     {

src/Form/Type/OrderItemType.php

@@ -22,7 +22,7 @@ final class OrderItemType extends AbstractResourceType
     public function __construct(
         string $dataClass,
         DataMapperInterface $dataMapper,
-        array $validationGroups = []
+        array $validationGroups = [],
     ) {
         parent::__construct($dataClass, $validationGroups);
 
@@ -82,7 +82,7 @@ public function configureOptions(OptionsResolver $resolver): void
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function getBlockPrefix(): string
     {

src/Form/Type/PaymentType.php

@@ -19,7 +19,7 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function getBlockPrefix(): string
     {

src/Form/Type/ShipmentType.php

@@ -19,7 +19,7 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
     }
 
     /**
-     * {@inheritdoc}
+     * @inheritdoc
      */
     public function getBlockPrefix(): string
     {

src/Preparator/NewOrderPreparator.php

@@ -26,7 +26,7 @@ final class NewOrderPreparator implements OrderPreparatorInterface
     public function __construct(
         OrderFactoryInterface $orderFactory,
         FormFactoryInterface $formFactory,
-        OrderProcessorInterface $orderProcessor
+        OrderProcessorInterface $orderProcessor,
     ) {
         $this->orderFactory = $orderFactory;
         $this->formFactory = $formFactory;

src/Provider/CustomerProvider.php

@@ -19,7 +19,7 @@ final class CustomerProvider implements CustomerProviderInterface
 
     public function __construct(
         CustomerRepositoryInterface $customerRepository,
-        FactoryInterface $customerFactory
+        FactoryInterface $customerFactory,
     ) {
         $this->customerRepository = $customerRepository;
         $this->customerFactory = $customerFactory;

src/Provider/PaymentTokenProvider.php

@@ -40,14 +40,14 @@ public function getPaymentToken(PaymentInterface $payment): TokenInterface
             return $tokenFactory->createAuthorizeToken(
                 $gatewayConfig->getGatewayName(),
                 $payment,
-                $this->afterPayRoute
+                $this->afterPayRoute,
             );
         }
 
         return $tokenFactory->createCaptureToken(
             $gatewayConfig->getGatewayName(),
             $payment,
-            $this->afterPayRoute
+            $this->afterPayRoute,
         );
     }
 }

src/ReorderProcessing/CompositeReorderProcessor.php

@@ -4,8 +4,8 @@
 
 namespace Sylius\AdminOrderCreationPlugin\ReorderProcessing;
 
-use Sylius\Component\Core\Model\OrderInterface;
 use Laminas\Stdlib\PriorityQueue;
+use Sylius\Component\Core\Model\OrderInterface;
 
 final class CompositeReorderProcessor implements ReorderProcessor
 {

src/ReorderProcessing/ReorderItemsProcessor.php

@@ -25,7 +25,7 @@ final class ReorderItemsProcessor implements ReorderProcessor
     public function __construct(
         FactoryInterface $orderItemFactory,
         OrderItemQuantityModifierInterface $orderItemQuantityModifier,
-        OrderModifierInterface $orderModifier
+        OrderModifierInterface $orderModifier,
     ) {
         $this->orderItemFactory = $orderItemFactory;
         $this->orderItemQuantityModifier = $orderItemQuantityModifier;

src/Sender/OrderPaymentLinkSender.php

@@ -39,7 +39,7 @@ public function sendPaymentLink(OrderInterface $order): void
                 [
                     'order' => $order,
                     'paymentLink' => $paymentDetails['payment-link'],
-                ]
+                ],
             )
         ;
     }

tests/Application/Kernel.php

@@ -103,14 +103,14 @@ private function registerBundlesFromFile(string $bundlesFile): iterable
         if (SyliusCoreBundle::MINOR_VERSION > 10) {
             $contents = array_merge(
                 ['Sylius\Calendar\SyliusCalendarBundle' => ['all' => true]],
-                $contents
+                $contents,
             );
         }
 
         if (SyliusCoreBundle::MINOR_VERSION > 12) {
             $contents = array_merge(
                 ['Sylius\Abstraction\StateMachine\SyliusStateMachineAbstractionBundle' => ['all' => true]],
-                $contents
+                $contents,
             );
         }
 
@@ -131,9 +131,9 @@ private function getBundleListFiles(): array
                 static function (string $directory): string {
                     return $directory . '/bundles.php';
                 },
-                $this->getConfigurationDirectories()
+                $this->getConfigurationDirectories(),
             ),
-            'file_exists'
+            'file_exists',
         );
     }
 

tests/Application/config/bootstrap.php

@@ -1,21 +1,23 @@
 <?php
 
+declare(strict_types=1);
+
 use Symfony\Component\Dotenv\Dotenv;
 
-require dirname(__DIR__).'../../../vendor/autoload.php';
+require dirname(__DIR__) . '../../../vendor/autoload.php';
 
 // Load cached env vars if the .env.local.php file exists
 // Run "composer dump-env prod" to create it (requires symfony/flex >=1.2)
-if (is_array($env = @include dirname(__DIR__).'/.env.local.php')) {
+if (is_array($env = @include dirname(__DIR__) . '/.env.local.php')) {
     $_SERVER += $env;
     $_ENV += $env;
 } elseif (!class_exists(Dotenv::class)) {
     throw new RuntimeException('Please run "composer require symfony/dotenv" to load the ".env" files configuring the application.');
 } else {
     // load all the .env files
-    (new Dotenv())->loadEnv(dirname(__DIR__).'/.env');
+    (new Dotenv())->loadEnv(dirname(__DIR__) . '/.env');
 }
 
 $_SERVER['APP_ENV'] = $_ENV['APP_ENV'] = ($_SERVER['APP_ENV'] ?? $_ENV['APP_ENV'] ?? null) ?: 'dev';
 $_SERVER['APP_DEBUG'] = $_SERVER['APP_DEBUG'] ?? $_ENV['APP_DEBUG'] ?? 'prod' !== $_SERVER['APP_ENV'];
-$_SERVER['APP_DEBUG'] = $_ENV['APP_DEBUG'] = (int) $_SERVER['APP_DEBUG'] || filter_var($_SERVER['APP_DEBUG'], FILTER_VALIDATE_BOOLEAN) ? '1' : '0';
+$_SERVER['APP_DEBUG'] = $_ENV['APP_DEBUG'] = (int) $_SERVER['APP_DEBUG'] || filter_var($_SERVER['APP_DEBUG'], \FILTER_VALIDATE_BOOLEAN) ? '1' : '0';

tests/Application/config/bundles.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 use Sylius\Bundle\CoreBundle\Application\Kernel;
 
 $bundles = [

tests/Behat/Context/Admin/ManagingOrdersContext.php

@@ -112,6 +112,7 @@ public function removeProductFromThisOrder(ProductInterface $product): void
     {
         /** @var ProductVariantInterface $productVariant */
         $productVariant = $product->getVariants()->first();
+
         try {
             $this->orderCreateFormElement->removeProduct($productVariant->getDescriptor());
         } catch (\InvalidArgumentException $exception) {
@@ -210,7 +211,7 @@ public function lowerItemWithProductPriceBy(ProductInterface $product, string $d
         $productCode = $product->getCode();
         $this->orderPreviewPage->lowerItemWithProductPriceBy(
             $productCode,
-            str_replace(['$', '€', '£'], '', $discount)
+            str_replace(['$', '€', '£'], '', $discount),
         );
     }
 
@@ -221,6 +222,7 @@ public function iChangeQuantityOfItemTo(ProductInterface $product, int $quantity
     {
         /** @var ProductVariantInterface $productVariant */
         $productVariant = $product->getVariants()->first();
+
         try {
             $this->orderCreateFormElement->specifyQuantity($productVariant->getDescriptor(), $quantity);
         } catch (\InvalidArgumentException $exception) {
@@ -297,7 +299,7 @@ public function shouldBeNotifiedAboutShippingMethodsSelectionRequirements(): voi
     {
         Assert::same(
             $this->orderCreateFormElement->getShippingMethodsValidationMessage(),
-            'You need to add some items and shipping address to select from eligible shipping method'
+            'You need to add some items and shipping address to select from eligible shipping method',
         );
     }
 
@@ -308,7 +310,7 @@ public function shouldBeNotifiedThatOrderHasBeenSuccessfullyCreated(): void
     {
         $this->notificationChecker->checkNotification(
             'Order has been successfully created',
-            NotificationType::success()
+            NotificationType::success(),
         );
     }
 
@@ -328,7 +330,7 @@ public function shouldBeNotifiedThatItemWithProductDiscountCannotBeBelow0(Produc
         /** @var string $productCode */
         $productCode = $product->getCode();
         Assert::true(
-            $this->orderPreviewPage->hasItemDiscountValidationMessage($productCode, 'Discount cannot be below 0')
+            $this->orderPreviewPage->hasItemDiscountValidationMessage($productCode, 'Discount cannot be below 0'),
         );
     }
 
@@ -363,7 +365,7 @@ public function thereShouldBePaymentLinkSentTo(string $email): void
     {
         Assert::true($this->emailChecker->hasMessageTo(
             'New order has been created for you in Admin panel. Check it out in your orders history. To pay for this order, click',
-            $email
+            $email,
         ));
     }
 
@@ -387,7 +389,7 @@ public function thereShouldBeOneOrderForInTheRegistry(string $channelName, Custo
             'state' => 'New',
             'paymentState' => 'Awaiting payment',
             'shippingState' => 'Ready',
-            'channel' => $channelName
+            'channel' => $channelName,
         ]));
     }
 
@@ -405,7 +407,7 @@ public function thereShouldBeOneOrdersForInTheRegistry(int $amountOfOrders, Cust
                 'state' => 'New',
                 'paymentState' => 'Awaiting payment',
                 'shippingState' => 'Ready',
-            ])
+            ]),
         );
     }
 
@@ -417,7 +419,7 @@ public function thisOrderShippingAddressShouldBe(
         string $street,
         string $postcode,
         string $city,
-        string $countryName
+        string $countryName,
     ): void {
         Assert::true($this->orderShowPage->hasShippingAddress($customerName, $street, $postcode, $city, $countryName));
     }
@@ -430,7 +432,7 @@ public function thisOrderBillingAddressShouldBe(
         string $street,
         string $postcode,
         string $city,
-        string $countryName
+        string $countryName,
     ): void {
         Assert::true($this->orderShowPage->hasBillingAddress($customerName, $street, $postcode, $city, $countryName));
     }