NGSTACK-836: add strict types declaration and fix CS

netgen · May 29, 2024 · 4217350 · 4217350
1 parent 1523364
commit 4217350
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/lib/Container/Compiler/ParentChildIndexingPass.php b/lib/Container/Compiler/ParentChildIndexingPass.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Netgen\IbexaSearchExtra\Container\Compiler;
 
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
@@ -26,6 +28,5 @@ public function process(ContainerBuilder $container) :void
             $definition = $container->getDefinition($serviceId);
             $definition->addTag(self::MessageHandlerTag);
         }
-
     }
-}
+}
diff --git a/lib/Core/Search/Common/EventSubscriber/ContentEventSubscriber.php b/lib/Core/Search/Common/EventSubscriber/ContentEventSubscriber.php
@@ -26,11 +26,14 @@
 
 class ContentEventSubscriber implements EventSubscriberInterface
 {
+    /**
+     * @var array<int, int[]>
+     */
     private array $parentLocationIdsByContentId = [];
+
     public function __construct(
         private readonly MessageBusInterface $messageBus,
-        private readonly LocationHandler  $locationHandler,
-
+        private readonly LocationHandler $locationHandler,
     ) {}
 
     public static function getSubscribedEvents(): array
@@ -60,6 +63,7 @@ public function onCopyContent(CopyContentEvent $event): void
     public function onBeforeDeleteContent(BeforeDeleteContentEvent $event): void
     {
         $contentLocations = $this->locationHandler->loadLocationsByContent($event->getContentInfo()->id);
+
         try {
             foreach ($contentLocations as $contentLocation){
                 $this->parentLocationIdsByContentId[$event->getContentInfo()->id][] = $contentLocation->parentId;
@@ -78,6 +82,7 @@ public function onDeleteContent(DeleteContentEvent $event): void
                 $this->parentLocationIdsByContentId[$event->getContentInfo()->id] ?? [],
             ),
         );
+
         unset($this->parentLocationIdsByContentId[$event->getContentInfo()->id]);
     }