https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21944

Marcel de Rooy <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|BLOCKED                     |Failed QA

--- Comment #26 from Marcel de Rooy <[email protected]> ---
    ($resfound, $resrec, undef) = C4::Reserves::CheckReserves(
$item->itemnumber, undef, $lookahead ) unless ( $item->withdrawn );
    # if a hold is found and is waiting at another branch, change the priority
back to 1 and trigger the hold (this will trigger a transfer and update the
hold status properly)
    if ( $resfound eq "Waiting" and $branch ne $resrec->{branchcode} ) {
        C4::Reserves::RevertWaitingStatus( { itemnumber => $item->itemnumber }
);
        #If the hold is reverted we need to refetch for the return values
        ($resfound, $resrec, undef) = C4::Reserves::CheckReserves(
$item->itemnumber, undef, $lookahead ) unless ( $item->withdrawn );
    }
    ($resfound, $resrec, undef) = C4::Reserves::CheckReserves(
$item->itemnumber, undef, $lookahead ) unless ( $item->withdrawn );

In short what do we see here:
call CheckReserves
if(  )
    call CheckReserves
endif
call CheckReserves
Three times in a row ? Please explain why you also add the last call. Rebase
problem?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to