Bug 32237

Summary: Batch delete records "no record IDs defined"
Product: Koha Reporter: geraud.frappier
Component: ListsAssignee: geraud.frappier
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: arthur.suzuki, elise.konya, hebah, jennifer.bauserman, jennifer.marlatt, lucas, m.de.rooy, sbcornell
Version: MainKeywords: regression
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32255
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
22.11.00, 22.05.10
Circulation function:
Attachments: Bug 32237 : Batch delete records "no record IDs defined". List of records not shown.
Bug 32237: Fix using a list as base for batch record deletion
Bug 32237: Fix using a list as base for batch record deletion

Description geraud.frappier 2022-11-16 21:43:14 UTC
To test : 
1. Go to Lists in the intranet.
2. Add a list by clicking on "New list". Choose a random name for it and select private or public and save.
3.Add one or more items into it and save.
4.Go back to the main page and select "Cataloging".
5.Go to "Batch record deletion".
6.In "upload a file" select "Select a list of records" and choose a list and keep the Bibliographic on.
7.Select continue and see that the list of records is not shown.
Comment 1 Katrin Fischer 2022-11-16 21:55:03 UTC
Confirmed on master.
Comment 2 geraud.frappier 2022-11-16 22:02:52 UTC
Created attachment 143975 [details] [review]
Bug 32237 : Batch delete records "no record IDs defined". List of records not shown.

To test patch carry out the same procedure.
Comment 3 Katrin Fischer 2022-11-16 22:05:11 UTC
Hi Geraud, please add a problem/solution description and the full test plans to the commit messages. This might be helpful: https://wiki.koha-community.org/wiki/Commit_messages
Comment 4 Katrin Fischer 2022-11-16 22:12:42 UTC
Created attachment 143976 [details] [review]
Bug 32237: Fix using a list as base for batch record deletion

When a list was used to select records for batch deletion, the
record IDs were no longer found.

To test:

1. Go to Lists in the intranet.
2. Add a list by clicking on "New list". Choose a random name for it and select private or public and save.
3.Add one or more items into it and save.
4.Go back to the main page and select "Cataloging".
5.Go to "Batch record deletion".
6.In "upload a file" select "Select a list of records" and choose a list and keep the Bibliographic on.
7.Select continue and see that the list of records is not shown.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Marcel de Rooy 2022-11-18 09:07:10 UTC
QA: Looking here
Comment 6 Marcel de Rooy 2022-11-18 09:10:46 UTC
Created attachment 144049 [details] [review]
Bug 32237: Fix using a list as base for batch record deletion

When a list was used to select records for batch deletion, the
record IDs were no longer found.

To test:

1. Go to Lists in the intranet.
2. Add a list by clicking on "New list". Choose a random name for it and select private or public and save.
3.Add one or more items into it and save.
4.Go back to the main page and select "Cataloging".
5.Go to "Batch record deletion".
6.In "upload a file" select "Select a list of records" and choose a list and keep the Bibliographic on.
7.Select continue and see that the list of records is not shown.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Tomás Cohen Arazi (tcohen) 2022-11-22 19:40:30 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!
Comment 8 hebah 2022-12-28 16:54:09 UTC
Any chance this can be backported to 22.05?
Comment 9 Jennifer Marlatt 2022-12-28 17:22:56 UTC
Yes, please! I would love to see this backported to 22.05. I use this feature frequently.
Comment 10 jennifer.bauserman 2023-01-09 17:07:18 UTC
Please backport 22.05!
Comment 11 Sarah Cornell 2023-02-09 01:28:05 UTC
same here!  We just discovered it wasn't working in 22.05.07.
Comment 12 Lucas Gass (lukeg) 2023-02-24 13:42:51 UTC
Backported to 22.05.x for upcoming 22.05.10
Comment 13 Arthur Suzuki 2023-02-28 07:51:10 UTC
can't reproduce on 21.11.x, won't backport