From e0421a2c5034d1e89850f592338323a5d5310daf Mon Sep 17 00:00:00 2001 From: Andrew Isherwood Date: Wed, 14 Nov 2018 16:08:32 +0000 Subject: [PATCH] Bug 21835: Fix display of request ID We now populate the id_prefix property in the API response. It turns out that explicitly casting to a string wasn't necessary as JS does "the right thing" when concatenating a string and an integer Test plan: - Before applying the patch view the ILL requests table - TEST: Observe that the Request number column displays as NaN - Apply the patch - Refresh the page - TEST: Observe that the request number now displays correctly - Add the following block to your koha-conf.xml: **The code of a branch that has an ILL request attached to it** YAY - Refresh the page - TEST: Observe the request made on the branch that you specified in the config are now prefixed with "YAY-" --- Koha/Illrequest.pm | 1 - Koha/REST/V1/Illrequests.pm | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/Koha/Illrequest.pm b/Koha/Illrequest.pm index 0b4d71c1ea..0fa4e49661 100644 --- a/Koha/Illrequest.pm +++ b/Koha/Illrequest.pm @@ -1018,7 +1018,6 @@ sub TO_JSON { my ( $self, $embed ) = @_; my $object = $self->SUPER::TO_JSON(); - $object->{id_prefix} = $self->id_prefix; return $object; } diff --git a/Koha/REST/V1/Illrequests.pm b/Koha/REST/V1/Illrequests.pm index 0b4c8f0d2a..a3a0d61c1c 100644 --- a/Koha/REST/V1/Illrequests.pm +++ b/Koha/REST/V1/Illrequests.pm @@ -107,6 +107,7 @@ sub list { my @output = (); foreach my $req(@requests) { my $to_push = $req->unblessed; + $to_push->{id_prefix} = $req->id_prefix; foreach my $p(@{$patron_arr}) { if ($p->{borrowernumber} == $req->borrowernumber) { $to_push->{patron} = { -- 2.11.0