Nick Clemens
41580a0a58
This patch does three things:
1 - Removes a specific query for withdrawn status of each item scanned - we can use the withdrawn field
2 - Removes a specific query for checkouts on each item scanned - we can use the onloan field
a - additionally we don't need to fetch the checkout as we check it in to the homebranch,
this is likely incorrect - we should use the current branch, but I preserve behavior for now
3 - Fetches the items ahead of time and builds a hash based on barcode, reduces DB lookups, may raise memory usage
To test:
1 - Checkout some items
2 - Withdraw some items
3 - Generate a lsit of barcodes including some checked out items and some withdrawn items
4 - Enter that list of barcodes into inventory tool
5 - Note your results
6 - Apply patch
7 - Issue the items again
8 - Repeat inventory
9 - Confirm results are the same as before patch
Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit
|
||
---|---|---|
.. | ||
csv-profiles | ||
access_files.pl | ||
additional-contents.pl | ||
ajax-inventory.pl | ||
automatic_item_modification_by_age.pl | ||
batch_delete_records.pl | ||
batch_extend_due_dates.pl | ||
batch_record_modification.pl | ||
batch_records_ajax.pl | ||
batchMod.pl | ||
cleanborrowers.pl | ||
copy-holidays.pl | ||
csv-profiles.pl | ||
exceptionHolidays.pl | ||
export.pl | ||
holidays.pl | ||
import_borrowers.pl | ||
inventory.pl | ||
letter.pl | ||
manage-marc-import.pl | ||
marc_modification_templates.pl | ||
modborrowers.pl | ||
newHolidays.pl | ||
overduerules.pl | ||
page.pl | ||
picture-upload.pl | ||
problem-reports.pl | ||
quotes-upload.pl | ||
quotes.pl | ||
scheduler.pl | ||
showdiffmarc.pl | ||
stage-marc-import.pl | ||
stockrotation.pl | ||
tools-home.pl | ||
upload-cover-image.pl | ||
upload-file.pl | ||
upload.pl | ||
viewlog.pl |