From e13572b9cb64484bb95fef586dc12c4ee93577a3 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi 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 Signed-off-by: David Nind Signed-off-by: Martin Renvoize Signed-off-by: Jonathan Druart (cherry picked from commit 2387ac731c03ecfa6cd20e7e71037bbd3c3ac046) Signed-off-by: Fridolin Somers (cherry picked from commit e1f35cf3474952a851e62a2366fd520f0b413228) Signed-off-by: Andrew Fuerste-Henry --- 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 21c95c1d15..726f6f20de 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.39.5