Ignore blank idFile values in SpectrumSourceFiles table (#597)

vagisha · web-flow · commit 8cbdd1041d80 · 2026-02-04T20:55:52.000-08:00
diff --git a/panoramapublic/src/org/labkey/panoramapublic/speclib/LibSourceFile.java b/panoramapublic/src/org/labkey/panoramapublic/speclib/LibSourceFile.java
@@ -1,6 +1,7 @@
 package org.labkey.panoramapublic.speclib;
 
 import org.apache.commons.io.FilenameUtils;
+import org.apache.commons.lang3.StringUtils;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.List;
@@ -32,7 +33,7 @@ public boolean hasSpectrumSourceFile()
 
     public boolean hasIdFile()
     {
-        return idFile != null;
+        return StringUtils.isNotBlank(idFile);
     }
 
     public @Nullable String getIdFile()

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`package org.labkey.panoramapublic.speclib;`
`2`	`2`
`3`	`3`	`import org.apache.commons.io.FilenameUtils;`
	`4`	`+import org.apache.commons.lang3.StringUtils;`
`4`	`5`	`import org.jetbrains.annotations.Nullable;`
`5`	`6`
`6`	`7`	`import java.util.List;`
`@@ -32,7 +33,7 @@ public boolean hasSpectrumSourceFile()`
`32`	`33`
`33`	`34`	`public boolean hasIdFile()`
`34`	`35`	`{`
`35`		`- return idFile != null;`
	`36`	`+ return StringUtils.isNotBlank(idFile);`
`36`	`37`	`}`
`37`	`38`
`38`	`39`	`public @Nullable String getIdFile()`