to reproduce: 1.) Go to ERM module and make sure you have more than 21 data providers stored in your database. 2.) Go to eUsage / reports and create new report (cgi-bin/koha/erm/eusage/reports) 3.) Check that only the first 20 data providers are displayed in the drop down
Created attachment 167217 [details] [review] Bug 36956: Allow all providers to be displayed in reports This patch fixes an incorrect usage of the API which was only returning 20 results. Test plan: 1) Go to ERM module and make sure you have more than 21 data providers stored in your database. 2) Go to eUsage / reports and create new report (cgi-bin/koha/erm/eusage/reports) 3) Check that only the first 20 data providers are displayed in the drop down 4) Apply patch 5) yarn build or reset_all to rebuild the javascript 6) Repeat steps 1-3, all providers should now be displayed
Created attachment 167221 [details] [review] Bug 36956: Allow all providers to be displayed in reports This patch fixes an incorrect usage of the API which was only returning 20 results. Test plan: 1) Go to ERM module and make sure you have more than 21 data providers stored in your database. 2) Go to eUsage / reports and create new report (cgi-bin/koha/erm/eusage/reports) 3) Check that only the first 20 data providers are displayed in the drop down 4) Apply patch 5) yarn build or reset_all to rebuild the javascript 6) Repeat steps 1-3, all providers should now be displayed Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Created attachment 167933 [details] [review] Bug 36956: Allow all providers to be displayed in reports This patch fixes an incorrect usage of the API which was only returning 20 results. Test plan: 1) Go to ERM module and make sure you have more than 20 data providers stored in your database. 1) (Optional) Run the following command (k-t-d) to create 21 data providers: perl -MKoha::Database -e ' my $schema = Koha::Database->schema; $schema->resultset("ErmUsageDataProvider")->create({ name => "name ".$_ }) for (1..21) ' 2) Go to eUsage / reports and create new report (cgi-bin/koha/erm/eusage/reports) 3) Check that only the first 20 data providers are displayed in the drop down 4) Apply patch 5) yarn build or reset_all to rebuild the javascript 6) Repeat steps 1-3, all providers should now be displayed PA amended patch: Updated commit message test plan Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Such a small change, such a big effect :)
Pushed for 24.11! Well done everyone, thank you!
Adding backport release keywords, can we please add this to 24.05.x and 23.11.x.
Backported to 24.05.x for upcoming 24.05.02
Pushed to 23.11.x for 23.11.07
not backporting to 23.05.x unless requested
I have: 1. Added a release note. 2. Removed the rel_23_11_candidate keyword, as it was pushed for 23.11.07. 3. Updated the status to RESOLVED FIXED - not backported to 23.05, no changes to the ERM chapter of the manual are required.