Bug 37856: Add service_platform field

The change from this patch itself should be self-explanatory.
Testing for this requires credentials to a specific provider.
But the parameter is only added to the URL if defined.

Test plan, ktd, apply patch, run:
$ yarn js:build
$ yarn api:bundle
$ koha-plack --restart kohadev

1) Enable ERMModule
2) Create a new provider, visit:
<staff_url>/cgi-bin/koha/erm/eusage/usage_data_providers/add
3) Ensure 'Service platform' now shows. Create the provider and put something in this field.
4) View the provider, ensure the field is shown, visit:
<staff_url>/cgi-bin/koha/erm/eusage/usage_data_providers/1

Ensure all tests are still passing:
$ cypress run --spec t/cypress/integration/ERM/DataProviders_spec.ts
$ prove t/db_dependent/api/v1/erm_usage_*

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
Pedro Amorim 2024-09-18 15:56:19 +00:00 committed by Katrin Fischer
parent 2e4b1ca42f
commit b852c042fe
Signed by: kfischer
GPG key ID: 0EF6E2C03357A834
4 changed files with 21 additions and 0 deletions

View file

@ -73,6 +73,11 @@ properties:
type:
- string
- "null"
service_platform:
description: service platform of the harvester
type:
- string
- "null"
metric_type:
description: metric type of the harvester when reporting
type:

View file

@ -3215,6 +3215,7 @@ CREATE TABLE `erm_usage_data_providers` (
`requestor_name` varchar(80) DEFAULT NULL COMMENT 'requestor name',
`requestor_email` varchar(80) DEFAULT NULL COMMENT 'requestor email',
`report_types` varchar(255) DEFAULT NULL COMMENT 'report types provided by the harvester',
`service_platform` varchar(80) DEFAULT NULL COMMENT 'platform if provider requires it',
PRIMARY KEY (`erm_usage_data_provider_id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;
/*!40101 SET character_set_client = @saved_cs_client */;

View file

@ -32,6 +32,12 @@
{{ usage_data_provider.report_types }}
</span>
</li>
<li>
<label>{{ $__("Service platform") }}:</label>
<span id="service_platform">
{{ usage_data_provider.service_platform }}
</span>
</li>
</ol>
</fieldset>
<legend>{{ $__("SUSHI credentials") }}</legend>

View file

@ -118,6 +118,15 @@
:disabled="!selected_provider && !manual_form"
/>
</li>
<li>
<label for="usage_data_provider_service_platform"
>{{ $__("Service platform") }}:
</label>
<input
id="usage_data_provider_service_platform"
v-model="usage_data_provider.service_platform"
/>
</li>
<li>
<label
class="required"