From a1c2c09c3bca0e5ba2c77cc5827703a2036caa43 Mon Sep 17 00:00:00 2001
From: IshavSohal <ishav.sohal@mail.utoronto.ca>
Date: Fri, 21 Feb 2025 14:55:34 -0500
Subject: [PATCH] fix behavior of uuid input on enter press

---
 src/components/metadata-editor.vue | 48 ++++++++++++++++++++----------
 1 file changed, 32 insertions(+), 16 deletions(-)

diff --git a/src/components/metadata-editor.vue b/src/components/metadata-editor.vue
index 59eebfff2..e7e0384d2 100644
--- a/src/components/metadata-editor.vue
+++ b/src/components/metadata-editor.vue
@@ -135,13 +135,7 @@
                                                 checkUuid();
                                             "
                                             v-model.trim="uuid"
-                                            v-on:keyup.enter="
-                                                () => {
-                                                    generateRemoteConfig()
-                                                        .then(fetchHistory)
-                                                        .catch(() => {}); // Prevent grousing in console
-                                                }
-                                            "
+                                            v-on:keyup.enter="handleUuidEnter"
                                             @focus="showDropdown = true"
                                             @blur="showDropdown = false"
                                             @keydown.down.prevent="highlightNext"
@@ -194,11 +188,7 @@
                                     <!-- Load storyline with the given UUID, if it exists on the server, and also
                                          any history associated with the product that can be found -->
                                     <button
-                                        @click="
-                                            () => {
-                                                generateRemoteConfig().then(fetchHistory);
-                                            }
-                                        "
+                                        @click="handleUuidLoad"
                                         class="editor-button editor-forms-button bg-black text-white ml-3 mr-0"
                                         :class="{ 'input-error': error }"
                                         :disabled="loadStatus === 'loading'"
@@ -257,7 +247,7 @@
                                     </span>
                                     <div>
                                         <span class="inline-block select-none text-sm">{{
-                                            $t(`editor.warning.${warning}`)
+                                            $t(`editor.warning.${warning}`, warning === 'uuidNotFound' ? { uuid } : {})
                                         }}</span>
                                     </div>
                                     <br />
@@ -594,7 +584,7 @@ import {
 import { VueSpinnerOval } from 'vue3-spinners';
 import { VueFinalModal } from 'vue-final-modal';
 import { useUserStore } from '../stores/userStore';
-import { computed } from "vue";
+import { computed } from 'vue';
 
 import JSZip from 'jszip';
 import axios from 'axios';
@@ -659,7 +649,7 @@ export default class MetadataEditorV extends Vue {
     loadingIntoEditor = false;
     loadEditor = false;
     error = false; // whether an error has occurred
-    warning: 'none' | 'uuid' | 'rename' | 'blank' = 'none'; // used for duplicate uuid warning
+    warning: 'none' | 'uuid' | 'rename' | 'blank' | 'uuidNotFound' = 'none'; // used for duplicate uuid warning
     configLang = 'en';
     currLang = 'en'; // page language
     showDropdown = false;
@@ -763,7 +753,7 @@ export default class MetadataEditorV extends Vue {
         // Initialize Storylines config and the configuration structure.
         this.configs = { en: undefined, fr: undefined };
         this.configFileStructure = undefined;
-        
+
         // set any metadata default values for creating new product
         if (!this.loadExisting) {
             // set current date as default
@@ -1092,6 +1082,7 @@ export default class MetadataEditorV extends Vue {
                             Message.error(this.$t('editor.editMetadata.message.error.noRequestedVersion'));
                         }
                         this.error = true;
+                        this.warning = 'uuidNotFound';
                         this.loadStatus = 'waiting';
                         this.clearConfig();
                         // Product was not found, unlock the UUID
@@ -1105,6 +1096,8 @@ export default class MetadataEditorV extends Vue {
                                 this.configFileStructureHelper(configZip);
                                 // Extend the session on load
                                 this.extendSession();
+                                this.error = false;
+                                this.warning = 'none';
                             });
                         });
                     }
@@ -2159,6 +2152,29 @@ export default class MetadataEditorV extends Vue {
         }
     }
 
+    async handleUuidEnter(): void {
+        if (this.editExisting) {
+            this.handleUuidLoad();
+        } else {
+            // UUID is not taken and is not blank so we proceed to editor-main
+            if (!this.error) {
+                this.continueToEditor();
+            } else if (this.warning !== 'none') {
+                Message.error(this.$t(`editor.warning.${this.warning}`));
+            }
+        }
+    }
+
+    handleUuidLoad(): void {
+        if (this.uuid) {
+            this.generateRemoteConfig().then(this.fetchHistory);
+        } else {
+            this.error = true;
+            this.warning = 'blank';
+            Message.error(this.$t(`editor.warning.${this.warning}`));
+        }
+    }
+
     highlightNext(): void {
         if (this.highlightedIndex < this.getStorylines.length - 1) {
             this.highlightedIndex++;