Updated internal processes to use empty lists as opposed to null

CI-CMG · Aug 7, 2024 · 31ad05d · 31ad05d
1 parent 29e5cff
commit 31ad05d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/...ranslator/src/main/java/edu/colorado/cires/pace/translator/converter/ConversionUtils.java b/...ranslator/src/main/java/edu/colorado/cires/pace/translator/converter/ConversionUtils.java
@@ -255,7 +255,7 @@ public static String stringFromProperty(ValueWithColumnNumber value) {
 
   public static List<Path> pathListFromString(String string) {
     if (string == null) {
-      return null;
+      return Collections.emptyList();
     }
 
     return Arrays.stream(string.split(";"))

diff --git a/...anslator/src/main/java/edu/colorado/cires/pace/translator/converter/PackageConverter.java b/...anslator/src/main/java/edu/colorado/cires/pace/translator/converter/PackageConverter.java
@@ -61,6 +61,7 @@
 import edu.colorado.cires.pace.translator.TranslationException;
 import edu.colorado.cires.pace.translator.ValueWithColumnNumber;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
@@ -508,7 +509,7 @@ private static List<Integer> getChannelNumbers(Map<String, ValueWithColumnNumber
       RuntimeException runtimeException) {
     String rawValue = stringFromMap(properties, property);
     if (StringUtils.isBlank(rawValue)) {
-      return null;
+      return Collections.emptyList();
     }
     return Arrays.stream(rawValue.split(";"))
         .map(v -> {