Merge pull request #7452 from christianbeeznest/fixes-updates245

Exercise: Fix exercise list query parameter binding (Doctrine too many parameters)

Author: christianbeeznest

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();