[ARVADOS] updated: ac5a69c5c0aaf0786023caa4599a6c55d67310bc

git at public.curoverse.com git at public.curoverse.com
Fri Feb 28 13:56:48 EST 2014


Summary of changes:
 .../controllers/pipeline_instances_controller.rb   |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

       via  ac5a69c5c0aaf0786023caa4599a6c55d67310bc (commit)
      from  7d0e688094e19f54f966747c8dbfbd00e7ce5149 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit ac5a69c5c0aaf0786023caa4599a6c55d67310bc
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Fri Feb 28 13:58:03 2014 -0500

    Fixed handling of optional/required parameters with no specified or default value when constructing initial components hash.

diff --git a/apps/workbench/app/controllers/pipeline_instances_controller.rb b/apps/workbench/app/controllers/pipeline_instances_controller.rb
index 12db3c3..b477223 100644
--- a/apps/workbench/app/controllers/pipeline_instances_controller.rb
+++ b/apps/workbench/app/controllers/pipeline_instances_controller.rb
@@ -59,7 +59,7 @@ class PipelineInstancesController < ApplicationController
                   pipeline[component_name][:script_parameters][param_name] = param_value[:value]
                 elsif param_value[:default]
                   pipeline[component_name][:script_parameters][param_name] = param_value[:default]
-                elsif param_value[:optional] and param_value.length == 1
+                elsif param_value[:optional] != nil or param_value[:required] != nil
                     pipeline[component_name][:script_parameters][param_name] = ""
                 else
                   pipeline[component_name][:script_parameters][param_name] = param_value

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list