---- Reported by dschust1@gmail.com 2009-01-27 06:56:23 ---- Currently this information only displays on the Detail screen - Since several borrowers don't use their legal name, but we load them that way schools are putting in the "Other name" information. It would be helpful if that would display right under the name or behind it on the CKO screen. We are having to put it in the "notes" field to show on the CKO screen. example - Charolett Elam actually goes by Mary - but you don't know that on the CKO screen. ---- Additional Comments From mjr@ttllp.co.uk 2009-05-11 14:33:27 ---- Does CKO mean checkout? ---- Additional Comments From gmcharlt@gmail.com 2009-05-11 14:37:14 ---- (In reply to comment #1) > Does CKO mean checkout? > Yes - usage from a couple legacy ILSs. ---- Additional Comments From dschust1@gmail.com 2010-02-11 14:25:43 ---- Created an attachment patch for this bug - modifies circulation.pl and circulation.tmpl --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:04 UTC --- This bug was previously known as _bug_ 2930 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=2930 Imported an attachment (id=973) Actual time not defined. Setting to 0.0 CC member mjr@ttllp.co.uk does not have an account here The original submitter of attachment 973 [details] [review] is unknown. Reassigning to the person who moved it here: chris@bigballofwax.co.nz.
The attached patch makes a lot of changes not related to the bug description. The display name of the patron is generated in an include file now: [% IF category_type == 'I' %] [% surname %] [% IF othernames %] ([% othernames %]) [% END %] [% ELSE %] [% firstname %] [% surname %] [% END %] ([% cardnumber %]) So for an "organisation" the othername will already be displyed. How should the display for othername look like? Can we agree on something here?
(In reply to comment #1) > So for an "organisation" the othername will already be displyed. How should > the display for othername look like? Can we agree on something here? I suspect it should be: [% firstname %] [% IF othernames %] ([% othernames %]) [% END %] [% surname %] Does anyone see a problem with that? It is one of the suggested forms in http://dictionary2.classic.reference.com/writing/styleguide/personal.html
Created attachment 11783 [details] [review] Bug 2930 - Put other name on checkout screen This patch adds other name to the include which is used to display patron names and adds this include in several places where it can be used in place of direct output. The patron-title include has been modified to handle the two possible variable scopes throughout patron-related templates. This is a hack similar to having both circ-menu.tt and circ-menu.inc, but keeping both in the same file. Changes to some scripts were necessary to make the othernames variable available to the include. This patch also corrects some <title> tags and fixes some incorrect capitalization (see Bug 2780). To test, view each of the pages affected by these template changes and confirm that the patron's name displays correctly, including their "other name."
This patch introduces a side-effect in the member results page. Links are not displayed. Marked as failed qa
Created attachment 12021 [details] [review] Bug 2930 [REVISED] Put other name on checkout screen This patch adds other name to the include which is used to display patron names and adds this include in several places where it can be used in place of direct output. The patron-title include has been modified to handle the two possible variable scopes throughout patron-related templates. This is a hack similar to having both circ-menu.tt and circ-menu.inc, but keeping both in the same file. Changes to some scripts were necessary to make the othernames variable available to the include. This patch also corrects some <title> tags and fixes some incorrect capitalization (see Bug 2780). To test, view each of the pages affected by these template changes and confirm that the patron's name displays correctly, including their "other name."
Created attachment 12321 [details] [review] [SIGNED-OFF] Bug 2930 [REVISED] Put other name on checkout screen This patch adds other name to the include which is used to display patron names and adds this include in several places where it can be used in place of direct output. The patron-title include has been modified to handle the two possible variable scopes throughout patron-related templates. This is a hack similar to having both circ-menu.tt and circ-menu.inc, but keeping both in the same file. Changes to some scripts were necessary to make the othernames variable available to the include. This patch also corrects some <title> tags and fixes some incorrect capitalization (see Bug 2780). To test, view each of the pages affected by these template changes and confirm that the patron's name displays correctly, including their "other name." Signed-off-by: Marc Veron <veron@veron.ch> Followed instructions above, works as expected.
Hi Owen, Is it normal that your last patch uses the include file in only 1 tt file? I think your first proposed patch was more interesting, it used the include in several places instead of just one. What problem did you encounter?
QA comment * small patch, template only * koha-qa.pl OK passed QA
Patch pushed to master