Bug 10452: (follow-up) remove duplicate check of damaged status
authorGalen Charlton <gmc@esilibrary.com>
Mon, 10 Mar 2014 18:13:16 +0000 (18:13 +0000)
committerGalen Charlton <gmc@esilibrary.com>
Mon, 10 Mar 2014 18:15:44 +0000 (18:15 +0000)
commit67d3985929d70b67e64d5f0c275109b109065d2d
tree6fec08c53213199376c6a19ff3d783dd824104b3
parent2e6d44bd3200d0dceef59f04a79a27dcadceab1d
Bug 10452: (follow-up) remove duplicate check of damaged status

While reviewing the main patch for this bug to verify that the
holds queue routines and C4::Reserves had the same conception of
when a damaged item could fill a hold request, I noticed that
GetItemsAvailableToFillHoldRequestsForBib() duplicated the code
for adding an SQL clause to filter out damaged items.  This patch
removes the duplication and improves the POD for that routine.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
C4/HoldsQueue.pm