Bug 10452: (follow-up) remove duplicate check of damaged status

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>
(cherry picked from commit 67d3985929d70b67e64d5f0c275109b109065d2d)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>