Bug 13083

Summary: Hard-coded " by " in opac-tags.pl
Product: Koha Reporter: paxed <pasi.kallinen>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: f.demians, katrin.fischer, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 13119, 13121    
Attachments: Bug 13083 - Hard-coded " by " in opac-tags.pl
Bug 13083 - Hard-coded " by " in opac-tags.pl
[PASSED QA] Bug 13083 - Hard-coded " by " in opac-tags.pl

Description paxed 2014-10-15 11:29:29 UTC
($biblio->{author}) and $_->{bib_summary} .= " by " . $biblio->{author};
Comment 1 Owen Leonard 2014-10-21 16:09:58 UTC Comment hidden (obsolete)
Comment 2 Frédéric Demians 2014-10-25 12:18:38 UTC
This is OK for MARC21 but won't be that good for UNIMARC: 

- In MARC21, if you have a title and a subtitle, the title (245$a) ends with
  ':', and so your patch will display correctly separated title and subtile.

- In UNIMARC, puncuation characters are not stored in field, ie the title
  doesn't end with a ':'. The display layer is in charge to add punctuation
  characters. That what UNIMARC XSL is doing.

Am I wrong or do you have already proposed a patch to display tagged biblio
records with XSL?
Comment 3 Chris Cormack 2014-10-26 02:13:15 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2014-10-26 10:23:19 UTC
I agree that the display in UNIMARC is not ideal with this patch, but I still believe it to be an improvement over the hardcoded and untranslatable 'by' that was used before.

Also, looking at some other code samples, the UNIMARC display of title + subtitle seems to be a more general problem - so maybe we can fix them all at once in a later patch. 

The XSLT patch from Owen depends on this one.
Comment 5 Katrin Fischer 2014-10-26 10:27:59 UTC
Created attachment 32770 [details] [review]
[PASSED QA] Bug 13083 - Hard-coded " by " in opac-tags.pl

The OPAC's list of a logged-in user's tagged titles displays title and
author, which for some reason are combined in the script into one
variable. I can't see any reason to do it this way.

This patch modifies the script so that title and author are passed as
separate variables. In addition, subtitle is now passed as well.

To test you must log into the OPAC as a user who has tagged multiple
titles, at least one of which should have an author and at least one
with a subtitle. View the list of tagged titles and confirm that this
information is being displayed correctly.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, passes tests and QA script.
Comment 6 Tomás Cohen Arazi 2014-10-28 13:24:03 UTC
Patch pushed to master.

Thanks Owen!