From 2e86ee338c69fd4970219222f7d9a9edb4cb7313 Mon Sep 17 00:00:00 2001
From: Simon Vieille <simon@deblan.fr>
Date: Wed, 6 Oct 2021 10:24:42 +0200
Subject: [PATCH] fix urlExists method: returns a bool and ignore nodes with
 alias and or a disabled url

---
 core/Repository/Site/NodeRepository.php | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/core/Repository/Site/NodeRepository.php b/core/Repository/Site/NodeRepository.php
index c83e32e..13a5c73 100644
--- a/core/Repository/Site/NodeRepository.php
+++ b/core/Repository/Site/NodeRepository.php
@@ -13,11 +13,13 @@ class NodeRepository extends NestedTreeRepository
         parent::__construct($manager, $manager->getClassMetadata(Node::class));
     }
 
-    public function urlExists($url, Node $node)
+    public function urlExists($url, Node $node): bool
     {
         $query = $this->createQueryBuilder('n')
             ->join('n.menu', 'm')
             ->where('n.url = :url')
+            ->andWhere('n.disableUrl = 0')
+            ->andWhere('n.aliasNode is null')
             ->andWhere('m.navigation = :navigation')
             ->setParameter(':url', $url)
             ->setParameter(':navigation', $node->getMenu()->getNavigation())
@@ -32,7 +34,7 @@ class NodeRepository extends NestedTreeRepository
 
         return $query->getQuery()
             ->setMaxResults(1)
-            ->getOneOrNullResult()
+            ->getOneOrNullResult() !== null
         ;
     }
 }