]> git.koha-community.org Git - koha.git/commit
Bug 18833: Plugin unimarc_field_210c pagination error
authorFridolin Somers <fridolin.somers@biblibre.com>
Wed, 21 Jun 2017 11:05:40 +0000 (13:05 +0200)
committerFridolin Somers <fridolin.somers@biblibre.com>
Wed, 20 Dec 2017 08:13:16 +0000 (09:13 +0100)
commit642355033e38bfb60144c4384f08e7a4b01b3e23
treeaaa8445bc34e0ced951cbb8d8e65b5f50874c5d5
parentaa3d1a469d4af69bdd5676cfc963905572574ae6
Bug 18833: Plugin unimarc_field_210c pagination error

In plugin unimarc_field_210c, the editors search results have pagination.
First page works but not other pages.

The main problem whas mainly because $startfrom var was missused. It is defined in C4::Output::pagination_bar as page number.
This patch corrects by using a $offset var.
Also removes dead code, obsoleted by using C4::Output::pagination_bar.
Also changes hardcoded results per page from 19 to 20.

Test plan :
- Use UNIMARC catalogue
- Define in framework value builder unimarc_field_210c on 210$c
- Create autority type EDITORS with heading 200$b
- Create 22 authorities of that type
- Index those new authorities
- Create a new biblio record
- Click on 210$c plugin
- Click on "Search"
- You see 20 results
- Click on page 2
=> Without patch you get empty table
=> With patch you get 2 results
- Check that you see "Results 21 to 22 of 22"
- Delete 2 autorities and retest

Works as intended.

Signed-off-by: Simon Pouchol <simon.pouchol@biblibre.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 455d94eec9c41f7bad718ac62c2367bea6202452)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
cataloguing/value_builder/unimarc_field_210c.pl