Skip to content

Commit e604a68

Browse files
Exercise: Fix exercise list query parameter binding (Doctrine too many parameters)
1 parent d11708e commit e604a68

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

public/main/exercise/exercise.class.php

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9157,10 +9157,11 @@ public static function exerciseGridResource(
91579157
if ($is_allowedToEdit) {
91589158
$qbPublished = clone $qb;
91599159
$qbDraft = clone $qb;
9160+
$qbPublished->andWhere('links.visibility = :visibility')
9161+
->setParameter('visibility', ResourceLink::VISIBILITY_PUBLISHED);
91609162

9161-
// If the query doesn't contain it, setting the parameter is harmless.
9162-
$qbPublished->setParameter('visibility', ResourceLink::VISIBILITY_PUBLISHED);
9163-
$qbDraft->setParameter('visibility', ResourceLink::VISIBILITY_DRAFT);
9163+
$qbDraft->andWhere('links.visibility = :visibility')
9164+
->setParameter('visibility', ResourceLink::VISIBILITY_DRAFT);
91649165

91659166
$listPublished = $qbPublished->getQuery()->getResult();
91669167
$listDraft = $qbDraft->getQuery()->getResult();

0 commit comments

Comments
 (0)