Attachment 38778 Details for Bug 13516 – [PASSED QA] Bug 13516 - Display issue for search results with items both checkout out and lost

[patch] [PASSED QA] Bug 13516 - Display issue for search results with items both checkout out and lost

PASSED-QA-Bug-13516---Display-issue-for-search-res.patch (text/plain), 1.57 KB, created by Katrin Fischer on 2015-05-03 16:25:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2015-05-03 16:25:36 UTC

Size: 1.57 KB

patch

obsolete

>From 08be417d518923b68f3f9663534aefc8cd4781c2 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Mon, 5 Jan 2015 10:10:57 -0500
>Subject: [PATCH] [PASSED QA] Bug 13516 - Display issue for search results with
> items both checkout out and lost
>
>If the staff search results, if an item is both checked out and lost,
>the result will appear as two item lines where one line has the lost
>status and the other line has the rest of the item's data.
>
>Test Plan:
>1) Check an item out to a patron
>2) Mark the item as lost *without* removing the item from the patron's
>   record, either by using longoverdue.pl or by editing the itemlost
>   field in the database directly.
>3) Perform a search where that item will be in the results
>4) Note the improper display of the item's data
>5) Apply this patch set
>6) Reload the search restults
>7) Note the item now displays correctly
>
>Signed-off-by: Nick <nick@quecheelibrary.org>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> C4/Search.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/C4/Search.pm b/C4/Search.pm
>index a98cb04..990ed39 100644
>--- a/C4/Search.pm
>+++ b/C4/Search.pm
>@@ -2115,7 +2115,7 @@ sub searchResults {
> 
>                 # if something's checked out and lost, mark it as 'long overdue'
>                 if ( $item->{itemlost} ) {
>-                    $onloan_items->{$prefix}->{longoverdue}++;
>+                    $onloan_items->{$key}->{longoverdue}++;
>                     $longoverdue_count++;
>                 }
>                 else {    # can place holds as long as item isn't lost
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 13516: 34954 | 34955 | 34956 | 34957 | 35097 | 35098 | 38776 | 38777 | 38778