Skip to content

Commit

Permalink
Merge pull request #17929 from mvdbeek/rename_pja_fix
Browse files Browse the repository at this point in the history
[24.0] Fix workflow run form for workflows with null rename PJA
  • Loading branch information
mvdbeek authored Apr 8, 2024
2 parents 249681d + 29e9cc3 commit c2f45dd
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions lib/galaxy/workflow/modules.py
Original file line number Diff line number Diff line change
Expand Up @@ -2381,13 +2381,14 @@ def __to_pja(self, key, value, step):
action_arguments = None
return PostJobAction(value["action_type"], step, output_name, action_arguments)

def get_informal_replacement_parameters(self, step) -> List[str]:
def get_informal_replacement_parameters(self, step: WorkflowStep) -> List[str]:
"""Return a list of replacement parameters."""
replacement_parameters = set()
for pja in step.post_job_actions:
for argument in pja.action_arguments.values():
for match in re.findall(r"\$\{(.+?)\}", unicodify(argument)):
replacement_parameters.add(match)
if action_arguments := pja.action_arguments:
for argument in action_arguments.values():
for match in re.findall(r"\$\{(.+?)\}", unicodify(argument)):
replacement_parameters.add(match)

return list(replacement_parameters)

Expand Down

0 comments on commit c2f45dd

Please sign in to comment.