[airavata] branch master updated: AIRAVATA-3234 Fix empty string test to skip staging optional inputs

[machristie]

This is an automated email from the ASF dual-hosted git repository.

machristie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airavata.git


The following commit(s) were added to refs/heads/master by this push:
     new 7ceaad2  AIRAVATA-3234 Fix empty string test to skip staging optional 
inputs
7ceaad2 is described below

commit 7ceaad21ff9eab5294faff49b7717f81574b12d2
Author: Marcus Christie <machr...@iu.edu>
AuthorDate: Sat Nov 2 16:14:04 2019 -0400

    AIRAVATA-3234 Fix empty string test to skip staging optional inputs
---
 .../apache/airavata/orchestrator/cpi/impl/SimpleOrchestratorImpl.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/modules/orchestrator/orchestrator-core/src/main/java/org/apache/airavata/orchestrator/cpi/impl/SimpleOrchestratorImpl.java
 
b/modules/orchestrator/orchestrator-core/src/main/java/org/apache/airavata/orchestrator/cpi/impl/SimpleOrchestratorImpl.java
index febda65..bd247c8 100644
--- 
a/modules/orchestrator/orchestrator-core/src/main/java/org/apache/airavata/orchestrator/cpi/impl/SimpleOrchestratorImpl.java
+++ 
b/modules/orchestrator/orchestrator-core/src/main/java/org/apache/airavata/orchestrator/cpi/impl/SimpleOrchestratorImpl.java
@@ -417,7 +417,7 @@ public class SimpleOrchestratorImpl extends 
AbstractOrchestrator{
                         break;
                     case URI:
                     case URI_COLLECTION:
-                        if ((processInput.getValue() == null || 
processInput.getValue() == "") && !processInput.isIsRequired()) {
+                        if ((processInput.getValue() == null || 
processInput.getValue().equals("")) && !processInput.isIsRequired()) {
                             logger.debug("Skipping input data staging task for 
{} since value is empty and not required", processInput.getName());
                             break;
                         }