Set VBP's status to unpublished when BE is not visible on front (and …

…do not show it in sitemap)
Victoire · Apr 1, 2019 · cf3c90e · cf3c90e
1 parent 2bdcf7d
commit cf3c90e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/Bundle/BusinessPageBundle/Builder/BusinessPageBuilder.php b/Bundle/BusinessPageBundle/Builder/BusinessPageBuilder.php
@@ -16,6 +16,7 @@
 use Victoire\Bundle\BusinessPageBundle\Entity\VirtualBusinessPage;
 use Victoire\Bundle\CoreBundle\Exception\IdentifierNotDefinedException;
 use Victoire\Bundle\CoreBundle\Helper\UrlBuilder;
+use Victoire\Bundle\PageBundle\Entity\PageStatus;
 use Victoire\Bundle\ViewReferenceBundle\Builder\ViewReferenceBuilder;
 
 /**
@@ -272,6 +273,9 @@ private function populatePage(VirtualBusinessPage $page, BusinessTemplate $busin
         $page->setSlug($pageSlug);
         $page->setName($pageName);
         $page->setReference($this->viewReferenceBuilder->buildViewReference($page, $em));
+        if (false === $entity->isVisibleOnFront()) {
+            $page->setStatus(PageStatus::UNPUBLISHED);
+        }
 
         return $page;
     }

diff --git a/Bundle/SitemapBundle/Domain/Export/SitemapExportHandler.php b/Bundle/SitemapBundle/Domain/Export/SitemapExportHandler.php
@@ -148,8 +148,10 @@ private function getBusinessPages(ViewReference $tree, $businessPages = [])
             ) {
                 /** @var WebViewInterface $businessPage */
                 $businessPage = $this->pageHelper->findPageByReference($child);
-                $businessPage->setReference($child);
-                $businessPages[] = $businessPage;
+                if ($businessPage->isPublished()) {
+                    $businessPage->setReference($child);
+                    $businessPages[] = $businessPage;
+                }
             }
             $businessPages = $this->getBusinessPages($child, $businessPages);
         }