Bug 28510: Remove unnecessary conditional
authorKyle M Hall <kyle@bywatersolutions.com>
Thu, 3 Jun 2021 11:29:52 +0000 (07:29 -0400)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 28 Sep 2021 13:12:45 +0000 (15:12 +0200)
commitd284735d05f67226feba1b4b9e7bf17259eaabaf
tree3d4ce1f56a80b0f4246551723b960e5935b55927
parentba23348db60fdf026723deef6e2907f6fc8093e6
Bug 28510: Remove unnecessary conditional

It makes sense to use items by branch to get the list of branches, as it
already tells use which branches have available items. We could use
branches to pull from instead, but all we would accomplish is added
extra ununsed loop iterations. We already know that any additional
branches in the branches to pull loop have no items to fill holds.
If they did, they would be in the items_by_branch hash.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
C4/HoldsQueue.pm