From 14506040292201a78733a9c1746bf7f590205fe3 Mon Sep 17 00:00:00 2001
From: Tomas Cohen Arazi <tomascohen@theke.io>
Date: Tue, 10 Nov 2020 17:38:26 -0300
Subject: [PATCH] Bug 26988: (QA follow-up) Make the API return proper library
 objects

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 Koha/REST/V1/Holds.pm           | 16 +++++++++++-----
 api/v1/swagger/paths/holds.json |  8 +++++++-
 2 files changed, 18 insertions(+), 6 deletions(-)

diff --git a/Koha/REST/V1/Holds.pm b/Koha/REST/V1/Holds.pm
index 8239a62186..20f76a13f6 100644
--- a/Koha/REST/V1/Holds.pm
+++ b/Koha/REST/V1/Holds.pm
@@ -406,7 +406,7 @@ sub pickup_locations {
     my $c = shift->openapi->valid_input or return;
 
     my $hold_id = $c->validation->param('hold_id');
-    my $hold = Koha::Holds->find($hold_id);
+    my $hold = Koha::Holds->find( $hold_id, { prefetch => [ 'patron' ] } );
 
     unless ($hold) {
         return $c->render(
@@ -416,11 +416,17 @@ sub pickup_locations {
     }
 
     return try {
-        my $pickup_locations = $hold->itemnumber ?
-        $hold->item->pickup_locations({ patron => $hold->patron }) : $hold->biblio->pickup_locations({ patron => $hold->patron });
-        warn Data::Dumper::Dumper( $pickup_locations );
+        my @pickup_locations =
+            $hold->itemnumber
+          ? @{ $hold->item->pickup_locations( { patron => $hold->patron } ) }
+          : @{ $hold->biblio->pickup_locations( { patron => $hold->patron } ) };
+
+        @pickup_locations = map { $_->to_api } @pickup_locations;
 
-        return $c->render( status => 200, openapi => $pickup_locations );
+        return $c->render(
+            status  => 200,
+            openapi => \@pickup_locations
+        );
     }
     catch {
         $c->unhandled_exception($_);
diff --git a/api/v1/swagger/paths/holds.json b/api/v1/swagger/paths/holds.json
index b375ea72dd..3792985c8c 100644
--- a/api/v1/swagger/paths/holds.json
+++ b/api/v1/swagger/paths/holds.json
@@ -625,7 +625,13 @@
       "produces": ["application/json"],
       "responses": {
         "200": {
-          "description": "Hold pickup location"
+          "description": "Hold pickup location",
+          "schema": {
+            "type": "array",
+            "items": {
+              "$ref": "../definitions.json#/library"
+            }
+          }
         },
         "400": {
           "description": "Missing or wrong parameters",
-- 
2.39.5