Bugzilla – Attachment 4499 Details for
Bug 6510
"Sort by" in intranet search alters search and number of hits
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 6510 - [REVISED] Sort by in intranet search alters search and number of hits
Bug-6510---REVISED-Sort-by-in-intranet-search-alte.patch (text/plain), 1.15 KB, created by
MJ Ray (software.coop)
on 2011-06-16 13:36:17 UTC
(
hide
)
Description:
Bug 6510 - [REVISED] Sort by in intranet search alters search and number of hits
Filename:
MIME Type:
Creator:
MJ Ray (software.coop)
Created:
2011-06-16 13:36:17 UTC
Size:
1.15 KB
patch
obsolete
>From b2845bd3f67fe6702439103408993c89616b8d61 Mon Sep 17 00:00:00 2001 >From: Magnus Enger <magnus@enger.priv.no> >Date: Thu, 16 Jun 2011 13:51:58 +0200 >Subject: [PATCH] Bug 6510 - [REVISED] Sort by in intranet search alters search and number of hits > >This patch uses .*? instead of .* > >To test: >- In advanced search in the intranet choose Author as the search index >- Do a search for an author, check the number of hits >- Choose another value than the default from Sort by >- Check that the number of hits is the same as for the original search, once > the hits have been re-ordered > >Signed-off-by: MJ Ray <mjr@phonecoop.coop> >--- > catalogue/search.pl | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > >diff --git a/catalogue/search.pl b/catalogue/search.pl >index a41b18a..8899d59 100755 >--- a/catalogue/search.pl >+++ b/catalogue/search.pl >@@ -471,7 +471,7 @@ my $scan_index_to_use; > > for my $this_cgi ( split('&',$query_cgi) ) { > next unless $this_cgi; >- $this_cgi =~ m/(.?)=(.*)/; >+ $this_cgi =~ m/(.*?)=(.*)/; > my $input_name = $1; > my $input_value = $2; > push @query_inputs, { input_name => $input_name, input_value => $input_value }; >-- >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 6510
:
4493
|
4494
| 4499