]> git.koha-community.org Git - koha.git/commit
Bug 34147: Load all categories for displaying patron results
authorNick Clemens <nick@bywatersolutions.com>
Tue, 23 Jul 2024 12:51:00 +0000 (12:51 +0000)
committerLucas Gass <lucas@bywatersolutions.com>
Thu, 29 Aug 2024 14:28:42 +0000 (14:28 +0000)
commitb9feb3655fb8a812af779ec4eb60480d41a1c035
tree0c3a21736df0492a2edf3a12fc64c7b1c80c00b1
parentb507aad0cb1876abcea6476035ed7a7570ad0860
Bug 34147: Load all categories for displaying patron results

Currently we are only loading unlimited categories into the patron search, so when we return
patrons from a category limited to another library we crash because we don't have the descriptions
loaded.

This patch removes the limit from 'categories_map' which is only used for displaying patrons. As patron
visibility is a feature of permissions and groups, I don't think we should enforce if based on category
limitations.

NOTE: the categories for searching are still limited in the left hand filters - but not in the dropdown if you click the options button in the top search form. This is beyond the scope of this bug.

To test:
1. Make sure you are logged in as Centerville
2. Do a patron search for 'Ac' -- there are two patrons: Henry (Staff) and Edna (Patron)
2. Go to Administration > Patron Categories
3. Edit the Patron (PT) category, set the Library limitations to Fairfield only
4. Do the patron search for 'Ac' again - it hangs up with a Processing box
5. Apply patch
6. Reload the page (clear cache) and confirm the results now load

Signed-off-by: Jason Robb <jrobb@sekls.org>
Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 476776cc5b828c7fad76c584f36855dea591fd3f)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc