Fixed route provider regarding published state and ghost content (#555)

* Fixed route provider regarding published state and ghost content * Added locale to test * Replaced tab with spaces * Fixed test
sulu · Mar 25, 2021 · b79daf0 · b79daf0
1 parent 077ea21
commit b79daf0
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/Routing/ArticleRouteDefaultProvider.php b/Routing/ArticleRouteDefaultProvider.php
@@ -118,7 +118,13 @@ public function getByEntity($entityClass, $id, $locale, $object = null)
      */
     public function isPublished($entityClass, $id, $locale)
     {
-        $object = $this->documentManager->find($id, $locale);
+        $object = $this->documentManager->find(
+            $id,
+            $locale,
+            [
+                'load_ghost_content' => false,
+            ]
+        );
 
         if (!$object instanceof ArticleInterface || WorkflowStage::PUBLISHED !== $object->getWorkflowStage()) {
             return false;

diff --git a/Tests/Unit/Routing/ArticleRouteDefaultProviderTest.php b/Tests/Unit/Routing/ArticleRouteDefaultProviderTest.php
@@ -130,7 +130,13 @@ public function testIsPublished(
             $this->webspaceResolver->resolveAdditionalWebspaces($document)->willReturn($documentAdditionalWebspaces);
         }
 
-        $this->documentManager->find($this->entityId, $this->locale)->willReturn($document);
+        $this->documentManager->find(
+            $this->entityId,
+            $this->locale,
+            [
+                'load_ghost_content' => false,
+            ]
+        )->willReturn($document);
 
         $webspace = $this->prophesize(Webspace::class);
         $webspace->getKey()->willReturn($webspaceKey);