Fixed issue where when empty notificatonType passed to notification s…

…earch, it attempts match against that empty string Signed-off-by: Jamie Briggs <jamie.briggs@digital.justice.gov.uk>
ministryofjustice · Jan 28, 2025 · e5b9873 · e5b9873
1 parent bfc0d1c
commit e5b9873
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/...rc/main/java/uk/gov/laa/ccms/data/repository/specification/NotificationSpecification.java b/...rc/main/java/uk/gov/laa/ccms/data/repository/specification/NotificationSpecification.java
@@ -92,7 +92,7 @@ public static Specification<Notification> withFilters(
       if (Boolean.FALSE.equals(includeClosed)) {
         predicates.add(criteriaBuilder.equal(root.get("isOpen"), "true"));
       }
-      if (Objects.nonNull(notificationType)) {
+      if (stringNotEmpty(notificationType)) {
         predicates.add(criteriaBuilder.equal(root.get("actionNotificationInd"), notificationType));
       }
       if (Objects.nonNull(dateFrom)) {