Created attachment 113641 [details] screenshot-of-the-issue == Test plan == 1. Place a hold (record level in my example) 2. Try a transfer 3. Get a hold warning 4. It doesn't display (at least, not checked everything) the patron's name and their branch. See screenshot-of-the-issue.png The code confirms that it should: Reserve found for [% name | html %] (<a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber | uri %]">[% borrowernumber | html %]</a>). [...] [% IF ( reserved ) %]Set reserve to waiting and transfer book to [% branchname | html %]: [% END %]
The display could be a little nicer, but it works now: Hold found for koha (42) (51). 42 = cardnumber 51 = linked borrowernumber ... and the heading still reads Reserve found, this should be Hold found. I am updating the bug description a little, we should use a standardized patron name display here, that also respects visibility settings etc.
I'm working on a patch for Bug 25682 which I think will solve this issue.