[Koha-bugs] [Bug 12631] New: Inventory: fix "wrong place" and "item not scanned" statuses

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12631

            Bug ID: 12631
           Summary: Inventory: fix "wrong place" and "item not scanned"
                    statuses
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P5 - low
         Component: Tools
          Assignee: gmcharlt at gmail.com
          Reporter: matthias.meusburger at biblibre.com
        QA Contact: testopia at bugs.koha-community.org

What's wrong:
-------------

The current behavior in the inventory tool for item statuses is wrong on two
points:

1) "item not scanned" status: when calling GetItemsForInventory, datelastseen
should be used when "compare barcodes list to result" is checked.

Otherwise, when loading multiple barcodes files for the same inventory, many
items will be marked as "item not scanned" when loading the last barcode file
("compare barcodes list to result" checked) even though they were scanned.


2) "wrong place" status: when searching for wrongly placed items, we should
only check for the location (callnumbers, location and branch). To fix this, A
new call to GetItemsForInventory has been made with location filters only.

Otherwise, any item with a different itemtype for instance will be marked as
wrong place even if the location is correct.



Test plan:
----------

Providing a strict and precise test plan is quite hard, as it is strongly
dependant of your database, but you can check as follow:

"item not scanned" status:

1) Split a barcode file in two.

2) Load the first barcode file without checking "compare barcodes list to
result".

3) Load the second barcode file with "compare barcodes list to result" checked.

4) Check in the csv report that you have a lot of scanned items with the "item
not scanned" status.

Then apply the patch, do the same, and check that the false "item not scanned"
statuses are gone.


"wrong place" status:

1) Load a barcode file with barcode matching an item that has a correct
location, but a different itemtype than what you're looking for.

2) Check in the results that this item will be marked as "change item status"
and "wrong place".

Then apply the patch, do the same, and check that the only status for this item
is "change item status", which is correct.



Remarks:
--------

I'm expecting concerns about the fact that GetItemsForInventory is now called
twice instead of once, but keep in mind that inventory is a quite rare
operation.

Also, I did not manage to find a more efficient way to fix this wrong behavior,
suggestions welcome.

-- 
You are receiving this mail because:
You are watching all bug changes.