This fix restricts the "Select all" (as well as the "Clear all") button from changing the "checked" status of disabled checkboxes in the Batch Record Deletion Tool. Patch and test plan will be added shortly.
Created attachment 72726 [details] [review] Bug 20376: Select all ignores disabled checkboxes Test Plan: 0.1 - find at least one biblionumber that has at least one item checked out 0.2 - find at least one biblionumber that has zero items checked out. 1 - Go into the module "Tools > Batch record deletion" 2 - in "List of biblionumbers or authority ids (one per line)" put the values you found in step 0.1 and 0.2 3 - click continue 4 - biblionumbers with at least one item checked out should not be clickable. 5 - biblionumbers with zero items checked out should be clickable. 6 - click "Select all" and "Clear all", both buttons should change the "checked" status of the enabled AND disabled biblionumber checkboxes. 7 - apply patch 8 - click "Select all" and "Clear all", both buttons should change the "checked" status of ONLY the enabled biblionumber checkboxes.
Patch tested with a sandbox, by JM Broust <jean-manuel.broust@univ-lyon2.fr>
Created attachment 72774 [details] [review] Bug 20376: Select all ignores disabled checkboxes Test Plan: 0.1 - find at least one biblionumber that has at least one item checked out 0.2 - find at least one biblionumber that has zero items checked out. 1 - Go into the module "Tools > Batch record deletion" 2 - in "List of biblionumbers or authority ids (one per line)" put the values you found in step 0.1 and 0.2 3 - click continue 4 - biblionumbers with at least one item checked out should not be clickable. 5 - biblionumbers with zero items checked out should be clickable. 6 - click "Select all" and "Clear all", both buttons should change the "checked" status of the enabled AND disabled biblionumber checkboxes. 7 - apply patch 8 - click "Select all" and "Clear all", both buttons should change the "checked" status of ONLY the enabled biblionumber checkboxes. Signed-off-by: JM Broust <jean-manuel.broust@univ-lyon2.fr>
Created attachment 72797 [details] [review] Bug 20376: Make select/unselect all ignore disabled checkboxes on batch record deletion Test Plan: 0.1 - find at least one biblionumber that has at least one item checked out 0.2 - find at least one biblionumber that has zero items checked out. 1 - Go into the module "Tools > Batch record deletion" 2 - in "List of biblionumbers or authority ids (one per line)" put the values you found in step 0.1 and 0.2 3 - click continue 4 - biblionumbers with at least one item checked out should not be clickable. 5 - biblionumbers with zero items checked out should be clickable. 6 - click "Select all" and "Clear all", both buttons should change the "checked" status of the enabled AND disabled biblionumber checkboxes. 7 - apply patch 8 - click "Select all" and "Clear all", both buttons should change the "checked" status of ONLY the enabled biblionumber checkboxes. Signed-off-by: JM Broust <jean-manuel.broust@univ-lyon2.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Patch pushed to master for 18.05 Thanks Maksim for your first patch in!
Awesome work all, backported to stable for 17.11.05
Pushed to 17.05.x for 17.05.11