From b9bb92cd1fe3b9533fc3a12ff086ff78f1ec0b7c Mon Sep 17 00:00:00 2001 From: mvdbeek Date: Wed, 10 May 2023 12:53:15 +0200 Subject: [PATCH] Don't build tool state for first param combination It'll always pull in the first dataset in the `other_values` dict, which isn'r correct. Tool parameters at this point know how to deal with collections, so this shouldn't be neceesary. --- lib/galaxy/tools/__init__.py | 6 ------ 1 file changed, 6 deletions(-) diff --git a/lib/galaxy/tools/__init__.py b/lib/galaxy/tools/__init__.py index a27c6ea904b..c4b87ff10ba 100644 --- a/lib/galaxy/tools/__init__.py +++ b/lib/galaxy/tools/__init__.py @@ -2437,12 +2437,6 @@ class Tool(Dictifiable): # create parameter object params = Params(kwd, sanitize=False) - # expand incoming parameters (parameters might trigger multiple tool executions, - # here we select the first execution only in order to resolve dynamic parameters) - expanded_incomings, _ = expand_meta_parameters(trans, self, params.__dict__) - if expanded_incomings: - params.__dict__ = expanded_incomings[0] - # do param translation here, used by datasource tools if self.input_translator: self.input_translator.translate(params)