From 0301415b8703ee8f13a0b413d314531da35bf9aa Mon Sep 17 00:00:00 2001
From: John Chilton <jmchilton@gmail.com>
Date: Wed, 15 May 2024 10:41:25 -0400
Subject: [PATCH] Revise buttons in ConfigTemplates, fill out remove
 functionality.

---
 .../ConfigTemplates/InstanceDropdown.vue      | 31 +++++++++----------
 .../Instances/InstanceDropdown.vue            | 14 ++++++++-
 .../FileSources/Instances/ManageIndex.vue     |  6 +++-
 .../FileSources/Instances/services.ts         |  8 +++++
 .../Grid/GridElements/GridOperations.vue      |  2 +-
 .../Instances/InstanceDropdown.vue            |  6 +++-
 .../ObjectStore/Instances/ManageIndex.vue     |  6 +++-
 7 files changed, 51 insertions(+), 22 deletions(-)

diff --git a/client/src/components/ConfigTemplates/InstanceDropdown.vue b/client/src/components/ConfigTemplates/InstanceDropdown.vue
index 80c60134495f..3314f58300e8 100644
--- a/client/src/components/ConfigTemplates/InstanceDropdown.vue
+++ b/client/src/components/ConfigTemplates/InstanceDropdown.vue
@@ -2,7 +2,6 @@
 import { library } from "@fortawesome/fontawesome-svg-core";
 import { faArrowUp, faCaretDown, faEdit, faTrash } from "@fortawesome/free-solid-svg-icons";
 import { FontAwesomeIcon } from "@fortawesome/vue-fontawesome";
-import { BLink } from "bootstrap-vue";
 import { useRouter } from "vue-router/composables";
 
 interface Props {
@@ -15,8 +14,6 @@ interface Props {
 
 library.add(faArrowUp, faCaretDown, faEdit, faTrash);
 
-const title = "";
-
 const router = useRouter();
 
 defineProps<Props>();
@@ -28,18 +25,18 @@ const emit = defineEmits<{
 
 <template>
     <div>
-        <BLink
-            v-b-tooltip.hover
-            :class="`${prefix}-instance-dropdown font-weight-bold`"
+        <button
+            id="instance-operations"
+            :class="`${prefix}-instance-dropdown`"
             data-toggle="dropdown"
-            :title="title"
             aria-haspopup="true"
-            aria-expanded="false">
-            <FontAwesomeIcon icon="caret-down" />
-            <span class="instance-dropdown-name">{{ name }}</span>
-        </BLink>
+            aria-expanded="false"
+            class="ui-link font-weight-bold text-nowrap">
+            <FontAwesomeIcon icon="caret-down" class="fa-lg" />
+            <span class="instance-dropdown-name font-weight-bold">{{ name }}</span>
+        </button>
         <div class="dropdown-menu" :aria-labelledby="`${prefix}-instance-dropdown`">
-            <a
+            <button
                 v-if="isUpgradable"
                 class="dropdown-item"
                 :href="routeUpgrade"
@@ -47,19 +44,19 @@ const emit = defineEmits<{
                 @click.prevent="router.push(routeUpgrade)">
                 <FontAwesomeIcon icon="arrowUp" />
                 <span v-localize>Upgrade</span>
-            </a>
-            <a
+            </button>
+            <button
                 class="dropdown-item"
                 :href="routeEdit"
                 @keypress="router.push(routeEdit)"
                 @click.prevent="router.push(routeEdit)">
                 <FontAwesomeIcon icon="edit" />
                 <span v-localize>Edit configuration</span>
-            </a>
-            <a class="dropdown-item" @keypress="emit('remove')" @click.prevent="emit('remove')">
+            </button>
+            <button class="dropdown-item" @keypress="emit('remove')" @click.prevent="emit('remove')">
                 <FontAwesomeIcon icon="trash" />
                 <span v-localize>Remove instance</span>
-            </a>
+            </button>
         </div>
     </div>
 </template>
diff --git a/client/src/components/FileSources/Instances/InstanceDropdown.vue b/client/src/components/FileSources/Instances/InstanceDropdown.vue
index f71e3fd23022..75b93ec5329e 100644
--- a/client/src/components/FileSources/Instances/InstanceDropdown.vue
+++ b/client/src/components/FileSources/Instances/InstanceDropdown.vue
@@ -4,6 +4,8 @@ import { computed } from "vue";
 import type { UserFileSourceModel } from "@/api/fileSources";
 import { useFileSourceTemplatesStore } from "@/stores/fileSourceTemplatesStore";
 
+import { hide } from "./services";
+
 import InstanceDropdown from "@/components/ConfigTemplates/InstanceDropdown.vue";
 
 const fileSourceTemplatesStore = useFileSourceTemplatesStore();
@@ -18,6 +20,15 @@ const routeUpgrade = computed(() => `/file_source_instances/${props.fileSource.i
 const isUpgradable = computed(() =>
     fileSourceTemplatesStore.canUpgrade(props.fileSource.template_id, props.fileSource.template_version)
 );
+
+async function onRemove() {
+    await hide(props.fileSource);
+    emit("entryRemoved");
+}
+
+const emit = defineEmits<{
+    (e: "entryRemoved"): void;
+}>();
 </script>
 
 <template>
@@ -26,5 +37,6 @@ const isUpgradable = computed(() =>
         :name="fileSource.name || ''"
         :is-upgradable="isUpgradable"
         :route-upgrade="routeUpgrade"
-        :route-edit="routeEdit" />
+        :route-edit="routeEdit"
+        @remove="onRemove" />
 </template>
diff --git a/client/src/components/FileSources/Instances/ManageIndex.vue b/client/src/components/FileSources/Instances/ManageIndex.vue
index 60e4b671bf31..dfb965275826 100644
--- a/client/src/components/FileSources/Instances/ManageIndex.vue
+++ b/client/src/components/FileSources/Instances/ManageIndex.vue
@@ -24,6 +24,10 @@ const fields = [NAME_FIELD, DESCRIPTION_FIELD, TYPE_FIELD, TEMPLATE_FIELD];
 const items = computed(() => fileSourceInstancesStore.getInstances);
 const loading = computed(() => fileSourceInstancesStore.loading);
 fileSourceInstancesStore.fetchInstances();
+
+function reload() {
+    fileSourceInstancesStore.fetchInstances();
+}
 </script>
 
 <template>
@@ -52,7 +56,7 @@ fileSourceInstancesStore.fetchInstances();
                 </b-alert>
             </template>
             <template v-slot:cell(name)="row">
-                <InstanceDropdown :file-source="row.item" />
+                <InstanceDropdown :file-source="row.item" @entryRemoved="reload" />
             </template>
             <template v-slot:cell(type)="row">
                 <FileSourceTypeSpan :type="row.item.type" />
diff --git a/client/src/components/FileSources/Instances/services.ts b/client/src/components/FileSources/Instances/services.ts
index d5d4b8553826..7e0b52376a47 100644
--- a/client/src/components/FileSources/Instances/services.ts
+++ b/client/src/components/FileSources/Instances/services.ts
@@ -1,4 +1,12 @@
+import type { UserFileSourceModel } from "@/api/fileSources";
 import { fetcher } from "@/api/schema/fetcher";
 
 export const create = fetcher.path("/api/file_source_instances").method("post").create();
 export const update = fetcher.path("/api/file_source_instances/{user_file_source_id}").method("put").create();
+
+export async function hide(instance: UserFileSourceModel) {
+    const payload = { hidden: true };
+    const args = { user_file_source_id: String(instance?.id) };
+    const { data: fileSource } = await update({ ...args, ...payload });
+    return fileSource;
+}
diff --git a/client/src/components/Grid/GridElements/GridOperations.vue b/client/src/components/Grid/GridElements/GridOperations.vue
index ff85e9832054..62248b224cee 100644
--- a/client/src/components/Grid/GridElements/GridOperations.vue
+++ b/client/src/components/Grid/GridElements/GridOperations.vue
@@ -42,7 +42,7 @@ function hasCondition(conditionHandler: (rowData: RowData, config: GalaxyConfigu
             <FontAwesomeIcon icon="caret-down" class="fa-lg" />
             <span class="font-weight-bold">{{ title }}</span>
         </button>
-        <div class="dropdown-menu" aria-labelledby="dataset-dropdown">
+        <div class="dropdown-menu" aria-labelledby="grid-operations">
             <span v-for="(operation, operationIndex) in operations" :key="operationIndex">
                 <button
                     v-if="operation && (!operation.condition || hasCondition(operation.condition))"
diff --git a/client/src/components/ObjectStore/Instances/InstanceDropdown.vue b/client/src/components/ObjectStore/Instances/InstanceDropdown.vue
index e62239dc6816..00ef24e61eaf 100644
--- a/client/src/components/ObjectStore/Instances/InstanceDropdown.vue
+++ b/client/src/components/ObjectStore/Instances/InstanceDropdown.vue
@@ -23,8 +23,12 @@ const isUpgradable = computed(() =>
 
 async function onRemove() {
     await hide(props.objectStore);
-    console.log("HIDING!!!");
+    emit("entryRemoved");
 }
+
+const emit = defineEmits<{
+    (e: "entryRemoved"): void;
+}>();
 </script>
 
 <template>
diff --git a/client/src/components/ObjectStore/Instances/ManageIndex.vue b/client/src/components/ObjectStore/Instances/ManageIndex.vue
index 0a06ee247c47..7f45d1907211 100644
--- a/client/src/components/ObjectStore/Instances/ManageIndex.vue
+++ b/client/src/components/ObjectStore/Instances/ManageIndex.vue
@@ -35,6 +35,10 @@ const allItems = computed<UserConcreteObjectStore[]>(() => objectStoreInstancesS
 const { activeInstances } = useFiltering(allItems);
 const loading = computed(() => objectStoreInstancesStore.loading);
 objectStoreInstancesStore.fetchInstances();
+
+function reload() {
+    objectStoreInstancesStore.fetchInstances();
+}
 </script>
 
 <template>
@@ -64,7 +68,7 @@ objectStoreInstancesStore.fetchInstances();
                 <ObjectStoreBadges size="1x" :badges="row.item.badges" />
             </template>
             <template v-slot:cell(name)="row">
-                <InstanceDropdown :object-store="row.item" />
+                <InstanceDropdown :object-store="row.item" @entryRemoved="reload" />
             </template>
             <template v-slot:cell(type)="row">
                 <ObjectStoreTypeSpan :type="row.item.type" />