Created attachment 19717 [details] [review] Bug 10601: Add a counter on batch items modifications After executing a batch items modifications, a dialog is displayed above the result table. It contains the number of items (and total fields) which has been modified. Test plan: 1/ Go to tools/batchMod.pl 2/ Enter a barcodes list 3/ Check/uncheck items and fill some values to apply 4/ Save 5/ The table summary will be displayed with a dialog box on top: XX item(s) modified (with YY fields modified) Check that XX and YY correspond with what you expected.
Created attachment 19741 [details] [review] Bug 10601: Add a counter on batch items modifications After executing a batch items modifications, a dialog is displayed above the result table. It contains the number of items (and total fields) which has been modified. Test plan: 1/ Go to tools/batchMod.pl 2/ Enter a barcodes list 3/ Check/uncheck items and fill some values to apply 4/ Save 5/ The table summary will be displayed with a dialog box on top: XX item(s) modified (with YY fields modified) Check that XX and YY correspond with what you expected. Signed-off-by: Liz Rea <liz@catalyst.net.nz> batch modification still seems to work correctly, with the helpful addition of the counter. Thanks!
Created attachment 20411 [details] [review] Bug 10601: Add a counter on batch items modifications After executing a batch items modifications, a dialog is displayed above the result table. It contains the number of items (and total fields) which has been modified. Test plan: 1/ Go to tools/batchMod.pl 2/ Enter a barcodes list 3/ Check/uncheck items and fill some values to apply 4/ Save 5/ The table summary will be displayed with a dialog box on top: XX item(s) modified (with YY fields modified) Check that XX and YY correspond with what you expected. Signed-off-by: Liz Rea <liz@catalyst.net.nz> batch modification still seems to work correctly, with the helpful addition of the counter. Thanks! Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 20412 [details] [review] Bug 10601 [QA Followup]: Add a counter on batch items modifications Fixes a koha-qa.pl error about the pod documentation.
Pushed to master, along with followups that add unit tests for the new methods in C4::BackgroundJob.
Thanks, Jonathan!