Bugzilla – Attachment 38777 Details for
Bug 13516
Display issue for search results with items both checkout out and lost
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 13516 - Tidy relevant code block
PASSED-QA-Bug-13516---Tidy-relevant-code-block.patch (text/plain), 2.92 KB, created by
Katrin Fischer
on 2015-05-03 16:25:17 UTC
(
hide
)
Description:
[PASSED QA] Bug 13516 - Tidy relevant code block
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2015-05-03 16:25:17 UTC
Size:
2.92 KB
patch
obsolete
>From 543b397d85a9e63ad90961a8f201d34cec34392b Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Mon, 5 Jan 2015 10:05:48 -0500 >Subject: [PATCH] [PASSED QA] Bug 13516 - Tidy relevant code block > >Signed-off-by: Nick <nick@quecheelibrary.org> > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > C4/Search.pm | 25 +++++++++++++++---------- > 1 file changed, 15 insertions(+), 10 deletions(-) > >diff --git a/C4/Search.pm b/C4/Search.pm >index 686f386..a98cb04 100644 >--- a/C4/Search.pm >+++ b/C4/Search.pm >@@ -2099,21 +2099,26 @@ sub searchResults { > my $prefix = $item->{$hbranch} . '--' . $item->{location} . $item->{itype} . $item->{itemcallnumber}; > # For each grouping of items (onloan, available, unavailable), we build a key to store relevant info about that item > my $userenv = C4::Context->userenv; >- if ( $item->{onloan} && !(C4::Members::GetHideLostItemsPreference($userenv->{'number'}) && $item->{itemlost}) ) { >+ if ( $item->{onloan} >+ && !( C4::Members::GetHideLostItemsPreference( $userenv->{'number'} ) && $item->{itemlost} ) ) >+ { > $onloan_count++; >- my $key = $prefix . $item->{onloan} . $item->{barcode}; >- $onloan_items->{$key}->{due_date} = format_date($item->{onloan}); >- $onloan_items->{$key}->{count}++ if $item->{$hbranch}; >- $onloan_items->{$key}->{branchname} = $item->{branchname}; >- $onloan_items->{$key}->{location} = $shelflocations->{ $item->{location} }; >- $onloan_items->{$key}->{itemcallnumber} = $item->{itemcallnumber}; >- $onloan_items->{$key}->{description} = $item->{description}; >- $onloan_items->{$key}->{imageurl} = getitemtypeimagelocation( $search_context, $itemtypes{ $item->{itype} }->{imageurl} ); >+ my $key = $prefix . $item->{onloan} . $item->{barcode}; >+ $onloan_items->{$key}->{due_date} = format_date( $item->{onloan} ); >+ $onloan_items->{$key}->{count}++ if $item->{$hbranch}; >+ $onloan_items->{$key}->{branchname} = $item->{branchname}; >+ $onloan_items->{$key}->{location} = $shelflocations->{ $item->{location} }; >+ $onloan_items->{$key}->{itemcallnumber} = $item->{itemcallnumber}; >+ $onloan_items->{$key}->{description} = $item->{description}; >+ $onloan_items->{$key}->{imageurl} = >+ getitemtypeimagelocation( $search_context, $itemtypes{ $item->{itype} }->{imageurl} ); >+ > # if something's checked out and lost, mark it as 'long overdue' > if ( $item->{itemlost} ) { > $onloan_items->{$prefix}->{longoverdue}++; > $longoverdue_count++; >- } else { # can place holds as long as item isn't lost >+ } >+ else { # can place holds as long as item isn't lost > $can_place_holds = 1; > } > } >-- >1.9.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 13516
:
34954
|
34955
|
34956
|
34957
|
35097
|
35098
|
38776
| 38777 |
38778