Fix CS

ecs.php

@@ -4,22 +4,18 @@
 
 use PhpCsFixer\Fixer\ClassNotation\VisibilityRequiredFixer;
 use Symplify\EasyCodingStandard\Config\ECSConfig;
-use Symplify\EasyCodingStandard\ValueObject\Option;
 
 return static function (ECSConfig $ecsConfig): void {
     $ecsConfig->import('vendor/sylius-labs/coding-standard/ecs.php');
 
-    $params = $ecsConfig->parameters();
-
-    $params->set(Option::PATHS, [
+    $ecsConfig->paths([
         'src',
         'tests/Behat',
         'tests/Integration',
         'tests/Stub',
     ]);
-    $ecsConfig->skip(['src/Kernel.php']);
-
-    $ecsConfig->parameters()->set(Option::SKIP, [
+    $ecsConfig->skip([
+        'src/Kernel.php',
         VisibilityRequiredFixer::class => ['*Spec.php'],
     ]);
 };

src/MessageHandler/Connection/ConnectionCreateHandler.php

@@ -52,7 +52,8 @@ public function __invoke(ConnectionCreate $message): void
         if ($activeCampaignId !== null) {
             throw new InvalidArgumentException(sprintf('The Channel with id "%s" has been already created on ActiveCampaign on the connection with id "%s"', $channelId, $activeCampaignId));
         }
-        try{
+
+        try {
             $createConnectionResponse = $this->activeCampaignConnectionClient->create($this->connectionMapper->mapFromChannel($channel));
         } catch (\Throwable $e) {
             $this->logger?->error($e->getMessage(), $e->getTrace());

src/MessageHandler/Connection/ConnectionUpdateHandler.php

@@ -52,6 +52,7 @@ public function __invoke(ConnectionUpdate $message): void
         if ($activeCampaignId !== $message->getActiveCampaignId()) {
             throw new InvalidArgumentException(sprintf('The Channel with id "%s" has an ActiveCampaign id that does not match. Expected "%s", given "%s".', $channelId, $message->getActiveCampaignId(), (string) $activeCampaignId));
         }
+
         try {
             $this->activeCampaignConnectionClient->update($message->getActiveCampaignId(), $this->connectionMapper->mapFromChannel($channel));
         } catch (\Throwable $e) {

src/MessageHandler/Contact/ContactCreateHandler.php

@@ -54,6 +54,7 @@ public function __invoke(ContactCreate $message): void
         if ($activeCampaignId !== null) {
             throw new InvalidArgumentException(sprintf('The Customer with id "%s" has been already created on ActiveCampaign on the contact with id "%s"', $customerId, $activeCampaignId));
         }
+
         try {
             $createContactResponse = $this->activeCampaignContactClient->create($this->contactMapper->mapFromCustomer($customer));
             $activeCampaignContactId = $createContactResponse->getResourceResponse()->getId();

src/MessageHandler/Contact/ContactUpdateHandler.php

@@ -52,7 +52,8 @@ public function __invoke(ContactUpdate $message): void
         if ($activeCampaignId !== $message->getActiveCampaignId()) {
             throw new InvalidArgumentException(sprintf('The Customer with id "%s" has an ActiveCampaign id that does not match. Expected "%s", given "%s".', $customerId, $message->getActiveCampaignId(), (string) $activeCampaignId));
         }
-        try{
+
+        try {
             $this->activeCampaignContactClient->update($message->getActiveCampaignId(), $this->contactMapper->mapFromCustomer($customer));
         } catch (\Throwable $e) {
             $this->logger?->error($e->getMessage(), $e->getTrace());

src/MessageHandler/EcommerceCustomer/EcommerceCustomerCreateHandler.php

@@ -76,6 +76,7 @@ public function __invoke(EcommerceCustomerCreate $message): void
 
             throw new InvalidArgumentException(sprintf('The Customer with id "%s" has been already created on ActiveCampaign on the EcommerceCustomer with id "%s"', $customerId, $activeCampaignId));
         }
+
         try {
             $response = $this->activeCampaignClient->create($this->ecommerceCustomerMapper->mapFromCustomerAndChannel($customer, $channel));
         } catch (\Throwable $e) {

src/MessageHandler/EcommerceCustomer/EcommerceCustomerUpdateHandler.php

@@ -70,6 +70,7 @@ public function __invoke(EcommerceCustomerUpdate $message): void
         if ($activeCampaignId !== $message->getActiveCampaignId()) {
             throw new InvalidArgumentException(sprintf('The Customer with id "%s" has an ActiveCampaign id that does not match. Expected "%s", given "%s".', $customerId, $message->getActiveCampaignId(), (string) $activeCampaignId));
         }
+
         try {
             $this->activeCampaignClient->update($message->getActiveCampaignId(), $this->ecommerceCustomerMapper->mapFromCustomerAndChannel($customer, $channel));
         } catch (\Throwable $e) {

src/MessageHandler/EcommerceOrder/EcommerceOrderCreateHandler.php

@@ -52,6 +52,7 @@ public function __invoke(EcommerceOrderCreate $message): void
         if ($activeCampaignId !== null) {
             throw new InvalidArgumentException(sprintf('The Order with id "%s" has been already created on ActiveCampaign on the ecommerce order with id "%s"', $orderId, $activeCampaignId));
         }
+
         try {
             $response = $this->activeCampaignEcommerceOrderClient->create($this->ecommerceOrderMapper->mapFromOrder($order, $message->isInRealTime()));
         } catch (\Throwable $e) {

src/MessageHandler/EcommerceOrder/EcommerceOrderUpdateHandler.php

@@ -52,6 +52,7 @@ public function __invoke(EcommerceOrderUpdate $message): void
         if ($activeCampaignId !== $message->getActiveCampaignId()) {
             throw new InvalidArgumentException(sprintf('The Order with id "%s" has an ActiveCampaign id that does not match. Expected "%s", given "%s".', $orderId, $message->getActiveCampaignId(), (string) $activeCampaignId));
         }
+
         try {
             $this->activeCampaignEcommerceOrderClient->update($message->getActiveCampaignId(), $this->ecommerceOrderMapper->mapFromOrder($order, $message->isInRealTime()));
         } catch (\Throwable $e) {

src/MessageHandler/Webhook/WebhookCreateHandler.php

@@ -53,6 +53,7 @@ public function __invoke(WebhookCreate $message): void
         if ($activeCampaignListId === null) {
             throw new InvalidArgumentException(sprintf('The Channel with id "%s" does not have an ActiveCampaign list id.', $channelId));
         }
+
         try {
             $this->activeCampaignWebhookClient->create($this->webhookMapper->map(
                 sprintf('Update Sylius newsletter subscription to list "%s"', $activeCampaignListId),

src/WebgriffeSyliusActiveCampaignPlugin.php

@@ -4,10 +4,10 @@
 
 namespace Webgriffe\SyliusActiveCampaignPlugin;
 
+use function dirname;
 use Sylius\Bundle\CoreBundle\Application\SyliusPluginTrait;
 use Sylius\Bundle\ResourceBundle\AbstractResourceBundle;
 use Sylius\Bundle\ResourceBundle\SyliusResourceBundle;
-use function dirname;
 
 final class WebgriffeSyliusActiveCampaignPlugin extends AbstractResourceBundle
 {

tests/Integration/Command/EnqueueContactAndEcommerceCustomerCommandTest.php

@@ -189,7 +189,7 @@ private static function getMessageFromCustomerTypeAndChannel(
         array $messages,
         CustomerInterface $customer,
         string $messageClass,
-        ChannelInterface $channel = null
+        ChannelInterface $channel = null,
     ): Envelope {
         $messages = array_filter($messages, static function (Envelope $envelope) use ($customer, $messageClass, $channel) {
             $message = $envelope->getMessage();

tests/Integration/EventSubscriber/CustomerSubscriberTest.php

@@ -131,7 +131,7 @@ private static function getMessageFromCustomerTypeAndChannel(
         array $messages,
         CustomerInterface $customer,
         string $messageClass,
-        ChannelInterface $channel = null
+        ChannelInterface $channel = null,
     ): Envelope {
         $messages = array_filter($messages, static function (Envelope $envelope) use ($customer, $messageClass, $channel) {
             $message = $envelope->getMessage();