Attachment 137853 Details for Bug 31183 – Bug 31183: Use filter_by_current in Koha::Holds->get_items_that_can_fill

[patch] Bug 31183: Use filter_by_current in Koha::Holds->get_items_that_can_fill

Bug-31183-Use-filterbycurrent-in-KohaHolds-getitem.patch (text/plain), 1.29 KB, created by Tomás Cohen Arazi (tcohen) on 2022-07-18 21:48:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2022-07-18 21:48:34 UTC

Size: 1.29 KB

patch

obsolete

>From 28e099c50af6ff9b8882f964aa3e6c319ade63df Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Mon, 18 Jul 2022 18:39:51 -0300
>Subject: [PATCH] Bug 31183: Use filter_by_current in
> Koha::Holds->get_items_that_can_fill
>
>This patch makes the method use the new
>Koha::Item::Transfers->filter_by_current one.
>
>To test:
>1. Run:
>   $ kshell
>  k$ prove t/db_dependent/Koha/Holds.t
>=> SUCCESS: Tests pass
>2. Apply this patch
>3. Repeat 1
>=> SUCCESS: Tests still pass
>4. Sign off :-D
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>---
> Koha/Holds.pm | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
>diff --git a/Koha/Holds.pm b/Koha/Holds.pm
>index 7d070f8513..53a6a00715 100644
>--- a/Koha/Holds.pm
>+++ b/Koha/Holds.pm
>@@ -120,12 +120,7 @@ sub get_items_that_can_fill {
>     push @bibs_or_items, 'me.itemnumber' => { in => \@itemnumbers } if @itemnumbers;
>     push @bibs_or_items, 'me.biblionumber' => { in => \@biblionumbers } if @biblionumbers;
> 
>-    my @branchtransfers = Koha::Item::Transfers->search(
>-        {
>-            datearrived => undef,
>-            datecancelled => undef,
>-        },
>-        {
>+    my @branchtransfers = Koha::Item::Transfers->filter_by_current->search({}, {
>             columns  => ['itemnumber'],
>             collapse => 1,
>         }
>-- 
>2.37.1

Actions: View | Diff | Splinter Review

Attachments on bug 31183: 137851 | 137852 | 137853 | 137944 | 137945 | 137946 | 138887 | 138888 | 138889