Attachment 116217 Details for Bug 27593 – Bug 27593: Consistent behavior for club holds route

[patch] Bug 27593: Consistent behavior for club holds route

Bug-27593-Consistent-behavior-for-club-holds-route.patch (text/plain), 1.54 KB, created by Tomás Cohen Arazi (tcohen) on 2021-02-02 14:27:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2021-02-02 14:27:04 UTC

Size: 1.54 KB

patch

obsolete

>From 8f355324f1e29d2c1acfd154aa18336c127975d7 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Tue, 2 Feb 2021 11:22:50 -0300
>Subject: [PATCH] Bug 27593: Consistent behavior for club holds route
>
>This patch:
>- Fixes a malformed response when biblio_id is invalid
>- Fixes the return status (400 => 404) when biblio_id is invalid
>- Adapts the error strings to be consistent with the Cities.pm file
>
>To test:
>1. Apply the regression tests patch
>2. Run:
>   $ kshell
>  k$ prove t/db_dependent/api/v1/clubs_holds.t
>=> FAIL: Tests fail
>3. Apply this patch
>4. Repeat 2
>=> SUCCESS: Tests pass!
>5. Sign off :-D
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>---
> Koha/REST/V1/Clubs/Holds.pm | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/Koha/REST/V1/Clubs/Holds.pm b/Koha/REST/V1/Clubs/Holds.pm
>index 21c95c1d151..726f6f20deb 100644
>--- a/Koha/REST/V1/Clubs/Holds.pm
>+++ b/Koha/REST/V1/Clubs/Holds.pm
>@@ -81,7 +81,7 @@ sub add {
>             unless ($item) {
>                 return $c->render(
>                     status  => 404,
>-                    openapi => { error => "item_id not found." }
>+                    openapi => { error => "Item not found" }
>                 );
>             }
>             else {
>@@ -100,8 +100,8 @@ sub add {
> 
>         unless ($biblio) {
>             return $c->render(
>-                status  => 400,
>-                openapi => "Biblio not found."
>+                status  => 404,
>+                openapi => { error => "Biblio not found" }
>             );
>         }
> 
>-- 
>2.30.0

Actions: View | Diff | Splinter Review

Attachments on bug 27593: 116216 | 116217 | 117007 | 117008 | 117155 | 117156