Bug 3629 - search limit by group is not OK
Summary: search limit by group is not OK
Alias: None
Product: Koha
Classification: Unclassified
Component: Searching (show other bugs)
Version: rel_3_2
Hardware: PC All
: PATCH-Sent (DO NOT USE) normal (vote)
Assignee: Paul Poulain
QA Contact:
Depends on:
Blocks: 5584
  Show dependency treegraph
Reported: 2009-09-15 01:04 UTC by Chris Cormack
Modified: 2012-10-25 23:04 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:

Proposed Patch (3.11 KB, patch)
2011-03-29 02:37 UTC, Chris Cormack
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Cormack 2010-05-21 01:13:03 UTC

---- Reported by laurence.lefaucheur@biblibre.com 2009-09-15 13:04:26 ----

When a group of searchdomain type is created, advanced search allow to limit by group.
Limiting by group does not take into account the searched words : 

Example : 
Limit by separated branch : 
No results match your search for 'kw,wrdl: psychologie' with limit(s): 'branch:Bibliothèque de l'IUFM
3 result(s) found for 'kw,wrdl: psychologie' with limit(s): 'branch:Bibliothèque de Médecine Timone'

Limit by group containing these two branches : 
250 result(s) found for 'kw,wrdl: psychologie' with limit(s): 'branch: U1BUFMAIX or branch: U2TIMONMED'
=>shows all data from the two branches

--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:13 UTC  ---

This bug was previously known as _bug_ 3629 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=3629

Actual time not defined. Setting to 0.0
The original reporter of this bug does not have
   an account here. Reassigning to the person who moved
   it here: chris@bigballofwax.co.nz.
   Previous reporter was laurence.lefaucheur@biblibre.com.

Comment 1 Chris Cormack 2011-03-29 02:37:49 UTC
Created attachment 3490 [details] [review]
Proposed Patch
Comment 2 Chris Cormack 2011-03-29 23:38:55 UTC
Pushed, please test