From 4c00c6d3a53f8506c2255b021ccfb2ff6edf7cb3 Mon Sep 17 00:00:00 2001 From: f-osorio Date: Thu, 12 Dec 2024 15:50:38 -0500 Subject: [PATCH 1/4] Fix issue when trying to reopen an authority --- src/components/panels/edit/fields/LookupComplex.vue | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/components/panels/edit/fields/LookupComplex.vue b/src/components/panels/edit/fields/LookupComplex.vue index 8c82f9be..2319aecf 100644 --- a/src/components/panels/edit/fields/LookupComplex.vue +++ b/src/components/panels/edit/fields/LookupComplex.vue @@ -764,9 +764,13 @@ export default { this.marcDeliminatedLCSHModeTimeout = null this.marcDeliminatedLCSHModeResults = [] - let selection = document.getElementById(this.guid) - let selected = selection.options[selection.selectedIndex].value - this.searchType = selected + try { + let selection = document.getElementById(this.guid) + let selected = selection.options[selection.selectedIndex].value + this.searchType = selected + } catch{ + + } this.displaySubjectModal = true } From ed7e39b37fd870ed42ba9bb2c4fb98477f69f0d3 Mon Sep 17 00:00:00 2001 From: f-osorio Date: Thu, 12 Dec 2024 16:03:11 -0500 Subject: [PATCH 2/4] Fix error when trying to open authority with copy/paste mode on copy paste IDs match the ids of the subject selection drop down, need a way to differntiate them --- .../panels/edit/fields/LookupComplex.vue | 21 +++++++++++-------- 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/src/components/panels/edit/fields/LookupComplex.vue b/src/components/panels/edit/fields/LookupComplex.vue index 2319aecf..69e954e9 100644 --- a/src/components/panels/edit/fields/LookupComplex.vue +++ b/src/components/panels/edit/fields/LookupComplex.vue @@ -737,9 +737,12 @@ export default { // Open the authority `panel` for an given authority openAuthority: function() { //Get the type of search - let selection = document.getElementById(this.guid) - let selected = selection.options[selection.selectedIndex].value - this.searchType = selected + console.info("guid", this.guid) + try { + let selection = document.getElementById(this.guid) + let selected = selection.options[selection.selectedIndex].value + this.searchType = selected + } catch {} let label = this.$refs.el[0].innerHTML this.profileData = this.profileStore.returnStructureByGUID(this.guid) @@ -764,13 +767,13 @@ export default { this.marcDeliminatedLCSHModeTimeout = null this.marcDeliminatedLCSHModeResults = [] - try { - let selection = document.getElementById(this.guid) - let selected = selection.options[selection.selectedIndex].value - this.searchType = selected - } catch{ + // try { + // let selection = document.getElementById(this.guid) + // let selected = selection.options[selection.selectedIndex].value + // this.searchType = selected + // } catch{ - } + // } this.displaySubjectModal = true } From 596238c98af0d05fcd3053675bf02a9408e2c193 Mon Sep 17 00:00:00 2001 From: f-osorio Date: Fri, 13 Dec 2024 07:11:01 -0500 Subject: [PATCH 3/4] Update `id` for dropdowns The `id` now has "-select" appended to it so there is no interference with the copy/paste ids. --- src/components/panels/edit/fields/LookupComplex.vue | 13 +++++-------- src/components/panels/edit/fields/Ref.vue | 2 +- 2 files changed, 6 insertions(+), 9 deletions(-) diff --git a/src/components/panels/edit/fields/LookupComplex.vue b/src/components/panels/edit/fields/LookupComplex.vue index 69e954e9..d610af3d 100644 --- a/src/components/panels/edit/fields/LookupComplex.vue +++ b/src/components/panels/edit/fields/LookupComplex.vue @@ -607,7 +607,7 @@ export default { this.marcDeliminatedLCSHMode = true //Get the type of search - let selection = document.getElementById(this.guid) + let selection = document.getElementById(this.guid+"-select") let selected = selection.options[selection.selectedIndex].value this.searchType = selected @@ -671,7 +671,7 @@ export default { this.authorityLookup = this.searchValue.trim() this.searchValue = this.searchValue.trim() - let selection = document.getElementById(this.guid) + let selection = document.getElementById(this.guid+"-select") let selected = selection.options[selection.selectedIndex].value this.searchType = selected this.displaySubjectModal=true @@ -737,12 +737,9 @@ export default { // Open the authority `panel` for an given authority openAuthority: function() { //Get the type of search - console.info("guid", this.guid) - try { - let selection = document.getElementById(this.guid) - let selected = selection.options[selection.selectedIndex].value - this.searchType = selected - } catch {} + let selection = document.getElementById(this.guid+"-select") + let selected = selection.options[selection.selectedIndex].value + this.searchType = selected let label = this.$refs.el[0].innerHTML this.profileData = this.profileStore.returnStructureByGUID(this.guid) diff --git a/src/components/panels/edit/fields/Ref.vue b/src/components/panels/edit/fields/Ref.vue index 98697ef6..bc464ad3 100644 --- a/src/components/panels/edit/fields/Ref.vue +++ b/src/components/panels/edit/fields/Ref.vue @@ -12,7 +12,7 @@ From 74cbe4f24d7f488feb16cdf2b12fa35fbbe00552 Mon Sep 17 00:00:00 2001 From: f-osorio Date: Fri, 13 Dec 2024 07:26:51 -0500 Subject: [PATCH 4/4] version++ --- src/stores/config.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/stores/config.js b/src/stores/config.js index 7e31f9fb..410406f8 100644 --- a/src/stores/config.js +++ b/src/stores/config.js @@ -7,7 +7,7 @@ export const useConfigStore = defineStore('config', { versionMajor: 0, versionMinor: 17, - versionPatch: 4, + versionPatch: 5, regionUrls: {