Bugzilla – Attachment 91227 Details for
Bug 23165
Renewals for Self Checkout (SCO) module do not record branchcode in statistics
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 23165: (QA follow-up) Don't set if it's already set for non-opac calls
Bug-23165-QA-follow-up-Dont-set-if-its-already-set.patch (text/plain), 835 bytes, created by
Kyle M Hall (khall)
on 2019-07-03 13:21:40 UTC
(
hide
)
Description:
Bug 23165: (QA follow-up) Don't set if it's already set for non-opac calls
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2019-07-03 13:21:40 UTC
Size:
835 bytes
patch
obsolete
>From f1009bbb6ec7132e135516437c1b598f68c63828 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Wed, 3 Jul 2019 09:21:15 -0400 >Subject: [PATCH] Bug 23165: (QA follow-up) Don't set if it's already set for > non-opac calls > >--- > C4/Circulation.pm | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index 82496fb13c..d7ed735bd1 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -2972,9 +2972,8 @@ sub AddRenewal { > $branch = 'OPACRenew'; > } > } >- else { >- $branch = >- C4::Context->userenv ? C4::Context->userenv->{branch} : $branch; >+ elsif ( !$branch && C4::Context->userenv ) { >+ $branch = C4::Context->userenv->{branch}; > } > > # Add the renewal to stats >-- >2.20.1 (Apple Git-117)
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=91227">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 23165
:
90815
|
90997
|
91016
|
91227
|
94364
|
94365