From c79f19260cf4be63010876f93f4a123483d7a85d Mon Sep 17 00:00:00 2001
From: Don Walizer <12420708+dwalizer@users.noreply.github.com>
Date: Fri, 8 Dec 2023 12:50:04 -0500
Subject: [PATCH] #2494 Update Spring Boot to 3.1.6, fix querying to handle
 null errors

---
 pom.xml                                       |  4 ++--
 service/pom.xml                               |  1 +
 .../UserActionsHistoryService.groovy          | 12 ++++++++---
 .../repos/UserActionsHistoryRepo.groovy       | 20 +++++++++----------
 4 files changed, 22 insertions(+), 15 deletions(-)

diff --git a/pom.xml b/pom.xml
index 9ed8a1a2a4..54fd7b7d22 100644
--- a/pom.xml
+++ b/pom.xml
@@ -33,7 +33,7 @@
         <guava.version>29.0-jre</guava.version>
 
         <!-- !!!!!IMPORTANT!!!!!: when changing the springboot.version property, make sure you also change it in the spring-boot-starter-parent definition -->
-        <springboot.version>3.1.4</springboot.version>
+        <springboot.version>3.1.6</springboot.version>
 
         <greenmail.version>2.0.0</greenmail.version>
         <wiremock.version>2.27.2</wiremock.version>
@@ -52,7 +52,7 @@
         <groupId>org.springframework.boot</groupId>
         <artifactId>spring-boot-starter-parent</artifactId>
         <!-- !!!!!IMPORTANT!!!!!: when changing this version make sure to also update springboot.version property -->
-        <version>3.1.4</version>
+        <version>3.1.6</version>
         <relativePath/>
     </parent>
 
diff --git a/service/pom.xml b/service/pom.xml
index 36c914a28f..6794abdb11 100644
--- a/service/pom.xml
+++ b/service/pom.xml
@@ -556,6 +556,7 @@
                                 <licenseMerge>The Apache Software License, Version 2.0|Apache License, Version 2.0</licenseMerge>
                                 <licenseMerge>The Apache Software License, Version 2.0|Apache Public License 2.0</licenseMerge>
                                 <licenseMerge>The Apache Software License, Version 2.0|Apache 2.0</licenseMerge>
+                                <licenseMerge>The Apache Software License, Version 2.0|Apache-2.0</licenseMerge>
                                 <licenseMerge>The Apache Software License, Version 2.0|Apache 2</licenseMerge>
                                 <licenseMerge>The Apache Software License, Version 2.0|The Apache License, Version 2.0</licenseMerge>
                                 <licenseMerge>The Apache Software License, Version 2.0|Apache License, version 2.0</licenseMerge>
diff --git a/service/src/main/java/skills/services/userActions/UserActionsHistoryService.groovy b/service/src/main/java/skills/services/userActions/UserActionsHistoryService.groovy
index 6d42b8c706..96956c838f 100644
--- a/service/src/main/java/skills/services/userActions/UserActionsHistoryService.groovy
+++ b/service/src/main/java/skills/services/userActions/UserActionsHistoryService.groovy
@@ -93,8 +93,12 @@ class UserActionsHistoryService {
                                 String quizFilter,
                                 String itemIdFilter,
                                 DashboardAction actionFilter) {
+        String projectIdFilterQuery = projectIdFilter ? '%' + projectIdFilter.toLowerCase() + '%' : null
+        String userFilterQuery = userFilter ? '%' + userFilter.toLowerCase() + '%' : null
+        String quizFilterQuery = quizFilter ? '%' + quizFilter.toLowerCase() + '%' : null
+        String itemIdFilterQuery = itemIdFilter ? '%' + itemIdFilter.toLowerCase() + '%' : null
         Page<UserActionsHistoryRepo.UserActionsPreview> userActionsPreviewFromDB = userActionsHistoryRepo.getActions(
-                projectIdFilter, itemFilter, userFilter, quizFilter, itemIdFilter, actionFilter, pageRequest)
+                projectIdFilterQuery, itemFilter, userFilterQuery, quizFilterQuery, itemIdFilterQuery, actionFilter, pageRequest)
         Long totalRows = userActionsPreviewFromDB.getTotalElements()
         List<DashboardUserActionRes> actionResList = userActionsPreviewFromDB.getContent().collect {
             new DashboardUserActionRes(
@@ -146,8 +150,10 @@ class UserActionsHistoryService {
     }
 
     DashboardUserActionsFilterOptions getUserActionsFilterOptions(String projectId = null, String quizId = null) {
-        List<DashboardAction> dashboardActions = userActionsHistoryRepo.findDistinctDashboardActions(projectId, quizId)
-        List<DashboardItem> dashboardItems = userActionsHistoryRepo.findDistinctDashboardItems(projectId, quizId)
+        String projectIdFilter = projectId ? projectId.toLowerCase() : null
+        String quizIdFilter = quizId ? quizId.toLowerCase() : null
+        List<DashboardAction> dashboardActions = userActionsHistoryRepo.findDistinctDashboardActions(projectIdFilter, quizIdFilter)
+        List<DashboardItem> dashboardItems = userActionsHistoryRepo.findDistinctDashboardItems(projectIdFilter, quizIdFilter)
 
         return new DashboardUserActionsFilterOptions(
                 actionFilterOptions: dashboardActions.collect { it.toString() },
diff --git a/service/src/main/java/skills/storage/repos/UserActionsHistoryRepo.groovy b/service/src/main/java/skills/storage/repos/UserActionsHistoryRepo.groovy
index 1d5b017f47..3f2f7d1af3 100644
--- a/service/src/main/java/skills/storage/repos/UserActionsHistoryRepo.groovy
+++ b/service/src/main/java/skills/storage/repos/UserActionsHistoryRepo.groovy
@@ -58,15 +58,15 @@ interface UserActionsHistoryRepo extends CrudRepository<UserActionsHistory, Long
                     userAttrs.lastName as lastName
                 from UserActionsHistory action, UserAttrs userAttrs
                 where action.userId = userAttrs.userId
-                    and (:projectIdFilter is null OR lower(action.projectId) like lower(concat('%', :projectIdFilter, '%')))
+                    and (:projectIdFilter is null OR lower(action.projectId) like :projectIdFilter)
                     and (:itemFilter is null OR action.item = :itemFilter)
                     and (:userFilter is null OR (
-                        lower(userAttrs.userIdForDisplay) like lower(concat('%', :userFilter, '%'))) OR 
-                        ((lower(CONCAT(userAttrs.firstName, ' ', userAttrs.lastName, ' (',  userAttrs.userIdForDisplay, ')')) like lower(CONCAT('%', :userFilter, '%'))) OR
-                        (lower(CONCAT(userAttrs.userIdForDisplay, ' (', userAttrs.lastName, ', ', userAttrs.firstName,  ')')) like lower(CONCAT('%', :userFilter, '%')))                        )
+                        lower(userAttrs.userIdForDisplay) like :userFilter) OR 
+                        ((lower(CONCAT(userAttrs.firstName, ' ', userAttrs.lastName, ' (',  userAttrs.userIdForDisplay, ')')) like :userFilter) OR
+                        (lower(CONCAT(userAttrs.userIdForDisplay, ' (', userAttrs.lastName, ', ', userAttrs.firstName,  ')')) like :userFilter))
                     )
-                    and (:quizFilter is null OR lower(action.quizId) like lower(concat('%', :quizFilter, '%')))
-                    and (:itemIdFilter is null OR lower(action.itemId) like lower(concat('%', :itemIdFilter, '%')))
+                    and (:quizFilter is null OR lower(action.quizId) like :quizFilter)
+                    and (:itemIdFilter is null OR lower(action.itemId) like :itemIdFilter)
                     and (:actionFilter is null OR action.action = :actionFilter)
     ''')
     Page<UserActionsPreview> getActions(@Nullable @Param("projectIdFilter") String projectIdFilter,
@@ -80,8 +80,8 @@ interface UserActionsHistoryRepo extends CrudRepository<UserActionsHistory, Long
     @Nullable
     @Query('''select distinct action.action
                 from UserActionsHistory action
-                where (:projectId is null OR lower(action.projectId) = lower(:projectId))
-                    and (:quizId is null OR lower(action.quizId) = lower(:quizId))
+                where (:projectId is null OR lower(action.projectId) = :projectId)
+                    and (:quizId is null OR lower(action.quizId) = :quizId)
     ''')
     List<DashboardAction> findDistinctDashboardActions(@Nullable @Param("projectId") String projectId,
                                                         @Nullable @Param("quizId") String quizId)
@@ -89,8 +89,8 @@ interface UserActionsHistoryRepo extends CrudRepository<UserActionsHistory, Long
     @Nullable
     @Query('''select distinct action.item
                 from UserActionsHistory action
-                where (:projectId is null OR lower(action.projectId) = lower(:projectId))
-                    and (:quizId is null OR lower(action.quizId) = lower(:quizId))
+                where (:projectId is null OR lower(action.projectId) = :projectId)
+                    and (:quizId is null OR lower(action.quizId) = :quizId)
     ''')
     List<DashboardItem> findDistinctDashboardItems(@Nullable @Param("projectId") String projectId,
                                                        @Nullable @Param("quizId") String quizId)