In previous versions of Koha, selecting an inventory date would filter out items that had been seen since the "Inventory date". Bug 12631 changed the inventory tool so that this behavior only occurs when a barcodes list file is uploaded for comparison.
Created attachment 37268 [details] [review] Bug 13915 - Inventory tool no longer hides items with date last seen greater than the inventory date In previous versions of Koha, selecting an inventory date would filter out items that had been seen since the "Inventory date". Bug 12631 changed the inventory tool so that this behavior only occurs when a barcodes list file is uploaded for comparison. Test Plan: 1) Apply this patch 2) Run the inventory tool with no set parameters 3) Mark the first item as seen 4) Rerun the tool with only a date last seen parameter of yesterday 5) Note the item you marked as seen is no longer on the list
Correction, bug 12631 is not the culprit in causing this regression. It was just the last patch to change this line of code.
Created attachment 37333 [details] [review] [Signed-off] Bug 13915 - Inventory tool no longer hides items with date last seen greater than the inventory date In previous versions of Koha, selecting an inventory date would filter out items that had been seen since the "Inventory date". Bug 12631 changed the inventory tool so that this behavior only occurs when a barcodes list file is uploaded for comparison. Test Plan: 1) Apply this patch 2) Run the inventory tool with no set parameters 3) Mark the first item as seen 4) Rerun the tool with only a date last seen parameter of yesterday 5) Note the item you marked as seen is no longer on the list Followed test plan. Works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
The search option is named 'inventory date' - If I understood correctly, this could be more clear as 'inventory date before'?
Bit confused as the following lines were introduced by bug 12631 and are undone here: Patch from 12631: + my $paramdatelastseen = ($compareinv2barcd) ? $datelastseen : ''; + ($inventorylist, $totalrecords) = GetItemsForInventory($minlocation, $maxlocation, $location, $itemtype, $ignoreissued, $paramdatelastseen, $branchcode, $branch, 0, undef, $staton); Patch attached to this bug: - my $paramdatelastseen = ($compareinv2barcd) ? $datelastseen : ''; - ($inventorylist, $totalrecords) = GetItemsForInventory($minlocation, $maxlocation, $location, $itemtype, $ignoreissued, $paramdatelastseen, $branchcode, $branch, 0, undef, $staton); + ($inventorylist, $totalrecords) = GetItemsForInventory($minlocation, $maxlocation, $location, $itemtype, $ignoreissued, $datelastseen, $branchcode, $branch, 0, undef, $staton); Are we sure there is no regression on bug 12631 by this change? I had problems testing 12631 when I tried, so not sure. Can you please take a look and explain? The test plan works for me - I'd just like to be careful.
Seems this could be closed in favor of bugs 12913/14399.