Fix module CI on Thelia 3: services exclusion and PHPStan errors

ColissimoLabel.php

@@ -101,7 +101,7 @@ class ColissimoLabel extends BaseModule
     /**
      * @param ConnectionInterface|null $con
      */
-    public function postActivation(ConnectionInterface $con = null): void
+    public function postActivation(?ConnectionInterface $con = null): void
     {
         static::checkLabelFolder();
 
@@ -114,7 +114,7 @@ public function postActivation(ConnectionInterface $con = null): void
         $this->checkConfigurationsValues();
     }
 
-    public function update($currentVersion, $newVersion, ConnectionInterface $con = null): void
+    public function update($currentVersion, $newVersion, ?ConnectionInterface $con = null): void
     {
         $finder = Finder::create()
             ->name('*.sql')
@@ -419,7 +419,7 @@ public static function removeAccents($str): bool|string
     public static function configureServices(ServicesConfigurator $servicesConfigurator): void
     {
         $servicesConfigurator->load(self::getModuleCode().'\\', __DIR__)
-            ->exclude([THELIA_MODULE_DIR.ucfirst(self::getModuleCode()).'/I18n/*'])
+            ->exclude([__DIR__.'/I18n/*'])
             ->autowire()
             ->autoconfigure();
     }

Config/module.xml

@@ -13,7 +13,7 @@
         <language>en_US</language>
         <language>fr_FR</language>
     </languages>
-    <version>3.0.2</version>
+    <version>3.0.3</version>
     <authors>
         <author>
             <name>Gilles Bourgeat</name>

EventListeners/GenerateLabelListener.php

@@ -28,7 +28,7 @@ public function __construct(
     ) {
     }
 
-    public function generateLabel(GenerateLabelEvent $event): void
+    public function generateLabel(object $event): void
     {
         $deliveryModuleCode = $event->getOrder()->getModuleRelatedByDeliveryModuleId()?->getCode();
         if ($deliveryModuleCode === "ColissimoHomeDelivery" || $deliveryModuleCode === "ColissimoPickupPoint"|| $deliveryModuleCode === "SoColissimo") {

EventListeners/UpdateDeliveryAddressListener.php

@@ -105,6 +105,7 @@ protected function generateLabel(Order $order, $weight, $signedDelivery): void
 
         /* Check if delivery is a relay point through ColissimoPickupPoint. Use relay point address if it is */
         if (AuthorizedModuleEnum::ColissimoPickupPoint->value === $order->getModuleRelatedByDeliveryModuleId()->getCode() &&
+            class_exists(OrderAddressColissimoPickupPointQuery::class) &&
             null !== ($AddressColissimoPickupPoint = OrderAddressColissimoPickupPointQuery::create()
                 ->findOneById($order->getDeliveryOrderAddressId()))
             && $AddressColissimoPickupPoint) {

Service/LabelService.php

@@ -85,7 +85,8 @@ public function generateLabel($data, EventDispatcherInterface $dispatcher): arra
                 $colissimoRequest = null;
 
                 /* Same thing with ColissimoPickupPoint */
-                if (AuthorizedModuleEnum::ColissimoPickupPoint->value === $order->getModuleRelatedByDeliveryModuleId()->getCode()) {
+                if (AuthorizedModuleEnum::ColissimoPickupPoint->value === $order->getModuleRelatedByDeliveryModuleId()->getCode()
+                    && class_exists(OrderAddressColissimoPickupPointQuery::class)) {
                     if (null !== $AddressColissimoPickupPoint = OrderAddressColissimoPickupPointQuery::create()
                             ->findOneById($order->getDeliveryOrderAddressId())) {
                         /* If the delivery is through a relay point, we create a new LabelRequest with the relay point and order infos */