Bugzilla – Attachment 34844 Details for
Bug 8190
Add a logging module to Koha
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 8190: Followup for opac-search
Bug-8190-Followup-for-opac-search.patch (text/plain), 1.70 KB, created by
Jonathan Druart
on 2014-12-31 12:05:17 UTC
(
hide
)
Description:
Bug 8190: Followup for opac-search
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2014-12-31 12:05:17 UTC
Size:
1.70 KB
patch
obsolete
>From 1489d047e16502e0a1c5268718c27631668ad704 Mon Sep 17 00:00:00 2001 >From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Date: Wed, 12 Jun 2013 13:35:03 +0200 >Subject: [PATCH] Bug 8190: Followup for opac-search > >Script should not create a second logger object, but use the one that the Auth >module already made. >Works fine now. Only question is: Should we leave it here. This is somewhat of >a debug statement. > >Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >--- > opac/opac-search.pl | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > >diff --git a/opac/opac-search.pl b/opac/opac-search.pl >index 503e7cb..2084bcd 100755 >--- a/opac/opac-search.pl >+++ b/opac/opac-search.pl >@@ -42,14 +42,13 @@ use C4::Branch; # GetBranches > use C4::SocialData; > use C4::Ratings; > use C4::External::OverDrive; >-use Koha::Utils::Logger; > > use POSIX qw(ceil floor strftime); > use URI::Escape; > use JSON qw/decode_json encode_json/; > use Business::ISBN; > >-my $log = Koha::Utils::Logger->new({level => C4::Context->preference("LogLevel")}); >+my $logger=C4::Context->logger(); > > my $DisplayMultiPlaceHold = C4::Context->preference("DisplayMultiPlaceHold"); > # create a new CGI object >@@ -551,7 +550,7 @@ if ($tag) { > $pasarParams .= '&count=' . uri_escape($results_per_page); > $pasarParams .= '&simple_query=' . uri_escape($simple_query); > $pasarParams .= '&query_type=' . uri_escape($query_type) if ($query_type); >- $log->info("OPAC: Search for $query"); >+ logger->info("OPAC: Search for $query"); > eval { > ($error, $results_hashref, $facets) = getRecords($query,$simple_query,\@sort_by,\@servers,$results_per_page,$offset,$expanded_facet,$branches,$itemtypes,$query_type,$scan,1); > }; >-- >2.1.0
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 8190
:
9921
|
10996
|
12257
|
12264
|
12265
|
12270
|
12282
|
12283
|
12284
|
12287
|
12743
|
12744
|
12745
|
12746
|
13739
|
13740
|
13762
|
13763
|
13925
|
13926
|
14294
|
14296
|
14318
|
14319
|
14320
|
14321
|
14324
|
14388
|
14703
|
14704
|
14705
|
16426
|
16427
|
16428
|
16429
|
16430
|
18799
|
18800
|
18801
|
18802
|
18803
|
18819
|
18820
|
18870
|
18871
|
18872
|
18873
|
18874
|
18875
|
18876
|
18877
|
18880
|
18881
|
18882
|
18883
|
18884
|
18886
|
18887
|
18888
|
18890
|
34837
|
34838
|
34839
|
34840
|
34841
|
34842
|
34843
| 34844 |
34845