@@ -, +, @@ A, B, C, item branches and staff branch are the same. Priorities are: A = Waiting, B = 1, C = 2 Koha asks: Item X has been waiting for patron A... Revert waiting status Confirm. Hold list shows: A = 1, C = 1 Database says: A = 1, C = 3 --- C4/Reserves.pm | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) --- a/C4/Reserves.pm +++ a/C4/Reserves.pm @@ -1663,6 +1663,7 @@ when _FixPriority calls itself. sub _FixPriority { my ( $params ) = @_; + warn "_FixPriority: " . Data::Dumper::Dumper( $params ); my $reserve_id = $params->{reserve_id}; my $rank = $params->{rank} // ''; my $ignoreSetLowestRank = $params->{ignoreSetLowestRank}; @@ -2113,7 +2114,7 @@ sub MergeHolds { =head2 RevertWaitingStatus - $success = RevertWaitingStatus({ itemnumber => $itemnumber }); + RevertWaitingStatus({ itemnumber => $itemnumber }); Reverts a 'waiting' hold back to a regular hold with a priority of 1. @@ -2168,7 +2169,8 @@ sub RevertWaitingStatus { reserve_id = ? "; $sth = $dbh->prepare( $query ); - return $sth->execute( $reserve->{'reserve_id'} ); + $sth->execute( $reserve->{'reserve_id'} ); + _FixPriority( { biblionumber => $reserve->{biblionumber} } ); } =head2 GetReserveId --