IBX-9060: Updated StatusCriterionHandler to use in expression for multi-status support (#627)

tbialcz · web-flow · commit 1c05b96cf760 · 2025-07-30T08:34:13.000+02:00
diff --git a/src/lib/Persistence/Legacy/Notification/Gateway/CriterionHandler/StatusCriterionHandler.php b/src/lib/Persistence/Legacy/Notification/Gateway/CriterionHandler/StatusCriterionHandler.php
@@ -27,7 +27,7 @@ public function supports(CriterionInterface $criterion): bool
 
     public function apply(QueryBuilder $qb, CriterionInterface $criterion): void
     {
-        $qb->andWhere($qb->expr()->eq(DoctrineDatabase::COLUMN_IS_PENDING, ':status'));
-        $qb->setParameter(':status', $criterion->getStatuses(), Connection::PARAM_STR_ARRAY);
+        $qb->andWhere($qb->expr()->in(DoctrineDatabase::COLUMN_IS_PENDING, ':status'));
+        $qb->setParameter(':status', $criterion->getStatuses(), Connection::PARAM_INT_ARRAY);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public function supports(CriterionInterface $criterion): bool`
`27`	`27`
`28`	`28`	`public function apply(QueryBuilder $qb, CriterionInterface $criterion): void`
`29`	`29`	`{`
`30`		`- $qb->andWhere($qb->expr()->eq(DoctrineDatabase::COLUMN_IS_PENDING, ':status'));`
`31`		`- $qb->setParameter(':status', $criterion->getStatuses(), Connection::PARAM_STR_ARRAY);`
	`30`	`+ $qb->andWhere($qb->expr()->in(DoctrineDatabase::COLUMN_IS_PENDING, ':status'));`
	`31`	`+ $qb->setParameter(':status', $criterion->getStatuses(), Connection::PARAM_INT_ARRAY);`
`32`	`32`	`}`
`33`	`33`	`}`