Bugzilla – Attachment 16189 Details for
Bug 9239
Koha should share Evergreen's QueryParser module for parsing queries
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 9239 QA follow-up: the last QA follow-up was missing a require
Bug-9239-QA-follow-up-the-last-QA-follow-up-was-mi.patch (text/plain), 2.41 KB, created by
Jared Camins-Esakov
on 2013-03-17 01:28:18 UTC
(
hide
)
Description:
Bug 9239 QA follow-up: the last QA follow-up was missing a require
Filename:
MIME Type:
Creator:
Jared Camins-Esakov
Created:
2013-03-17 01:28:18 UTC
Size:
2.41 KB
patch
obsolete
>From 0165ca7ff031c8e95edc037974bd41717656272b Mon Sep 17 00:00:00 2001 >From: Jared Camins-Esakov <jcamins@cpbibliography.com> >Date: Sun, 10 Mar 2013 09:17:21 -0400 >Subject: [PATCH] Bug 9239 QA follow-up: the last QA follow-up was missing a > require > >This patch also corrects the definition of the an= index, which was >missing exactness. > >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com> >--- > C4/Search.pm | 1 + > acqui/neworderbiblio.pl | 6 +----- > etc/searchengine/queryparser.yaml | 3 +++ > 3 files changed, 5 insertions(+), 5 deletions(-) > >diff --git a/C4/Search.pm b/C4/Search.pm >index 2c3e605..d20e403 100644 >--- a/C4/Search.pm >+++ b/C4/Search.pm >@@ -1212,6 +1212,7 @@ sub parseQuery { > $QParser->parse( $query ); > $operands[0] = "pqf=" . $QParser->target_syntax('biblioserver'); > } else { >+ require Koha::QueryParser::Driver::PQF; > my $modifier_re = '#(' . join( '|', @{Koha::QueryParser::Driver::PQF->modifiers}) . ')'; > s/$modifier_re//g for @operands; > } >diff --git a/acqui/neworderbiblio.pl b/acqui/neworderbiblio.pl >index 5824f6b..bc782af 100755 >--- a/acqui/neworderbiblio.pl >+++ b/acqui/neworderbiblio.pl >@@ -93,13 +93,9 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user( > ); > > # Searching the catalog. >-my @operands = $query; >-my ( @operators, @indexes, @sort_by, @limits ) = (); >-my ( $builterror, $builtquery, $simple_query, $query_cgi, $query_desc, $limit, $limit_cgi, $limit_desc, $stopwords_removed, $query_type ) = >- buildQuery( \@operators, \@operands, \@indexes, @limits, \@sort_by, undef, undef ); > > # find results >-my ( $error, $marcresults, $total_hits ) = SimpleSearch( $builtquery, $results_per_page * ( $page - 1 ), $results_per_page ); >+my ( $error, $marcresults, $total_hits ) = SimpleSearch( $query, $results_per_page * ( $page - 1 ), $results_per_page ); > > if (defined $error) { > $template->param( >diff --git a/etc/searchengine/queryparser.yaml b/etc/searchengine/queryparser.yaml >index 94f9d8e..f54ac3b 100644 >--- a/etc/searchengine/queryparser.yaml >+++ b/etc/searchengine/queryparser.yaml >@@ -1075,6 +1075,9 @@ field_mappings: > bib1_mapping: > biblioserver: > 1: Koha-Auth-Number >+ 3: 1 >+ 4: 1 >+ 5: 100 > enabled: 1 > index: authority-number > label: Authority-number >-- >1.7.9.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 9239
:
14020
|
14021
|
14022
|
14023
|
14172
|
14173
|
14174
|
14175
|
14180
|
14181
|
14182
|
14183
|
14847
|
14949
|
14950
|
14951
|
14952
|
14953
|
15121
|
15437
|
15865
|
15867
|
15929
|
15999
|
16004
|
16051
|
16125
|
16130
|
16131
|
16132
|
16133
|
16136
|
16140
|
16141
|
16142
|
16143
|
16144
|
16145
|
16146
|
16147
|
16148
|
16149
|
16150
|
16151
|
16152
|
16153
|
16154
|
16155
|
16180
|
16181
|
16182
|
16183
|
16184
|
16185
|
16186
|
16187
|
16188
| 16189 |
16190
|
16191
|
16192
|
16193
|
16194
|
16195