From 8c0df6662de9c2b48ab85db62470552637fde562 Mon Sep 17 00:00:00 2001
From: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Date: Tue, 12 Sep 2023 14:53:33 +0000
Subject: [PATCH] Bug 34587: Add Create from registry button

Currently there is no option to go back to using the registry API to create a provider once "Create manually" has been clicked

Signed-off-by: Jessica Zairo <jzairo@bywatersolutions.com>
Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 .../UsageStatisticsDataProvidersFormAdd.vue   | 61 +++++++++++++++----
 1 file changed, 50 insertions(+), 11 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersFormAdd.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersFormAdd.vue
index 3bd8db0344..8dd5de4523 100644
--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersFormAdd.vue
+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/UsageStatisticsDataProvidersFormAdd.vue
@@ -71,6 +71,14 @@
                             >
                                 {{ $__("Create manually") }}
                             </button>
+                            <button
+                                v-else
+                                type="button"
+                                style="margin-left: 1em"
+                                @click="createFromRegistry()"
+                            >
+                                {{ $__("Create from registry") }}
+                            </button>
                         </li>
                         <li>
                             <label for="usage_data_provider_description"
@@ -183,7 +191,11 @@
                     <ol class="credentials_form">
                         <li>
                             <label
-                                class="required"
+                                :class="
+                                    required_fields.includes('URL')
+                                        ? 'required'
+                                        : ''
+                                "
                                 for="usage_data_provider_service_url"
                                 >{{ $__("Service URL") }}:
                             </label>
@@ -191,14 +203,22 @@
                                 id="usage_data_provider_service_url"
                                 style="min-width: 60%"
                                 v-model="usage_data_provider.service_url"
-                                required
+                                :required="required_fields.includes('URL')"
                                 :disabled="!selected_provider && !manual_form"
                             />
-                            <span class="required">{{ $__("Required") }}</span>
+                            <span
+                                class="required"
+                                v-if="required_fields.includes('URL')"
+                                >{{ $__("Required") }}
+                            </span>
                         </li>
                         <li>
                             <label
-                                class="required"
+                                :class="
+                                    required_fields.includes('Release')
+                                        ? 'required'
+                                        : ''
+                                "
                                 for="usage_data_provider_report_release"
                                 >{{ $__("Report release") }}:
                             </label>
@@ -206,14 +226,22 @@
                                 id="usage_data_provider_report_release"
                                 style="min-width: 60%"
                                 v-model="usage_data_provider.report_release"
-                                required
+                                :required="required_fields.includes('Release')"
                                 :disabled="!selected_provider && !manual_form"
                             />
-                            <span class="required">{{ $__("Required") }}</span>
+                            <span
+                                class="required"
+                                v-if="required_fields.includes('Release')"
+                                >{{ $__("Required") }}
+                            </span>
                         </li>
                         <li>
                             <label
-                                class="required"
+                                :class="
+                                    required_fields.includes('Customer')
+                                        ? 'required'
+                                        : ''
+                                "
                                 for="usage_data_provider_customer_id"
                                 >{{ $__("Customer Id") }}:
                             </label>
@@ -221,10 +249,14 @@
                                 id="usage_data_provider_customer_id"
                                 style="min-width: 60%"
                                 v-model="usage_data_provider.customer_id"
-                                required
+                                :required="required_fields.includes('Customer')"
                                 :disabled="!selected_provider && !manual_form"
                             />
-                            <span class="required">{{ $__("Required") }}</span>
+                            <span
+                                class="required"
+                                v-if="required_fields.includes('Customer')"
+                                >{{ $__("Required") }}
+                            </span>
                         </li>
                         <li>
                             <label
@@ -469,7 +501,7 @@ export default {
                     const { url, api_key_required, requestor_id_required } =
                         sushi_service
                     this.usage_data_provider.service_url = url
-                    this.required_fields = []
+                    this.required_fields = ["URL", "Release", "Customer"]
                     api_key_required && this.required_fields.push("API")
                     requestor_id_required &&
                         this.required_fields.push("Requestor")
@@ -507,7 +539,14 @@ export default {
             this.manual_form = true
             this.selected_provider = null
             this.sushi_service = null
-            this.required_fields = ["API", "Requestor"]
+            this.required_fields = []
+        },
+        createFromRegistry() {
+            this.manual_form = false
+            this.selected_provider = null
+            this.sushi_service = null
+            this.required_fields = []
+            this.usage_data_provider.report_types = []
         },
         selectAll() {
             this.usage_data_provider.report_types = this.valid_report_types.map(
-- 
2.39.5