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

David Nind <da...@davidnind.com> changed:

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

--- Comment #8 from David Nind <da...@davidnind.com> ---
Created attachment 182282
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=182282&action=edit
Bug 39637: Build the patron search query on the server side

Building the patron search query on the browser side causes the URI to
grow rapidly and can reach the HTTP server limit, resulting in a 414
error (Request-URI Too Long)

This patch moves the logic to the backend (more precisely, in
Koha::REST::V1::Patrons) so that only user input needs to be sent as URL
parameters.

Test plan:
1. Log in to the staff interface
2. In the top search bar, select "Patrons search" and type "a b c d e f
   g h i j k l m n o p q r s t u v w x y z" and submit the form
3. The patrons table should load without errors (but will probably be
   empty)
4. Test the various filters on the left sidebar and verify that they
   work exactly as before the patch.
5. Modify system preferences DefaultPatronSearchFields and
   DefaultPatronSearchMethod and verify that they work exactly as before
   the patch.
6. If not done already, create searchable patron attributes and verify
   that search works for those too.
7. prove t/db_dependent/api/v1/patrons.t

Signed-off-by: David Nind <da...@davidnind.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