Set login-only URL as parent for other access URLs, except the main one - refs BT#22639

Implemented logic to assign the login-only access URL as the parent for other access URLs, excluding the main URL

Author: AngelFQC

src/CoreBundle/Entity/Listener/AccessUrlListener.php

@@ -8,12 +8,15 @@
 
 use Chamilo\CoreBundle\Entity\AccessUrl;
 use Chamilo\CoreBundle\Helpers\AccessUrlHelper;
+use Chamilo\CoreBundle\Repository\Node\AccessUrlRepository;
+use Doctrine\ORM\Event\PostPersistEventArgs;
 use Doctrine\ORM\Event\PrePersistEventArgs;
 
-class AccessUrlListener
+readonly class AccessUrlListener
 {
     public function __construct(
-        private readonly AccessUrlHelper $accessUrlHelper,
+        private AccessUrlHelper $accessUrlHelper,
+        private AccessUrlRepository $accessUrlRepo,
     ) {}
 
     public function prePersist(AccessUrl $accessUrl, PrePersistEventArgs $args): void
@@ -26,7 +29,7 @@ public function prePersist(AccessUrl $accessUrl, PrePersistEventArgs $args): voi
             return;
         }
 
-        if ($loginOnlyAccessUrl = $this->accessUrlHelper->getOnlyLoginAccessUrl()) {
+        if ($loginOnlyAccessUrl = $this->accessUrlRepo->getOnlyLoginAccessUrl()) {
             $accessUrl
                 ->setIsLoginOnly(false)
                 ->setSuperior($loginOnlyAccessUrl)
@@ -45,4 +48,31 @@ public function prePersist(AccessUrl $accessUrl, PrePersistEventArgs $args): voi
             ->setParentResourceNode($firstAccessUrl->resourceNode->getId())
         ;
     }
+
+    public function postPersist(AccessUrl $currentAccessUrl, PostPersistEventArgs $args): void
+    {
+        if (!$currentAccessUrl->isLoginOnly()) {
+            return;
+        }
+
+        $om = $args->getObjectManager();
+
+        /** @var array<int, AccessUrl> $all */
+        $all = $this->accessUrlRepo->findAll();
+
+        $firstAccessUrl = $this->accessUrlHelper->getFirstAccessUrl();
+
+        foreach ($all as $accessUrl) {
+            if (in_array($accessUrl->getId(), [$firstAccessUrl->getId(), $currentAccessUrl->getId()])) {
+                continue;
+            }
+
+            $accessUrl
+                ->setSuperior($currentAccessUrl)
+                ->resourceNode->setParent($currentAccessUrl->resourceNode)
+            ;
+        }
+
+        $om->flush();
+    }
 }

src/CoreBundle/Helpers/AccessUrlHelper.php

@@ -37,11 +37,6 @@ public function getFirstAccessUrl(): ?AccessUrl
         return $this->accessUrlRepository->find($urlId) ?: null;
     }
 
-    public function getOnlyLoginAccessUrl(): ?AccessUrl
-    {
-        return $this->accessUrlRepository->findOneBy(['isLoginOnly' => true]);
-    }
-
     public function getCurrent(): ?AccessUrl
     {
         static $accessUrl;

src/CoreBundle/Repository/Node/AccessUrlRepository.php

@@ -69,4 +69,9 @@ public function getUserActivePortals(User $user): QueryBuilder
             ->setParameter('user', $user->getId())
         ;
     }
+
+    public function getOnlyLoginAccessUrl(): ?AccessUrl
+    {
+        return $this->findOneBy(['isLoginOnly' => true]);
+    }
 }