https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24236

ByWater Sandboxes <bws.sandbo...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #96264|0                           |1
        is obsolete|                            |

--- Comment #5 from ByWater Sandboxes <bws.sandbo...@gmail.com> ---
Created attachment 97407
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=97407&action=edit
Bug 24236: Fix pagination to use query_cgi

This is still wrong, we should use page-numbers.inc, but that is a larger
refactoring
This patch fixes non-QueryParser searches by simple getting an using the
escaped query (query_cgi)

For QueryParser, we do what the QP code does, just uri escape the query - the
way that SimpleSearch does the query parsing instead of the usual buildQuery
suggest to me that we should probably switch all of this to use the general
Zebra search.

The issues with reservoir should be their own bug

To test:
1 - Search for cat in cataloging search
2 - Get results and see you can paginate
3 - Search for "cat"
4 - Get results but subsequent pages empty
5 - Apply patch
6 - Repeat search for "cat"
7 - You can paginate
8 - Enable UseQueryParser syspref (don't worry about installing)
9 - Confirm can still search with quotes and paginate

Signed-off-by: Nick Clemens <n...@bywatersolutions.com>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to