Summary: | Manual transfers: patron name display in hold warning should be standardized | ||
---|---|---|---|
Product: | Koha | Reporter: | Victor Grousset/tuxayo <victor> |
Component: | Circulation | Assignee: | Bugs List <koha-bugs> |
Status: | NEW --- | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | gmcharlt, kyle.m.hall |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25682 | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: | screenshot-of-the-issue |
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. |
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 %]