Bugzilla – Attachment 18888 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
Marcel de Rooy
on 2013-06-12 12:23:40 UTC
(
hide
)
Description:
Bug 8190: Followup for opac-search
Filename:
MIME Type:
Creator:
Marcel de Rooy
Created:
2013-06-12 12:23:40 UTC
Size:
1.70 KB
patch
obsolete
>From 7894a4a126cc010428f249584b4d502460418791 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 >Content-Type: text/plain; charset=utf-8 > >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 files changed, 2 insertions(+), 3 deletions(-) > >diff --git a/opac/opac-search.pl b/opac/opac-search.pl >index 9046062..e2857db 100755 >--- a/opac/opac-search.pl >+++ b/opac/opac-search.pl >@@ -51,14 +51,13 @@ use C4::Tags qw(get_tags); > use C4::Branch; # GetBranches > use C4::SocialData; > use C4::Ratings; >-use Koha::Utils::Logger; > > use POSIX qw(ceil floor strftime); > use URI::Escape; > use Storable qw(thaw freeze); > 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 >@@ -521,7 +520,7 @@ if ($tag) { > $pasarParams .= '&count=' . $results_per_page; > $pasarParams .= '&simple_query=' . $simple_query; > $pasarParams .= '&query_type=' . $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); > }; >-- >1.7.7.6
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