This line: <td class="hq-patron"><p><a href="/cgi-bin/koha/circ/circulation.pl?findborrower=[% itemsloo.cardnumber %]#reserves"> Should be: <td class="hq-patron"><p><a href="cgi-bin/koha/members/moremember.pl?borrowernumber=[% itemsloo.borrowernumber %]#reserves"> or something similar
I don't understand what is the problem here.
It seems like we add a call to GetMember to get the borrowernumber from the cardnumber when we could just as easily provide the borrowernumber directly. Maybe it doesn't have an impact, just feels like it would be more clean the other way. (In reply to Jonathan Druart from comment #1) > I don't understand what is the problem here
circ/circulation.pl expects a cardnumber: 246 if ($findborrower) { 247 my $borrower = C4::Members::GetMember( cardnumber => $findborrower ); 248 if ( $borrower ) { 249 $borrowernumber = $borrower->{borrowernumber}; 250 } else { If not found, the script will try and search the $findborrower using the C4::Utils::DataTables::Members::search subroutine. We will have to change all occurrences if we want to change this behavior, not only view_holdsqueue.tt
It look slike this has been resovled, maybe with another patch set: /cgi-bin/koha/circ/circulation.pl?borrowernumber=51#reserves