From 77e2b90c190533eabec906774cbc48c7a7b905c5 Mon Sep 17 00:00:00 2001 From: Olivier Hubert Date: Wed, 26 Oct 2022 12:29:20 -0400 Subject: [PATCH] Bug 31994: DataTable next button does not work when using data in the HTML This patch resolves the issue that occurs when the next button of a DataTable with data straight from the HTML is clicked. Test plan: 1. Open itemtypes.pl. 2. If there are fewer than 11 item types, create additional item types until there are more than 10, so that the next button can be used. 3. Click on the next button / link. 4. Notice that nothing happens. 5. Apply the patch. 6. Restart plack / memcached if using. 7. Reload itemtypes.pl. 8. Click on the next button / link. 9. Notice that the next page is displayed. --- koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc index 8df6eb56cc..b0955d3367 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/columns_settings.inc @@ -202,7 +202,8 @@ function KohaTable(id_selector, dt_parameters, table_settings, add_filters) { if ( table_settings ) { if ( table_settings.hasOwnProperty('default_display_length') && table_settings['default_display_length'] != null ) { - new_parameters["pageLength"] = table_settings['default_display_length']; + // pageLength needs to be a number, not a string, or it can cause issues with DataTable's next button. + new_parameters["pageLength"] = parseInt(table_settings['default_display_length']); } if ( table_settings.hasOwnProperty('default_sort_order') && table_settings['default_sort_order'] != null ) { new_parameters["order"] = [[ table_settings['default_sort_order'], 'asc' ]]; -- 2.34.1