Merge pull request #6467 from christianbeeznest/fixes-updates104

Internal: Ensure persisted AccessUrl used during installation to avoid ORM errors

Author: christianbeeznest

public/main/install/install.lib.php

@@ -1510,7 +1510,16 @@ function finishInstallationWithContainer(
     /** @var User $admin */
     $admin = $repo->findOneBy(['username' => 'admin']);
 
-    $accessUrl = Container::getAccessUrlUtil()->getCurrent();
+    $em = Container::getEntityManager();
+    $accessUrlRepo = $em->getRepository(AccessUrl::class);
+
+    $accessUrl = $accessUrlRepo->findOneBy([]);
+
+    if (!$accessUrl) {
+        $accessUrl = Container::getAccessUrlUtil()->getCurrent();
+        $em->persist($accessUrl);
+        $em->flush();
+    }
 
     $admin
         ->setLastname($adminLastName)

src/CoreBundle/DataFixtures/ColorThemeFixtures.php

@@ -6,6 +6,7 @@
 
 namespace Chamilo\CoreBundle\DataFixtures;
 
+use Chamilo\CoreBundle\Entity\AccessUrl;
 use Chamilo\CoreBundle\Entity\AccessUrlRelColorTheme;
 use Chamilo\CoreBundle\Entity\ColorTheme;
 use Chamilo\CoreBundle\Helpers\AccessUrlHelper;
@@ -30,7 +31,6 @@ public function load(ObjectManager $manager): void
             ->findOneBy(['slug' => 'chamilo']);
 
         if ($existing) {
-            echo "Chamilo color theme already exists. Skipping fixture.\n";
             return;
         }
 
@@ -66,6 +66,11 @@ public function load(ObjectManager $manager): void
 
         $accessUrl = $this->accessUrlHelper->getCurrent();
 
+        if (!$accessUrl->getId()) {
+            $manager->persist($accessUrl);
+            $manager->flush();
+        }
+
         $accessUrlRel = (new AccessUrlRelColorTheme())
             ->setUrl($accessUrl)
             ->setColorTheme($theme)
@@ -76,7 +81,5 @@ public function load(ObjectManager $manager): void
         $manager->persist($theme);
         $manager->persist($accessUrlRel);
         $manager->flush();
-
-        echo "Chamilo color theme fixture loaded successfully.\n";
     }
 }