Attachment 17205 Details for Bug 9588 – Bug 9588: weighted search query with index

[patch] Bug 9588: weighted search query with index

Bug-9588-weighted-search-query-with-index.patch (text/plain), 3.07 KB, created by Marcel de Rooy on 2013-04-05 09:24:05 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2013-04-05 09:24:05 UTC

Size: 3.07 KB

patch

obsolete

>From 130c65e06f8ebb5934377acdc79f8409c9cd17d5 Mon Sep 17 00:00:00 2001
>From: Fridolyn SOMERS <fridolyn.somers@biblibre.com>
>Date: Mon, 11 Feb 2013 14:47:09 +0100
>Subject: [PATCH] Bug 9588: weighted search query with index
>Content-Type: text/plain; charset=utf-8
>
>When QueryWeightFields is enabled, the searching query is created with several options.
>In C4::Search::_build_weighted_query, when no index is defined, the query is build with fuzzy and stemming options. When an index is defined, theses options are missing, only unconditional right truncation is used.
>The consequence is that when QueryStemming is disabled, a search with index can give more results (due to right truncation) that a search without.
>
>This patch adds stemming and fuzzy on search with index, conditioned with QueryFuzzy and QuerryStemming sysprefs.
>Also changes world list search (wrld) weight to r6 in order to set fuzzy search to r8 and stemming search to r9 (like search without index).
>
>Test plan :
>- Go to searching preferences (admin/preferences.pl?tab=searching)
>- Set QueryAutoTruncate to "only if * is added"
>- Set QueryFuzzy and QuerryStemming to "Don't try"
>- Set QueryWeightFields to "Enable"
>- Go to advanced search page
>- Select an indexe (ie Title) and perform a search on a short word
>=> Look at zebrarv log and see that query does not contain right truncation : @attr 5=1
>- Set QueryFuzzy to "Try"
>- Perform same search
>=> Look at zebrarv log and see that query contains fuzzy : @attr 5=103
>- Set QueryFuzzy to "Don't try" and QuerryStemming to "Try"
>- Perform same search
>=> Look at zebrarv log and see that query contains rigth truncation on stemmed word : @attr 5=1
>
>Signed-off-by: koha.aixmarseille <koha.aixmarseille@gmail.com>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>This patch makes Fuzzy and Stemming influence search results on weighted
>queries when using an index. Side-effect is however that the results for a
>search like index=term* (add truncation manually too) could be LOWER than the
>the number of hits for index=term. Further comments on Bugzilla.
>---
> C4/Search.pm |    7 +++++--
> 1 files changed, 5 insertions(+), 2 deletions(-)
>
>diff --git a/C4/Search.pm b/C4/Search.pm
>index d4ce9fb..8261e0a 100644
>--- a/C4/Search.pm
>+++ b/C4/Search.pm
>@@ -890,8 +890,11 @@ sub _build_weighted_query {
>         $weighted_query .= " $index,ext,r1=\"$operand\"";    # exact index
>           #$weighted_query .= " or (title-sort-az=0 or $index,startswithnt,st-word,r3=$operand #)";
>         $weighted_query .= " or $index,phr,r3=\"$operand\"";    # phrase index
>-        $weighted_query .=
>-          " or $index,rt,wrdl,r3=\"$operand\"";    # word list index
>+        $weighted_query .= " or $index,wrdl,r6=\"$operand\"";    # word list index
>+        $weighted_query .= " or $index,wrdl,fuzzy,r8=\"$operand\""
>+          if $fuzzy_enabled;    # add fuzzy, word list
>+        $weighted_query .= " or $index,wrdl,rt,r9=\"$stemmed_operand\""
>+          if ( $stemming and $stemmed_operand );    # add stemming, right truncation
>     }
> 
>     $weighted_query .= "))";                       # close rank specification
>-- 
>1.7.7.6

Actions: View | Diff | Splinter Review

Attachments on bug 9588: 15239 | 15271 | 17205