Fixes potential NPE.

cytoscape · Sep 10, 2018 · 8ef30a3 · 8ef30a3
1 parent 055d8ff
commit 8ef30a3
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/jobs-impl/src/main/java/org/cytoscape/jobs/internal/JobStatusBar.java b/jobs-impl/src/main/java/org/cytoscape/jobs/internal/JobStatusBar.java
@@ -114,7 +114,7 @@ public void updateIcon(Collection<CyJobStatus> values) {
 		// 	ERROR/FAILED
 		// 	CANCELED/PURGED/TERMINATED/UNKNOWN
 		// 	QUEUED/SUBMITTTED/RUNNING
-		if (values != null || values.size() > 0) {
+		if (values != null && values.size() > 0) {
 			for (CyJobStatus status: values) {
 				CyJobStatus.Status jobStatus = status.getStatus();
 				if (jobStatus.equals(CyJobStatus.Status.FINISHED)) {