From e604a685b59e969390eb5daf5599bc5a1a4491b7 Mon Sep 17 00:00:00 2001 From: Christian Beeznest Date: Tue, 27 Jan 2026 22:23:49 -0500 Subject: [PATCH] Exercise: Fix exercise list query parameter binding (Doctrine too many parameters) --- public/main/exercise/exercise.class.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/public/main/exercise/exercise.class.php b/public/main/exercise/exercise.class.php index f48a051863b..934a64473bf 100644 --- a/public/main/exercise/exercise.class.php +++ b/public/main/exercise/exercise.class.php @@ -9157,10 +9157,11 @@ public static function exerciseGridResource( if ($is_allowedToEdit) { $qbPublished = clone $qb; $qbDraft = clone $qb; + $qbPublished->andWhere('links.visibility = :visibility') + ->setParameter('visibility', ResourceLink::VISIBILITY_PUBLISHED); - // If the query doesn't contain it, setting the parameter is harmless. - $qbPublished->setParameter('visibility', ResourceLink::VISIBILITY_PUBLISHED); - $qbDraft->setParameter('visibility', ResourceLink::VISIBILITY_DRAFT); + $qbDraft->andWhere('links.visibility = :visibility') + ->setParameter('visibility', ResourceLink::VISIBILITY_DRAFT); $listPublished = $qbPublished->getQuery()->getResult(); $listDraft = $qbDraft->getQuery()->getResult();