Bugzilla – Attachment 41201 Details for
Bug 14598
itemtype is not set on statistics by C4::Circulation::AddReturn
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 14598 [QA Followup] - Don't die on bad barcode
Bug-14598-QA-Followup---Dont-die-on-bad-barcode.patch (text/plain), 1.21 KB, created by
Kyle M Hall (khall)
on 2015-07-27 15:18:55 UTC
(
hide
)
Description:
Bug 14598 [QA Followup] - Don't die on bad barcode
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2015-07-27 15:18:55 UTC
Size:
1.21 KB
patch
obsolete
>From 17fba053993af44728f039dafeb950f2c29c3e23 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Mon, 27 Jul 2015 11:16:56 -0400 >Subject: [PATCH] Bug 14598 [QA Followup] - Don't die on bad barcode > >--- > C4/Circulation.pm | 10 +++++----- > 1 files changed, 5 insertions(+), 5 deletions(-) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index d7cda99..b8244fb 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -1751,14 +1751,14 @@ sub AddReturn { > my $stat_type = 'return'; > > # get information on item >- my $item = GetItem( undef, $barcode ) >- or die "GetItem( undef, $barcode ) failed"; >+ my $item = GetItem( undef, $barcode ); >+ unless ($item) { >+ return ( 0, { BadBarcode => $barcode } ); # no barcode means no item or borrower. bail out. >+ } >+ > my $itemnumber = $item->{ itemnumber }; > my $itemtype = $item->{ itype }; > >- unless ($itemnumber) { >- return (0, { BadBarcode => $barcode }); # no barcode means no item or borrower. bail out. >- } > my $issue = GetItemIssue($itemnumber); > if ($issue and $issue->{borrowernumber}) { > $borrower = C4::Members::GetMemberDetails($issue->{borrowernumber}) >-- >1.7.2.5
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 14598
:
41198
|
41199
|
41200
|
41201
|
41202
|
41203
|
41204
|
41205
|
41383
|
41384
|
41385
|
41627
|
41628
|
41629
|
41637
|
42171
|
42177
|
44665
|
44666
|
44667
|
44668
|
45030
|
45047
|
45838
|
45843
|
45844
|
46546
|
46877
|
47108
|
47109
|
47110
|
47111
|
47112
|
47113
|
47114
|
47115
|
47116
|
49797
|
49798
|
49799
|
49800
|
49801
|
49802
|
49803
|
49804
|
49805
|
49807
|
49963
|
49964
|
49965
|
49966
|
49967
|
49968
|
49969
|
49970
|
49971
|
49972
|
49978
|
49979
|
49980
|
49981
|
49982
|
49983
|
49984
|
49985
|
49986
|
49987
|
49988
|
50018
|
56941
|
56942
|
56943
|
56944
|
56945
|
56946
|
56947
|
56948
|
56949
|
56950
|
56951
|
56952
|
56953
|
62539
|
62555
|
62682