From c0132f6a800c4dd805533a92e836a3353b888d7f Mon Sep 17 00:00:00 2001
From: Ahmed Awan <qe66653@umbc.edu>
Date: Wed, 15 Nov 2023 16:51:52 -0600
Subject: [PATCH 1/3] client: require name for workflows on save, set default
 Unnamed

Also, use `Toast` instead of modal for empty name, as well as
`Toast.success` on newly created workflow
---
 .../src/components/Workflow/Editor/Index.vue  | 30 ++++++++++++-------
 .../components/Workflow/Editor/Options.vue    |  2 +-
 2 files changed, 21 insertions(+), 11 deletions(-)

diff --git a/client/src/components/Workflow/Editor/Index.vue b/client/src/components/Workflow/Editor/Index.vue
index 871521c935f8..73494320a0fa 100644
--- a/client/src/components/Workflow/Editor/Index.vue
+++ b/client/src/components/Workflow/Editor/Index.vue
@@ -165,6 +165,7 @@
 
 <script>
 import axios from "axios";
+import { Toast } from "composables/toast";
 import { storeToRefs } from "pinia";
 import Vue, { computed, onUnmounted, ref, unref } from "vue";
 
@@ -315,7 +316,7 @@ export default {
             license: null,
             creator: null,
             annotation: null,
-            name: null,
+            name: "Unnamed Workflow",
             tags: this.workflowTags,
             stateMessages: [],
             insertedStateMessages: [],
@@ -591,14 +592,7 @@ export default {
             this.$router.replace({ query: { id } });
         },
         async onCreate() {
-            if (!this.name) {
-                const response = "Please provide a name for your workflow.";
-                this.onWorkflowError("Creating workflow failed", response, {
-                    Ok: () => {
-                        this.hideModal();
-                    },
-                });
-                this.onAttributes();
+            if (!this.nameValidate()) {
                 return;
             }
             try {
@@ -610,12 +604,17 @@ export default {
                         workflow_tags: this.tags,
                     };
                     const { data } = await axios.put(`${getAppRoot()}workflow/create`, payload);
-                    const { id } = data;
+                    const { id, message } = data;
 
                     await this.routeToWorkflow(id);
+                    Toast.success(message);
                 } else {
                     // otherwise, use `save_as` endpoint to include steps, etc.
                     await this.doSaveAs(true);
+                    const stepCount = Object.keys(this.steps).length;
+                    Toast.success(
+                        `Created workflow ${this.name} with ${stepCount} ${stepCount === 1 ? "step" : "steps"}.`
+                    );
                 }
             } catch (e) {
                 this.onWorkflowError("Creating workflow failed"),
@@ -627,6 +626,14 @@ export default {
                     };
             }
         },
+        nameValidate() {
+            if (!this.name) {
+                Toast.error("Please provide a name for your workflow.");
+                this.onAttributes();
+                return false;
+            }
+            return true;
+        },
         onSetData(stepId, newData) {
             this.lastQueue
                 .enqueue(() => getModule(newData, stepId, this.stateStore.setLoadingState))
@@ -688,6 +695,9 @@ export default {
             });
         },
         onSave(hideProgress = false) {
+            if (!this.nameValidate()) {
+                return;
+            }
             !hideProgress && this.onWorkflowMessage("Saving workflow...", "progress");
             return saveWorkflow(this)
                 .then((data) => {
diff --git a/client/src/components/Workflow/Editor/Options.vue b/client/src/components/Workflow/Editor/Options.vue
index 1c3f3110e44f..c5e959598772 100644
--- a/client/src/components/Workflow/Editor/Options.vue
+++ b/client/src/components/Workflow/Editor/Options.vue
@@ -77,7 +77,7 @@ async function onSave() {
             @click="$emit('onAttributes')">
             <span class="fa fa-pencil-alt" />
         </BButton>
-        <b-button-group v-b-tooltip class="editor-button-save-group" :title="saveHover">
+        <b-button-group v-b-tooltip.hover.noninteractive class="editor-button-save-group" :title="saveHover">
             <BButton
                 id="workflow-save-button"
                 role="button"

From ea8cffde358c662b41e946d10a4c4534d063025c Mon Sep 17 00:00:00 2001
From: Ahmed Awan <qe66653@umbc.edu>
Date: Thu, 16 Nov 2023 12:38:02 -0600
Subject: [PATCH 2/3] backend: throw error if name is empty for `PUT
 /api/workflows/{id}`

This is handled at the frontend in the editor to prevent this,
but still add additional checks in backend
---
 lib/galaxy/managers/workflows.py           | 2 ++
 lib/galaxy/webapps/galaxy/api/workflows.py | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/lib/galaxy/managers/workflows.py b/lib/galaxy/managers/workflows.py
index 6527839cf160..51a4d85d3777 100644
--- a/lib/galaxy/managers/workflows.py
+++ b/lib/galaxy/managers/workflows.py
@@ -712,6 +712,8 @@ def update_workflow_from_raw_description(
             update_dict = raw_workflow_description.as_dict
             if "name" in update_dict:
                 sanitized_name = sanitize_html(update_dict["name"])
+                if not sanitized_name:
+                    raise exceptions.RequestParameterInvalidException("Workflow must have a valid name")
                 workflow.name = sanitized_name
                 stored_workflow.name = sanitized_name
             if update_dict.get("annotation") is not None:
diff --git a/lib/galaxy/webapps/galaxy/api/workflows.py b/lib/galaxy/webapps/galaxy/api/workflows.py
index 508b2e481887..e362befa9608 100644
--- a/lib/galaxy/webapps/galaxy/api/workflows.py
+++ b/lib/galaxy/webapps/galaxy/api/workflows.py
@@ -475,6 +475,8 @@ def update(self, trans: GalaxyWebTransaction, id, payload, **kwds):
             steps_updated = "steps" in workflow_dict
             if name_updated and not steps_updated:
                 sanitized_name = sanitize_html(new_workflow_name or old_workflow.name)
+                if not sanitized_name:
+                    raise exceptions.MessageException("Workflow must have a valid name.")
                 workflow = old_workflow.copy(user=trans.user)
                 workflow.stored_workflow = stored_workflow
                 workflow.name = sanitized_name

From 3da9c41434f45f209d7a17dfd867bba98d2f53d1 Mon Sep 17 00:00:00 2001
From: Ahmed Awan <qe66653@umbc.edu>
Date: Thu, 16 Nov 2023 14:47:44 -0600
Subject: [PATCH 3/3] add api test for update workflow with empty name

---
 lib/galaxy_test/api/test_workflows.py | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/lib/galaxy_test/api/test_workflows.py b/lib/galaxy_test/api/test_workflows.py
index c41a4cdb7607..73c4064e4055 100644
--- a/lib/galaxy_test/api/test_workflows.py
+++ b/lib/galaxy_test/api/test_workflows.py
@@ -849,6 +849,24 @@ def test_update_name(self):
         workflow_dict = self.workflow_populator.download_workflow(workflow_id)
         assert workflow_dict["license"] == "AAL"
 
+    def test_update_name_empty(self):
+        # Update doesn't allow empty names.
+
+        # Load a workflow with a given name.
+        original_name = "test update name"
+        workflow_object = self.workflow_populator.load_workflow(name=original_name)
+        upload_response = self.__test_upload(workflow=workflow_object, name=original_name)
+        workflow = upload_response.json()
+        assert workflow["name"] == original_name
+
+        # Try to update the name to an empty string (also change steps to force an update).
+        data = {"name": "", "steps": {}}
+        update_response = self._update_workflow(workflow["id"], data)
+        assert update_response.json()["err_msg"] == "Workflow must have a valid name"
+        self._assert_status_code_is(update_response, 400)
+        workflow_dict = self.workflow_populator.download_workflow(workflow["id"])
+        assert workflow_dict["name"] == original_name
+
     def test_refactor(self):
         workflow_id = self.workflow_populator.upload_yaml_workflow(
             """