Attachment 114799 Details for Bug 27330 – Bug 27330: Return 403 if club has no enrollments

[patch] Bug 27330: Return 403 if club has no enrollments

Bug-27330-Return-403-if-club-has-no-enrollments.patch (text/plain), 1.71 KB, created by Tomás Cohen Arazi (tcohen) on 2021-01-04 14:37:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2021-01-04 14:37:14 UTC

Size: 1.71 KB

patch

obsolete

>From d8a0cb3dfa8cf7e757c7bb534d761019866f33d8 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Mon, 4 Jan 2021 11:24:51 -0300
>Subject: [PATCH] Bug 27330: Return 403 if club has no enrollments
>
>This patch makes the controller script return a 403 as it should [1]
>instead of a plain 500.
>
>To test:
>1. Apply the regressions 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
>
>[1] https://www.restapitutorial.com/httpstatuscodes.html
>---
> Koha/REST/V1/Clubs/Holds.pm | 14 ++------------
> 1 file changed, 2 insertions(+), 12 deletions(-)
>
>diff --git a/Koha/REST/V1/Clubs/Holds.pm b/Koha/REST/V1/Clubs/Holds.pm
>index 4fad9c3938..1e4b94e706 100644
>--- a/Koha/REST/V1/Clubs/Holds.pm
>+++ b/Koha/REST/V1/Clubs/Holds.pm
>@@ -130,19 +130,9 @@ sub add {
>     }
>     catch {
>         if ( blessed $_ ) {
>-            if ( $_->isa('Koha::Exceptions::Object::FKConstraint') ) {
>-                my $broken_fk = $_->broken_fk;
>-
>-                if ( grep { $_ eq $broken_fk } keys %{$Koha::REST::V1::Clubs::Holds::to_api_mapping} ) {
>-                    $c->render(
>-                        status  => 404,
>-                        openapi => $Koha::REST::V1::Clubs::Holds::to_api_mapping->{$broken_fk} . ' not found.'
>-                    );
>-                }
>-            }
>-            elsif ($_->isa('Koha::Exceptions::ClubHold')) {
>+            if ($_->isa('Koha::Exceptions::ClubHold::NoPatrons')) {
>                 return $c->render(
>-                    status  => 500,
>+                    status  => 403,
>                     openapi => { error => $_->description }
>                 );
>             }
>-- 
>2.30.0

Actions: View | Diff | Splinter Review

Attachments on bug 27330: 114798 | 114799 | 114800 | 115863 | 115864 | 115865 | 115866 | 115867 | 115968 | 115969 | 115970 | 115971 | 115972 | 116109 | 116110 | 116111 | 116112 | 116113