| Summary: | Unify patron name display | ||
|---|---|---|---|
| Product: | Koha | Reporter: | paxed <pasi.kallinen> |
| Component: | Templates | Assignee: | Owen Leonard <oleonard> |
| Status: | RESOLVED DUPLICATE | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | marjorie.barry-vila, martin.renvoize, paul.poulain |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Bug Depends on: | |||
| Bug Blocks: | 7680 | ||
*** Bug 7183 has been marked as a duplicate of this bug. *** |
The way patron's name is displayed should be unified into a single place. There's template code in patron-title.inc for it, but it's not used in enough places. Using that would make the patron name display consistent, and if change is needed (eg. for GDPR), it would be easy to change. For example: koha-tmpl/intranet-tmpl/prog/en/includes/cat-toolbar.inc: <li><a href="/cgi-bin/koha/reserve/request.pl?biblionumber=[% biblionumber | uri %]&findborrower=[% holdfor_cardnumber | uri %]">Place hold for [% holdfor_firstname | html %] [% holdfor_surname | html %] ([% holdfor_cardnumber | html %])</a></li> koha-tmpl/intranet-tmpl/prog/en/includes/js_includes.inc: .append( "<a>" + item.surname + ", " + item.firstname + " (" + item.cardnumber + ") <small>" + item.dateofbirth + " " + item.address + " " + item.city + " " + item.zipcode + " " + item.country + "</small></a>" ) koha-tmpl/intranet-tmpl/prog/en/includes/circ-patron-search-results.inc: <td><a href="[% data_url | url %]">[% borrower.surname | html %], [% borrower.firstname | html %]</a></td> koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-print.tt: <title>Summary for [% patron.firstname | html %] [% patron.surname | html %] ([% patron.cardnumber | html %])</title> koha-tmpl/intranet-tmpl/prog/en/modules/members/paycollect.tt: <a href="/cgi-bin/koha/members/pay.pl?borrowernumber=[% patron.borrowernumber | uri %]">Pay fines for [% patron.firstname | html %] [% patron.surname | html %]</a> › and many, many more.