Skip to content

Commit a0c3976

Browse files
author
Peter Amstutz
committed
Strip leading/trailing whitespace before interpolating.
1 parent 1d2a340 commit a0c3976

File tree

2 files changed

+12
-10
lines changed

2 files changed

+12
-10
lines changed

cwltool/sandboxjs.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,7 @@ def scanner(scan):
104104
return None
105105

106106
def interpolate(scan, jslib):
107+
scan = scan.strip()
107108
parts = []
108109
w = scanner(scan)
109110
while w:

cwltool/update.py

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -110,28 +110,29 @@ def _draftDraft3dev1toDev2(doc):
110110
doc[a] = "$(inputs[%s])" % ']['.join(sp)
111111
else:
112112
sc = updateScript(ent["script"])
113-
if sc == "{":
113+
if sc[0] == "{":
114114
doc[a] = "$" + sc
115115
else:
116116
doc[a] = "$(%s)" % sc
117117
else:
118118
doc[a] = _draftDraft3dev1toDev2(doc[a])
119119

120-
if "class" in doc and (doc["class"] == "CommandLineTool" or doc["class"] == "Workflow"):
121-
found = False
120+
if "class" in doc and (doc["class"] in ("CommandLineTool", "Workflow", "ExpressionTool")):
121+
added = False
122122
if "requirements" in doc:
123123
for r in doc["requirements"]:
124-
if r["class"] == "ExpressionEngineRequirement" and "engineConfig" in r:
125-
doc["requirements"].append({
126-
"class":"InlineJavascriptRequirement",
127-
"expressionLib": [updateScript(sc) for sc in aslist(r["engineConfig"])]
128-
})
124+
if r["class"] == "ExpressionEngineRequirement":
125+
if "engineConfig" in r:
126+
doc["requirements"].append({
127+
"class":"InlineJavascriptRequirement",
128+
"expressionLib": [updateScript(sc) for sc in aslist(r["engineConfig"])]
129+
})
130+
added = True
129131
doc["requirements"] = [rq for rq in doc["requirements"] if rq["class"] != "ExpressionEngineRequirement"]
130-
found = True
131132
break
132133
else:
133134
doc["requirements"] = []
134-
if not found:
135+
if not added:
135136
doc["requirements"].append({"class":"InlineJavascriptRequirement"})
136137

137138
elif isinstance(doc, list):

0 commit comments

Comments
 (0)