3698: Fix: components keys are symbols, not strings. Tweaked error message to

tell you what the invalid output_of actually is.

diff --git a/sdk/cli/bin/arv-run-pipeline-instance b/sdk/cli/bin/arv-run-pipeline-instance
index 6125343fdcce2b5c36d9367a791df6fe98117bcd..63313fc8082a18b7c637a222d92cabafc76e4ab3 100755 (executable)
--- a/sdk/cli/bin/arv-run-pipeline-instance
+++ b/sdk/cli/bin/arv-run-pipeline-instance
@@ -445,11 +445,10 @@ class WhRunPipelineInstance
         if value.nil? and
             ![false,'false',0,'0'].index parameter[:required]
           if parameter[:output_of]
-            if not @components[parameter[:output_of]]
-              errors << [componentname, parametername, "output_of refers to nonexistent component"]
-            else
-              next
+            if not @components[parameter[:output_of].intern]
+              errors << [componentname, parametername, "output_of refers to nonexistent component '#{parameter[:output_of]}'"]
             end
+            next
           end
           errors << [componentname, parametername, "required parameter is missing"]
         end