From 6d500ce443c447a3f52e1398c6f08ab99edc44d5 Mon Sep 17 00:00:00 2001 From: Andrew Auld Date: Thu, 3 Apr 2025 10:34:47 +0000 Subject: [PATCH] Bug 39543: Error modal when trying to add two controlling licences to an agreement duplicates error message If you try to add two licences with the status 'controlling' to an agreement record, the error modal duplicates the error message 'Only one controlling license is allowed'. Test Plan: 1) Enable ERMModule sys pref 2) Add an agreement record 3) Add two licence records 4) Edit the agreement record and associate both license records to the agreement 5) Try to set both licenses to status 'controlling' 6) Note error modal duplicates the error message 'Only one controlling license is allowed' 7) Apply the patch, run yarn js:build and then hard refresh the browser 8) Repeat steps 1 to 5 and enjoy the single error message! Signed-off-by: Magnus Enger I took the liberty of fixing up the commit message a bit. --- .../prog/js/vue/components/ERM/AgreementsFormAdd.vue | 9 --------- 1 file changed, 9 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsFormAdd.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsFormAdd.vue index 643bb0acc4..3b4412430a 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsFormAdd.vue +++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementsFormAdd.vue @@ -292,15 +292,6 @@ export default { ); } - if ( - agreement_licenses.filter(al => al.status == "controlling") - .length > 1 - ) { - errors.push( - this.$__("Only one controlling license is allowed") - ); - } - let documents_with_uploaded_files = agreement.documents.filter( doc => typeof doc.file_content !== "undefined" ); -- 2.34.1