Attachment 63295 Details for Bug 18276 – [SIGNED-OFF] Bug 18276: Remove GetBiblioFromItemNumber - ILSDI

[patch] [SIGNED-OFF] Bug 18276: Remove GetBiblioFromItemNumber - ILSDI

SIGNED-OFF-Bug-18276-Remove-GetBiblioFromItemNumbe.patch (text/plain), 1.72 KB, created by Josef Moravec on 2017-05-09 15:14:28 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Josef Moravec

Created: 2017-05-09 15:14:28 UTC

Size: 1.72 KB

patch

obsolete

>From a594ad6c3b3892ace9e77e76c371d70d69aa9600 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Wed, 15 Mar 2017 10:41:29 -0300
>Subject: [PATCH] [SIGNED-OFF] Bug 18276: Remove GetBiblioFromItemNumber -
> ILSDI
>
>Test plan:
>Hit /cgi-bin/koha/ilsdi.pl?service=GetPatronInfo&patron_id=51&show_holds=1
>The results should be the same before and after this patch
>
>Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
>---
> C4/ILSDI/Services.pm | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
>diff --git a/C4/ILSDI/Services.pm b/C4/ILSDI/Services.pm
>index 287ac02..0e8663c 100644
>--- a/C4/ILSDI/Services.pm
>+++ b/C4/ILSDI/Services.pm
>@@ -418,15 +418,20 @@ sub GetPatronInfo {
>         foreach my $reserve (@reserves) {
> 
>             # Get additional informations
>-            my $item = GetBiblioFromItemNumber( $reserve->{'itemnumber'}, undef );
>+            my $item = Koha::Items->find( $reserve->{itemnumber} );
>+            my $biblio = $item->biblio;
>+            my $biblioitem = $biblio->biblioitem;
>             my $library = Koha::Libraries->find( $reserve->{branchcode} );
>             my $branchname = $library ? $library->branchname : '';
> 
>             # Remove unwanted fields
>+            $item = $item->unblessed;
>             delete $item->{'more_subfields_xml'};
>+            $biblio = $biblio->unblessed;
>+            $biblioitem = $biblioitem->unblessed;
> 
>             # Add additional fields
>-            $reserve->{'item'}       = $item;
>+            $reserve->{'item'}       = { %$item, %$biblio, %$biblioitem };
>             $reserve->{'branchname'} = $branchname;
>             $reserve->{'title'}      = GetBiblio( $reserve->{'biblionumber'} )->{'title'};
>         }
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 18276: 61122 | 61123 | 61124 | 61125 | 61126 | 61177 | 61178 | 61179 | 61180 | 61181 | 62081 | 62082 | 62083 | 62084 | 62085 | 62476 | 62477 | 62478 | 62479 | 62480 | 62481 | 63121 | 63122 | 63123 | 63124 | 63125 | 63126 | 63166 | 63294 | 63295 | 63296 | 63297 | 63298 | 63299 | 63300 | 64748 | 64809 | 64810 | 64811 | 64812 | 64813 | 64814 | 64815 | 64816 | 64817 | 64878 | 64879 | 64880 | 64881 | 64882 | 64883 | 64884 | 64885 | 64886 | 64887 | 64974 | 64975