Closes #2171 - Refactor toLowerCopy

common/taskana-common/src/main/java/pro/taskana/common/api/BaseQuery.java

@@ -76,7 +76,7 @@ public interface BaseQuery<T, U extends Enum<U> & QueryColumnName> {
    */
   long count();
 
-  default String[] toLowerCopy(String... source) {
+  static String[] toLowerCopy(String... source) {
     if (source == null || source.length == 0) {
       return null;
       // we are currently aware that this is a code smell. Unfortunately the resolution of this

history/taskana-simplehistory-provider/src/main/java/pro/taskana/simplehistory/impl/ClassificationHistoryQueryImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.simplehistory.impl;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.List;

history/taskana-simplehistory-provider/src/main/java/pro/taskana/simplehistory/impl/TaskHistoryQueryImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.simplehistory.impl;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.List;

history/taskana-simplehistory-provider/src/main/java/pro/taskana/simplehistory/impl/WorkbasketHistoryQueryImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.simplehistory.impl;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.sql.SQLException;
 import java.util.ArrayList;
 import java.util.List;

lib/taskana-core/src/main/java/pro/taskana/classification/internal/ClassificationQueryImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.classification.internal;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;

lib/taskana-core/src/main/java/pro/taskana/monitor/internal/reports/TimeIntervalReportBuilderImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.monitor.internal.reports;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.time.Instant;
 import java.util.Collections;
 import java.util.List;
@@ -628,19 +630,4 @@ abstract class TimeIntervalReportBuilderImpl<
     }
     return false;
   }
-
-  private String[] toLowerCopy(String... source) {
-    if (source == null || source.length == 0) {
-      // we are currently aware that this is a code smell. Unfortunately the resolution of this
-      // would cause havoc in our queries, since we do not have a concept
-      // for a user input validation yet. As soon as that is done we can resolve this code smell.
-      return null;
-    } else {
-      String[] target = new String[source.length];
-      for (int i = 0; i < source.length; i++) {
-        target[i] = source[i].toLowerCase();
-      }
-      return target;
-    }
-  }
 }

lib/taskana-core/src/main/java/pro/taskana/monitor/internal/reports/WorkbasketPriorityReportBuilderImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.monitor.internal.reports;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.util.Collections;
 import java.util.List;
 import pro.taskana.common.api.IntInterval;
@@ -543,19 +545,4 @@ public class WorkbasketPriorityReportBuilderImpl implements WorkbasketPriorityRe
 
     return this;
   }
-
-  private String[] toLowerCopy(String... source) {
-    if (source == null || source.length == 0) {
-      // we are currently aware that this is a code smell. Unfortunately the resolution of this
-      // would cause havoc in our queries, since we do not have a concept
-      // for a user input validation yet. As soon as that is done we can resolve this code smell.
-      return null;
-    } else {
-      String[] target = new String[source.length];
-      for (int i = 0; i < source.length; i++) {
-        target[i] = source[i].toLowerCase();
-      }
-      return target;
-    }
-  }
 }

lib/taskana-core/src/main/java/pro/taskana/task/internal/TaskCommentQueryImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.task.internal;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;

lib/taskana-core/src/main/java/pro/taskana/task/internal/TaskQueryImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.task.internal;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;

lib/taskana-core/src/main/java/pro/taskana/workbasket/internal/WorkbasketAccessItemQueryImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.workbasket.internal;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;

lib/taskana-core/src/main/java/pro/taskana/workbasket/internal/WorkbasketQueryImpl.java

@@ -1,5 +1,7 @@
 package pro.taskana.workbasket.internal;
 
+import static pro.taskana.common.api.BaseQuery.toLowerCopy;
+
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;