SonarCloud fixes

src/DependencyInjection/Compiler/SearchModifierHandlerPass.php

@@ -198,7 +198,7 @@ private function checkArgumentInstanceOf(
                     || in_array($classOrInterface, class_implements($type->getName()), true)
                 );
 
-        } catch(Exception) {
+        } catch (Exception) {
             return false;
         }
     }

src/Repository/IndexQueueRepository.php

@@ -56,7 +56,7 @@ public function dispatchableItemExists(): bool
                 ->getOneOrNullResult();
 
             return $result !== null;
-        } catch(NonUniqueResultException) {
+        } catch (NonUniqueResultException) {
             return true;
         }
     }

src/SearchIndexAdapter/OpenSearch/Asset/FieldDefinitionAdapter/AbstractAdapter.php

@@ -121,13 +121,11 @@ protected function validateArray(array $value): void
 
     protected function getSearchFilterFieldPath(AssetMetaDataFilter|AssetMetaDataAggregation $filter): string
     {
-        return implode('.',
-            [
+        return implode('.', [
                 FieldCategory::STANDARD_FIELDS->value,
                 $filter->getName(),
                 $filter->getLanguage() ?? MappingProperty::NOT_LOCALIZED_KEY,
-            ]
-        );
+       ]);
     }
 
     /**

src/SearchIndexAdapter/OpenSearch/IndexStatsService.php

@@ -62,7 +62,7 @@ private function countIndexQueueEntries(): int
     {
         try {
             return $this->indexQueueRepository->countIndexQueueEntries();
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             $this->logger->error('Error while counting index queue entries: '. $e->getMessage());
         }
 

src/SearchIndexAdapter/OpenSearch/PathService.php

@@ -70,8 +70,8 @@ public function rewriteChildrenIndexPaths(ElementInterface $element): void
 
         if ($countResult > $this->searchIndexConfigService->getMaxSynchronousChildrenRenameLimit()) {
             $msg = sprintf(
-                'Direct rewrite of children paths in OpenSearch was skipped as more than %s 
-                items need an update (%s items). 
+                'Direct rewrite of children paths in OpenSearch was skipped as more than %s
+                items need an update (%s items).
                 The index will be updated asynchronously via index update queue command cronjob.',
                 $this->searchIndexConfigService->getMaxSynchronousChildrenRenameLimit(),
                 $countResult

src/SearchIndexAdapter/OpenSearch/QueryLanguage/SubQueriesProcessor.php

@@ -65,7 +65,7 @@ public function processSubQueries(
                     $subQuery->getTargetQuery(),
                     $indexEntity,
                 );
-            } catch(ParsingException $e) {
+            } catch (ParsingException $e) {
                 throw new ParsingException(
                     $originalQuery,
                     $e->getExpected(),

src/SearchIndexAdapter/OpenSearch/Search/SearchExecutionService.php

@@ -75,14 +75,16 @@ public function executeSearch(AdapterSearchInterface $search, string $indexName)
                 throw new ResultWindowTooLargeException(
                     $searchInformation,
                     'Result window too large: ' . $e->getMessage(),
-                    $e->getCode(), $e
+                    $e->getCode(),
+                    $e
                 );
             }
 
             throw new SearchFailedException(
                 $searchInformation,
                 'Search failed: ' . $e->getMessage(),
-                $e->getCode(), $e
+                $e->getCode(),
+                $e
             );
         }
 

src/Service/SearchIndex/IndexQueue/QueueMessageService.php

@@ -39,7 +39,7 @@ public function handleMessage(
         int $entriesCount,
         int $maxBatchSize
     ): void {
-        while(true) {
+        while (true) {
             $entries = $this->indexQueueRepository->getUnhandledIndexQueueEntries(
                 true,
                 $maxBatchSize

src/Traits/ElementNormalizerTrait.php

@@ -91,7 +91,8 @@ private function getTagsByElement(ElementInterface $element): array
         $tag = new Tag();
 
         return $tag->getDao()->getTagsForElement(
-            Service::getElementType($element), $element->getId()
+            Service::getElementType($element),
+            $element->getId()
         );
 
     }