Summary: | Background tasks can be called multiple times on batch item deletion | ||
---|---|---|---|
Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
Component: | Hold requests | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | CLOSED FIXED | QA Contact: | Tomás Cohen Arazi (tcohen) <tomascohen> |
Severity: | normal | ||
Priority: | P5 - low | CC: | david, gmcharlt, jonathan.druart, martin.renvoize, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Small patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
22.05.00
|
Circulation function: | |
Bug Depends on: | 29346, 30467 | ||
Bug Blocks: | 30725, 30727, 30728 | ||
Attachments: |
Bug 30710: Build holds queue once per biblio when batch deleting items
Bug 30710: Build holds queue once per biblio when batch deleting items Bug 30710: Regression tests Bug 30710: Build holds queue once per biblio when batch deleting items Bug 30710: (QA follow-up) Add POD |
Description
Nick Clemens (kidclamp)
2022-05-06 13:00:32 UTC
This is a bug. This also impacts background ES indexing To test: 1 - Find a record with multiple items 2 - On details page click 'delete items in a batch' 3 - Delete items 4 - Check background jobs list 5 - Note one reindex per item deleted, plus a non backgrounded deletion from index if record deleted (In reply to Nick Clemens from comment #2) > This also impacts background ES indexing It's fixed on master (or patch is in PQA queue), search for "task does not deal with indexation correctly" Created attachment 134742 [details] [review] Bug 30710: Build holds queue once per biblio when batch deleting items Created attachment 134743 [details] [review] Bug 30710: Build holds queue once per biblio when batch deleting items Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Great work, clear fix. Signing off Changing status to signed off. Created attachment 134817 [details] [review] Bug 30710: Regression tests Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Created attachment 134818 [details] [review] Bug 30710: Build holds queue once per biblio when batch deleting items Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Created attachment 134819 [details] [review] Bug 30710: (QA follow-up) Add POD Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Pushed to master for 22.05, thanks to everybody involved [U+1F984] |