Search for: "edna acosta" OR "henry acevedo" OR you favorite default patron No results! The query is search for both terms in each field: i.e. firstname LIKE 'edna acosta'
Created attachment 134251 [details] [review] Bug 30639: Split search terms for patron search If several terms are passed we should split them. Test plan: Search for "edna acosta" (without quotes) in the filters from the left side of the main patron search Do some regression tests
This is still not perfect for patron attributes. If you have: attribute 1 = xxx attribute 2 = yyy and you search for "xxx yyy", the patron won't be returned.
(In reply to Jonathan Druart from comment #2) > This is still not perfect for patron attributes. > If you have: > attribute 1 = xxx > attribute 2 = yyy > > and you search for "xxx yyy", the patron won't be returned. Reported on bug 30645.
Created attachment 134263 [details] [review] Bug 30639: Split search terms for patron search If several terms are passed we should split them. Test plan: Search for "edna acosta" (without quotes) in the filters from the left side of the main patron search Do some regression tests Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Looking here
# Pattern terms must be split $s->fill_form( { search_patron_filter => 'test patron' } ); $s->submit_form; $s->wait_for_ajax; is( $driver->find_element('//div[@id="'.$table_id.'_info"]')->get_text, sprintf('Showing 1 to %s of %s entries (filtered from %s total entries)', $PatronsPerPage, 26, $total_number_of_patrons) ); Hmm I cant run this test. But Nick tested it. Just dont understand the 26 here. Reading the test I expect 25 ?
(In reply to Marcel de Rooy from comment #6) > # Pattern terms must be split > $s->fill_form( { search_patron_filter => 'test patron' } ); > $s->submit_form; > > $s->wait_for_ajax; > is( $driver->find_element('//div[@id="'.$table_id.'_info"]')->get_text, > sprintf('Showing 1 to %s of %s entries (filtered from %s total entries)', > $PatronsPerPage, 26, $total_number_of_patrons) ); > > Hmm I cant run this test. But Nick tested it. > Just dont understand the 26 here. Reading the test I expect 25 ? For most of the test we are filtering by library and so we get the 25 from 77 for my $i ( 1 .. 25 ) { 78 push @patrons, 79 $builder->build_object( But there is a 26th patron matching test_patron_* 97 push @patrons, 98 $builder->build_object( 99 { 100 class => 'Koha::Patrons', 101 value => { 102 surname => "test_patron_26", Which is from a different library.
Created attachment 134709 [details] [review] Bug 30639: Split search terms for patron search If several terms are passed we should split them. Test plan: Search for "edna acosta" (without quotes) in the filters from the left side of the main patron search Do some regression tests Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Pushed to master for 22.05, thanks to everybody involved [U+1F984]