Bug 24185: Make holds page faster - Improved "if"

`$can_item_be_reserved eq 'OK'` moved in `&&` before `IsAvailableForItemLevelRequest`
to cut away with static known values before calling to more resource consuming subroutine.

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

diff --git a/reserve/request.pl b/reserve/request.pl
index 5864b3d..f69f706 100755 (executable)
--- a/reserve/request.pl
+++ b/reserve/request.pl
@@ -557,8 +557,8 @@ foreach my $biblionumber (@biblionumbers) {
                 if (
                        !$item->{cantreserve}
                     && !$exceeded_maxreserves
-                    && IsAvailableForItemLevelRequest($item_object, $patron)
                     && $can_item_be_reserved eq 'OK'
+                    && IsAvailableForItemLevelRequest($item_object, $patron)
                   )
                 {
                     $item->{available} = 1;