Chill-project/chill-bundles
6
0
Fork 0
mirror of https://gitlab.com/Chill-Projet/chill-bundles.git synced 2025-06-07 18:44:08 +00:00
Code Issues Actions Projects Releases Wiki Activity

[export][person] Fixed: fixed inconsistency in requestor query

Browse Source 
A condition was not present in a subquery, causing to take all courses into account,
not the one concerned by the filter
This commit is contained in:
Julien Fastré 2022-09-21 16:14:07 +02:00
parent 01fb93e9e0
commit ca6fde934b
1 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/Bundle/ChillPersonBundle/Export/Filter/AccompanyingCourseFilters/RequestorFilter.php
View File

@ -70,10 +70,6 @@ final class RequestorFilter implements FilterInterface
case 'other_person': case 'other_person':
$expr = $this->em->getExpressionBuilder(); $expr = $this->em->getExpressionBuilder();
if (!in_array('acppart', $qb->getAllAliases(), true)) {
$qb->join('acp.participations', 'acppart');
}
$clause = $expr->andX( $clause = $expr->andX(
$expr->isNotNull('acp.requestorPerson'), $expr->isNotNull('acp.requestorPerson'),
$expr->notIn( $expr->notIn(
@ -85,6 +81,7 @@ final class RequestorFilter implements FilterInterface
->join('acp2.participations', 'acppart2') ->join('acp2.participations', 'acppart2')
->select('identity(acp2.requestorPerson)') ->select('identity(acp2.requestorPerson)')
->where($expr->eq('acp2.requestorPerson', 'acppart2.person')) ->where($expr->eq('acp2.requestorPerson', 'acppart2.person'))
->andWhere('acp2.id = acp.id')
->getDQL() ->getDQL()
) )
); );