]> git.koha-community.org Git - koha.git/commit
Bug 26929: Fetching macros for Advanced Editor should ignore default pagesize
authorPhil Ringnalda <phil@chetcolibrary.org>
Fri, 13 Sep 2024 23:52:05 +0000 (16:52 -0700)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 30 Sep 2024 14:28:44 +0000 (15:28 +0100)
commite47be6f895dff0c50d347cd776b8deee07eb9156
tree2ffb666f58d23391d26eb7fad07564ed600dfef6
parentaecfaa5911e7ca99f73b28d3ab24bfa7f45fb661
Bug 26929: Fetching macros for Advanced Editor should ignore default pagesize

The REST API has a default page size of 20, controlled by the preference
RESTdefaultPageSize. That's fine for things that are fetched to display
in pages of 20, but for things like macros in the Advanced Editor that
don't implement paging, it's awkward, since when you create your 21st
macro, it disappears the next time you open the editor.

The Advanced Editor should instead disable paging by passing the param
_per_page=-1

Test plan:

1. Set the preference EnableAdvancedCatalogingEditor to Enable. While you
   are there, set the preference RESTdefaultPageSize to 2, so you don't have
   to create 21 macros to see one disappear
2. Cataloging - Advanced editor - Macros
3. Click New macro..., give it the name 1, then repeat for 2 and 3
4. You have a list of macros open with all three listed, so far so good.
   Refresh the browser page and reopen Macros. You now have two macros
5. Apply patch, refresh the page, reopen Macros. You now have three macros
6. You should probably reset that RESTdefaultPageSize back to 20 if you
   aren't about to reset ktd

Sponsored-by: Chetco Community Public Library
Signed-off-by: Olivier V <olivier.vezina@inLibro.com>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc