From d4678ddffa11d571ffce6ba0ffba99220ccd5573 Mon Sep 17 00:00:00 2001
From: Tomas Cohen Arazi <tomascohen@theke.io>
Date: Mon, 4 Jan 2021 09:59:49 -0300
Subject: [PATCH] Bug 23857: Make adding club holds set the Location header

This patch makes the club holds route add the Location header to the
response as specified in the API coding guidelines.

To test:
1. Apply the regression tests patch
2. Run:
   $ kshell
  k$ prove t/db_dependent/api/v1/clubs_holds.t
=> FAIL: Tests don't pass!
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 | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Koha/REST/V1/Clubs/Holds.pm b/Koha/REST/V1/Clubs/Holds.pm
index 78f741acd0..4fad9c3938 100644
--- a/Koha/REST/V1/Clubs/Holds.pm
+++ b/Koha/REST/V1/Clubs/Holds.pm
@@ -121,6 +121,8 @@ sub add {
             default_patron_home => $default_patron_home
         });
 
+        $c->res->headers->location( $c->req->url->to_string . '/' );
+
         return $c->render(
             status  => 201,
             openapi => $club_hold->to_api
-- 
2.30.0