Fixed issues reported by phpstan

ibexa · Oct 2, 2023 · d9a5eaa · d9a5eaa
1 parent 5e57fb2
commit d9a5eaa
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 3 deletions.
diff --git a/src/bundle/ApiLoader/BoostFactorProviderFactory.php b/src/bundle/ApiLoader/BoostFactorProviderFactory.php
@@ -7,6 +7,7 @@
 namespace Ibexa\Bundle\Solr\ApiLoader;
 
 use Ibexa\Bundle\Core\ApiLoader\RepositoryConfigurationProvider;
+use Ibexa\Solr\FieldMapper\BoostFactorProvider;
 use Symfony\Component\DependencyInjection\ContainerAwareInterface;
 use Symfony\Component\DependencyInjection\ContainerAwareTrait;
 
@@ -48,6 +49,10 @@ public function __construct(
 
     public function buildService()
     {
+        if ($this->container === null) {
+            return new BoostFactorProvider();
+        }
+
         $repositoryConfig = $this->repositoryConfigurationProvider->getRepositoryConfig();
 
         $connection = $this->defaultConnection;

diff --git a/src/bundle/ApiLoader/IndexingDepthProviderFactory.php b/src/bundle/ApiLoader/IndexingDepthProviderFactory.php
@@ -9,6 +9,7 @@
 namespace Ibexa\Bundle\Solr\ApiLoader;
 
 use Ibexa\Bundle\Core\ApiLoader\RepositoryConfigurationProvider;
+use Ibexa\Solr\FieldMapper\IndexingDepthProvider;
 use Symfony\Component\DependencyInjection\ContainerAwareInterface;
 use Symfony\Component\DependencyInjection\ContainerAwareTrait;
 
@@ -43,6 +44,10 @@ public function __construct(
 
     public function buildService()
     {
+        if ($this->container === null) {
+            return new IndexingDepthProvider();
+        }
+
         $repositoryConfig = $this->repositoryConfigurationProvider->getRepositoryConfig();
 
         $connection = $this->defaultConnection;

diff --git a/src/lib/Query/Common/CriterionVisitor/ContentTypeIdentifierIn.php b/src/lib/Query/Common/CriterionVisitor/ContentTypeIdentifierIn.php
@@ -70,7 +70,7 @@ public function visit(Criterion $criterion, CriterionVisitor $subVisitor = null)
 
         foreach ($criterion->value as $identifier) {
             try {
-                $validIds[] = $contentTypeHandler->loadByIdentifier($identifier)->id;
+                $validIds[] = $contentTypeHandler->loadByIdentifier((string) $identifier)->id;
             } catch (NotFoundException $e) {
                 // Filter out non-existing content types, but track for code below
                 $invalidIdentifiers[] = $identifier;

diff --git a/src/lib/Query/Content/CriterionVisitor/Ancestor.php b/src/lib/Query/Content/CriterionVisitor/Ancestor.php
@@ -36,7 +36,7 @@ public function visit(Criterion $criterion, CriterionVisitor $subVisitor = null)
     {
         $idSet = [];
         foreach ($criterion->value as $value) {
-            foreach (explode('/', trim($value, '/')) as $id) {
+            foreach (explode('/', trim((string)$value, '/')) as $id) {
                 $idSet[$id] = true;
             }
         }

diff --git a/src/lib/Query/Location/CriterionVisitor/Ancestor.php b/src/lib/Query/Location/CriterionVisitor/Ancestor.php
@@ -36,7 +36,7 @@ public function visit(Criterion $criterion, CriterionVisitor $subVisitor = null)
     {
         $idSet = [];
         foreach ($criterion->value as $value) {
-            foreach (explode('/', trim($value, '/')) as $id) {
+            foreach (explode('/', trim((string)$value, '/')) as $id) {
                 $idSet[$id] = true;
             }
         }