To recreate: - find a bib with at least one item - click Edit > Delete all items - end up on additem.pl Looks like this is a regression, not sure what caused it. BZ shows Bug 7227 for this long ago.
Created attachment 103477 [details] [review] Bug 25248: Item is returned if deletion successful, not 1 To test: 1 - Have a record with some items 2 - Click 'Delete all' under 'Edit' 3 - Confirm deletion 4 - Note you are redirected to additem.pl 5 - Add an item 6 - Apply patch 7 - Delete all items again 8 - Note you are redirected to detail.pl
Created attachment 103481 [details] [review] Bug 25248: (bug 23463 follow-up) Item is returned if deletion successful, not 1 To test: 1 - Have a record with some items 2 - Click 'Delete all' under 'Edit' 3 - Confirm deletion 4 - Note you are redirected to additem.pl 5 - Add an item 6 - Apply patch 7 - Delete all items again 8 - Note you are redirected to detail.pl Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Amended patch, add relation to 23463 in commit title and remove debug statements
Trivial patch, skipping QA.
Thanks, Jonathan!
Nice work everyone! Pushed to master for 20.05
missing dependencies - not backported
Created attachment 104693 [details] [review] Bug 25248: (bug 23463 follow-up) Adjust second occurrence Same as the precedent patch for a single item deletion (vs delete all items) Item is returned if deletion successful, not 1