Attachment 133530 Details for Bug 30534 – Bug 30534: Remove guarantor_id attribute from the patron object

[patch] Bug 30534: Remove guarantor_id attribute from the patron object

Bug-30534-Remove-guarantorid-attribute-from-the-pa.patch (text/plain), 1.80 KB, created by Jonathan Druart on 2022-04-21 09:27:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2022-04-21 09:27:36 UTC

Size: 1.80 KB

patch

obsolete

>From 4208c49112ca0785e132fa63b459366c34b07f80 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Wed, 13 Apr 2022 17:07:55 -0300
>Subject: [PATCH] Bug 30534: Remove guarantor_id attribute from the patron
> object
>
>This patch removes an attribute that was actually removed 3 years ago
>and causes an exception when trying to search for it. The API responses
>don't include it anyways.
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>
>Signed-off-by: David Nind <david@davidnind.com>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> Koha/Patron.pm                    | 1 -
> api/v1/swagger/paths/patrons.yaml | 5 -----
> 2 files changed, 6 deletions(-)
>
>diff --git a/Koha/Patron.pm b/Koha/Patron.pm
>index 3a26db0d6a0..c0cbf99ec9d 100644
>--- a/Koha/Patron.pm
>+++ b/Koha/Patron.pm
>@@ -1896,7 +1896,6 @@ sub to_api_mapping {
>         emailpro            => 'secondary_email',
>         flags               => undef,    # permissions manipulation handled in /permissions
>         gonenoaddress       => 'incorrect_address',
>-        guarantorid         => 'guarantor_id',
>         lastseen            => 'last_seen',
>         lost                => 'patron_card_lost',
>         opacnote            => 'opac_notes',
>diff --git a/api/v1/swagger/paths/patrons.yaml b/api/v1/swagger/paths/patrons.yaml
>index 5164cb2858d..227bc40419b 100644
>--- a/api/v1/swagger/paths/patrons.yaml
>+++ b/api/v1/swagger/paths/patrons.yaml
>@@ -204,11 +204,6 @@
>         description: Filter search by restricted
>         required: false
>         type: boolean
>-      - name: guarantor_id
>-        in: query
>-        description: Search on guarantor_id
>-        required: false
>-        type: string
>       - name: staff_notes
>         in: query
>         description: Case insensitive search on staff_notes
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 30534: 133282 | 133398 | 133530