fix compatability with content-bundle (#102)

sulu · Nov 15, 2021 · 8544626 · 8544626
1 parent aed0fca
commit 8544626
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 10 deletions.
diff --git a/Content/ContentTypeResolver/AccountSelectionResolver.php b/Content/ContentTypeResolver/AccountSelectionResolver.php
@@ -58,6 +58,6 @@ public function resolve($data, PropertyInterface $property, string $locale, arra
             $content[] = $this->accountSerializer->serialize($account->getEntity(), $locale, $serializationContext);
         }
 
-        return new ContentView($content, ['ids' => $data ?: []]);
+        return new ContentView($content, ['ids' => $data]);
     }
 }
diff --git a/Content/ContentTypeResolver/CategorySelectionResolver.php b/Content/ContentTypeResolver/CategorySelectionResolver.php
@@ -59,6 +59,6 @@ public function resolve($data, PropertyInterface $property, string $locale, arra
             $serializedCategories[] = $this->categorySerializer->serialize($category, $locale, $serializationContext);
         }
 
-        return new ContentView($serializedCategories, ['ids' => $data ?: []]);
+        return new ContentView($serializedCategories, ['ids' => $data]);
     }
 }
diff --git a/Content/ContentTypeResolver/ContactAccountSelectionResolver.php b/Content/ContentTypeResolver/ContactAccountSelectionResolver.php
@@ -82,6 +82,6 @@ public function resolve($data, PropertyInterface $property, string $locale, arra
             }
         }
 
-        return new ContentView($content, ['ids' => $data ?: []]);
+        return new ContentView($content, ['ids' => $data]);
     }
 }
diff --git a/Content/ContentTypeResolver/ContactSelectionResolver.php b/Content/ContentTypeResolver/ContactSelectionResolver.php
@@ -58,6 +58,6 @@ public function resolve($data, PropertyInterface $property, string $locale, arra
             $content[] = $this->contactSerializer->serialize($contact->getEntity(), $locale, $serializationContext);
         }
 
-        return new ContentView($content, ['ids' => $data ?: []]);
+        return new ContentView($content, ['ids' => $data]);
     }
 }
diff --git a/Content/ContentTypeResolver/PageSelectionResolver.php b/Content/ContentTypeResolver/PageSelectionResolver.php
@@ -100,6 +100,6 @@ public function resolve($data, PropertyInterface $property, string $locale, arra
             $pages[] = $this->structureResolver->resolveProperties($pageStructure, $propertyMap, $locale);
         }
 
-        return new ContentView($pages, ['ids' => $data ?: []]);
+        return new ContentView($pages, ['ids' => $data]);
     }
 }
diff --git a/Content/StructureResolver.php b/Content/StructureResolver.php
@@ -15,7 +15,6 @@
 
 use Sulu\Bundle\DocumentManagerBundle\Bridge\DocumentInspector;
 use Sulu\Bundle\PageBundle\Document\BasePageDocument;
-use Sulu\Bundle\PageBundle\Preview\PageRouteDefaultsProvider;
 use Sulu\Bundle\WebsiteBundle\ReferenceStore\ReferenceStoreNotExistsException;
 use Sulu\Bundle\WebsiteBundle\ReferenceStore\ReferenceStorePoolInterface;
 use Sulu\Component\Content\Compat\PropertyInterface;
@@ -218,8 +217,10 @@ private function getProperties(StructureInterface $targetStructure, StructureInt
      *
      * @return mixed[]
      */
-    private function getStructureData(StructureInterface $targetStructure, StructureInterface $requestedStructure): array
-    {
+    private function getStructureData(
+        StructureInterface $targetStructure,
+        StructureInterface $requestedStructure
+    ): array {
         $targetDocument = $targetStructure->getDocument();
         $requestedDocument = $requestedStructure->getDocument();
 
@@ -416,9 +417,11 @@ private function getTemplateType(StructureInterface $structure, object $document
     private function getTargetStructure(StructureInterface $structure): StructureInterface
     {
         $document = $structure->getDocument();
+        if (!$document instanceof RedirectTypeBehavior) {
+            return $structure;
+        }
 
-        while ($document instanceof RedirectTypeBehavior
-            && RedirectType::INTERNAL === $document->getRedirectType()) {
+        while ($document instanceof RedirectTypeBehavior && RedirectType::INTERNAL === $document->getRedirectType()) {
             $redirectTargetDocument = $document->getRedirectTarget();
 
             if ($redirectTargetDocument instanceof StructureBehavior) {