fix urlExists method: returns a bool and ignore nodes with alias and or a disabled url
This commit is contained in:
parent
e4216c8974
commit
2e86ee338c
|
@ -13,11 +13,13 @@ class NodeRepository extends NestedTreeRepository
|
||||||
parent::__construct($manager, $manager->getClassMetadata(Node::class));
|
parent::__construct($manager, $manager->getClassMetadata(Node::class));
|
||||||
}
|
}
|
||||||
|
|
||||||
public function urlExists($url, Node $node)
|
public function urlExists($url, Node $node): bool
|
||||||
{
|
{
|
||||||
$query = $this->createQueryBuilder('n')
|
$query = $this->createQueryBuilder('n')
|
||||||
->join('n.menu', 'm')
|
->join('n.menu', 'm')
|
||||||
->where('n.url = :url')
|
->where('n.url = :url')
|
||||||
|
->andWhere('n.disableUrl = 0')
|
||||||
|
->andWhere('n.aliasNode is null')
|
||||||
->andWhere('m.navigation = :navigation')
|
->andWhere('m.navigation = :navigation')
|
||||||
->setParameter(':url', $url)
|
->setParameter(':url', $url)
|
||||||
->setParameter(':navigation', $node->getMenu()->getNavigation())
|
->setParameter(':navigation', $node->getMenu()->getNavigation())
|
||||||
|
@ -32,7 +34,7 @@ class NodeRepository extends NestedTreeRepository
|
||||||
|
|
||||||
return $query->getQuery()
|
return $query->getQuery()
|
||||||
->setMaxResults(1)
|
->setMaxResults(1)
|
||||||
->getOneOrNullResult()
|
->getOneOrNullResult() !== null
|
||||||
;
|
;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue