Attachment 138803 Details for Bug 25375 – Bug 25375: Match current Zebra availability limit

[patch] Bug 25375: Match current Zebra availability limit

Bug-25375-Match-current-Zebra-availability-limit.patch (text/plain), 1.79 KB, created by Nick Clemens (kidclamp) on 2022-08-08 13:19:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2022-08-08 13:19:52 UTC

Size: 1.79 KB

patch

obsolete

>From 6b3d158b8902235d4f386669fe6d56d60c96e4e0 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Mon, 8 Aug 2022 12:15:16 +0000
>Subject: [PATCH] Bug 25375: Match current Zebra availability limit
>
>In Zebra the availability limit code states:
>'available' is defined as (items.onloan is NULL) and (items.itemlost = 0)
>
>We should make ES follow the same behaviour so that changing the engine does not
>change this.
>
>To test:
>1 - Find or create several records with two items each
>2 - For those records setup items as:
>    a - one checked out, one available
>    b - both marked lost
>    c - both damaged
>    d - both withdrawn
>    e - both notforloan
>3 - Enable ES and reindex to confirm everything is set
>4 - Perform an advanced search to return those records and 'limit to available items'
>5 - Confirm you see records b,c,d,e and records say '2 items, none available'
>6 - Apply patch
>7 - Run `misc/search_tools/rebuild_elasticsearch.pl -d -b`
>8 - Perform an advanced search to return those records and 'limit to available items'
>9 - Confirm you see records a,b,c,d and al but a say '2 items, none available'
>---
> Koha/SearchEngine/Elasticsearch.pm | 3 ---
> 1 file changed, 3 deletions(-)
>
>diff --git a/Koha/SearchEngine/Elasticsearch.pm b/Koha/SearchEngine/Elasticsearch.pm
>index b2273c5ace..28dd05c2f3 100644
>--- a/Koha/SearchEngine/Elasticsearch.pm
>+++ b/Koha/SearchEngine/Elasticsearch.pm
>@@ -782,9 +782,6 @@ sub marc_records_to_documents {
>                     biblionumber => $biblionumber,
>                     onloan       => undef,
>                     notforloan   => 0,
>-                    withdrawn    => 0,
>-                    itemlost     => 0,
>-                    damaged      => 0
>                 })->count;
> 
>                 $record_document->{available} = $avail_items ? \1 : \0;
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 25375: 104356 | 109647 | 109659 | 109660 | 109951 | 109952 | 109953 | 109954 | 131310 | 131311 | 131312 | 131313 | 132327 | 132328 | 132329 | 132330 | 138800 | 138802 | 138803 | 140053 | 140054 | 140055 | 140056 | 140057 | 140572 | 140899 | 140900 | 140979