[Koha-bugs] [Bug 13915] Inventory tool no longer hides items with date last seen greater than the inventory date

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Apr 6 00:23:23 CEST 2015


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

Katrin Fischer <katrin.fischer at bsz-bw.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #5 from Katrin Fischer <katrin.fischer at bsz-bw.de> ---
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.

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


More information about the Koha-bugs mailing list