Skip to content

Commit e5da20f

Browse files
committed
Fix tool name handling
1 parent 0bdf808 commit e5da20f

4 files changed

Lines changed: 15 additions & 3 deletions

File tree

lib/galaxy/celery/tasks.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,9 +92,14 @@ def cached_create_tool_from_representation(
9292
raw_tool_source: str,
9393
tool_source_class: TOOL_SOURCE_CLASS,
9494
tool_dir: Optional[str] = None,
95+
tool_id: Optional[str] = None,
9596
):
9697
return create_tool_from_representation(
97-
app=app, raw_tool_source=raw_tool_source, tool_dir=tool_dir, tool_source_class=tool_source_class
98+
app=app,
99+
raw_tool_source=raw_tool_source,
100+
tool_dir=tool_dir,
101+
tool_source_class=tool_source_class,
102+
guid=tool_id,
98103
)
99104

100105

@@ -441,6 +446,7 @@ def queue_jobs(request: QueueJobs, app: MinimalManagerApp, job_submitter: JobSub
441446
raw_tool_source=raw_tool_source,
442447
tool_dir=request.tool_source.tool_dir,
443448
tool_source_class=tool_source_class,
449+
tool_id=request.tool_source.tool_id,
444450
)
445451

446452
job_submitter.queue_jobs(

lib/galaxy/schema/tasks.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -182,6 +182,7 @@ class ToolSource(Model):
182182
raw_tool_source: str
183183
tool_dir: Optional[str] = None
184184
tool_source_class: TOOL_SOURCE_CLASS = "XmlToolSource"
185+
tool_id: Optional[str] = None
185186

186187

187188
class QueueJobs(Model):

lib/galaxy/tools/__init__.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -469,10 +469,14 @@ def create_tool_from_source(app, tool_source: ToolSource, config_file: Optional[
469469

470470

471471
def create_tool_from_representation(
472-
app, raw_tool_source: str, tool_dir: Optional[StrPath] = None, tool_source_class="XmlToolSource"
472+
app,
473+
raw_tool_source: str,
474+
tool_dir: Optional[StrPath] = None,
475+
tool_source_class="XmlToolSource",
476+
guid: Optional[str] = None,
473477
) -> "Tool":
474478
tool_source = get_tool_source(tool_source_class=tool_source_class, raw_tool_source=raw_tool_source)
475-
return create_tool_from_source(app, tool_source=tool_source, tool_dir=tool_dir)
479+
return create_tool_from_source(app, tool_source=tool_source, tool_dir=tool_dir, guid=guid)
476480

477481

478482
class NullToolTagManager(AbstractToolTagManager):

lib/galaxy/webapps/galaxy/services/jobs.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -282,6 +282,7 @@ def create(self, trans: ProvidesHistoryContext, job_request: JobRequest) -> JobC
282282
raw_tool_source=tool_source_model.source,
283283
tool_dir=tool.tool_dir,
284284
tool_source_class=tool_source_model.source_class,
285+
tool_id=tool.id,
285286
)
286287
task_request = QueueJobs(
287288
user=trans.async_request_user,

0 commit comments

Comments
 (0)