Bug 24106 - In returns.pl, don't search for item if no barcode is provided
Summary: In returns.pl, don't search for item if no barcode is provided
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Kyle M Hall (khall)
QA Contact: Tomás Cohen Arazi (tcohen)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-11-25 18:18 UTC by Kyle M Hall (khall)
Modified: 2021-06-14 21:29 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.01
Circulation function:


Attachments
Bug 24106: In returns.pl, don't search for item if no barcode is provided (2.43 KB, patch)
2019-11-25 18:20 UTC, Kyle M Hall (khall)
Details | Diff | Splinter Review
Bug 24106: In returns.pl, don't search for item if no barcode is provided (2.46 KB, patch)
2019-11-25 19:27 UTC, Liz Rea
Details | Diff | Splinter Review
Bug 24106: In returns.pl, don't search for item if no barcode is provided (2.51 KB, patch)
2019-11-29 15:03 UTC, Tomás Cohen Arazi (tcohen)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Kyle M Hall (khall) 2019-11-25 18:18:01 UTC
When loading returns.pl, code to check rotating collections fires off an search for items by barcode, but doesn't check for a barcode first. This means the code will search for items where barcode is NULL, which is definitely not the intended function.
Comment 1 Kyle M Hall (khall) 2019-11-25 18:20:36 UTC
Created attachment 95805 [details] [review]
Bug 24106: In returns.pl, don't search for item if no barcode is provided

When loading returns.pl, code to check rotating collections fires off an search for items by barcode, but doesn't check for a barcode first. This means the code will search for items where barcode is NULL, which is definitely not the intended function.

Test Plan:
1) Apply this patch
2) Set up a rotating collection with items
3) Transfer the collection
4) Check in a rotating collection item
5) Note no change in functionality
Comment 2 Liz Rea 2019-11-25 19:27:40 UTC
Created attachment 95806 [details] [review]
Bug 24106: In returns.pl, don't search for item if no barcode is provided

When loading returns.pl, code to check rotating collections fires off an search for items by barcode, but doesn't check for a barcode first. This means the code will search for items where barcode is NULL, which is definitely not the intended function.

Test Plan:
1) Apply this patch
2) Set up a rotating collection with items
3) Transfer the collection
4) Check in a rotating collection item
5) Note no change in functionality

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Comment 3 Tomás Cohen Arazi (tcohen) 2019-11-29 15:03:14 UTC
Created attachment 95883 [details] [review]
Bug 24106: In returns.pl, don't search for item if no barcode is provided

When loading returns.pl, code to check rotating collections fires off an search for items by barcode, but doesn't check for a barcode first. This means the code will search for items where barcode is NULL, which is definitely not the intended function.

Test Plan:
1) Apply this patch
2) Set up a rotating collection with items
3) Transfer the collection
4) Check in a rotating collection item
5) Note no change in functionality

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 4 Liz Rea 2019-12-02 15:01:14 UTC
It appears that this can cause Koha to hang looking for all the null barcode items when processing holds, enough retries and all of the plack processes get eaten up.

It doesn't happen for every library, however, so it must be related to a system preference, which I have not tracked down (yet).
Comment 5 Martin Renvoize (ashimema) 2019-12-13 14:07:31 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 6 Joy Nelson 2019-12-19 22:06:17 UTC
Pushed to 19.11.01
Comment 7 Lucas Gass (lukeg) 2019-12-20 18:15:39 UTC
backported to 19.05.x for 19.05.06