Bug 27028 - Manual transfers: patron name display in hold warning should be standardized
Summary: Manual transfers: patron name display in hold warning should be standardized
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-11-14 12:32 UTC by Victor Grousset/tuxayo
Modified: 2024-04-24 14:59 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
screenshot-of-the-issue (15.79 KB, image/png)
2020-11-14 12:32 UTC, Victor Grousset/tuxayo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Victor Grousset/tuxayo 2020-11-14 12:32:06 UTC
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 %]
Comment 1 Katrin Fischer 2023-01-26 23:33:19 UTC
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.
Comment 2 Owen Leonard 2024-04-24 14:59:25 UTC
I'm working on a patch for Bug 25682 which I think will solve this issue.