Bug 8877 - MARC21 field 511 missing from XSLT
Summary: MARC21 field 511 missing from XSLT
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Mirko Tietgen
QA Contact: Katrin Fischer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-04 20:56 UTC by Mirko Tietgen
Modified: 2013-05-23 06:23 UTC (History)
2 users (show)

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


Attachments
Bug 8877 MARC21 field 511 missing from XSLT (1.39 KB, patch)
2012-10-04 21:10 UTC, Mirko Tietgen
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 8877 [ENH] MARC21 field 511 missing from XSLT (1.49 KB, patch)
2012-10-05 12:43 UTC, Marc Véron
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 8877 [ENH] MARC21 field 511 missing from XSLT (1.69 KB, patch)
2012-12-28 22:28 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Mirko Tietgen 2012-10-04 20:56:16 UTC
There is no MARC21 field 511 (Participant or Performer Note) in the XSLT for OPAC detail.
Comment 1 Mirko Tietgen 2012-10-04 21:10:09 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2012-10-05 12:43:18 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2012-11-26 07:32:36 UTC
Hi Mirko, 
can you please provide a follow up for the staff XSLT?
Comment 4 Katrin Fischer 2012-11-26 07:40:59 UTC
Hi Mirko,

while testing I found 2 other things that concern me:

1) If you have multiple 511, each will show on a separate line and the label will be repeated:

Cast: Tietgen, Mirko
Cast: Fischer, Katrin

I think that's not consistent with how other repeatable fields display, could you make it so the label appears only once?

2) The display of a label depends on the first indicator set to 1. I see that this is how it's documented in the MARC21 standard (http://www.loc.gov/marc/bibliographic/bd511.html), but it looks a bit weird in the OPAC. 

Maybe we could display another label if the indicator is not set?
Comment 5 Mirko Tietgen 2012-12-07 16:00:00 UTC
Katrin,

I think both things you talk about are done exactly as described by the LOC. Please check the examples at http://www.loc.gov/marc/bibliographic/bd511.html and also have a look at the field in Koha cataloguing. Field 511 (in MARC and in Koha) is designed to take a list of names per field. The field is repeatable, the subfield a is not. No display constant generation is necessary to have several 511 fields for different lists, like 'voices', 'presenter' etc. in the LOC examples, where these descriptions are part of the field value.

It would be nicer to have a subfield for display constant generation, but that is not how MARC works for this field. And who are we to doubt MARC?
Comment 6 Mirko Tietgen 2012-12-13 19:06:10 UTC
Setting this back to needing QA as I disagree with the reasons for failing it and Katrin seemed to possibly agree with my objections on IRC.
Comment 7 Katrin Fischer 2012-12-13 19:50:45 UTC
It's ok - I will take another look :)
Comment 8 Katrin Fischer 2012-12-28 22:28:34 UTC
Created attachment 14323 [details] [review]
[SIGNED-OFF] Bug 8877 [ENH] MARC21 field 511 missing from XSLT

Adds MARC21 field 511 to the OPAC XSLT file.

Signed-off-by: Marc Veron <veron@veron.ch>

Contents of field 511 appears as expected.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Patch changes OPAC XSLT display,
511 is shown as defined by the MARC21 standard.
If 1st indicator is 1, the label cast is shown in front of the 511 field.
If 1st indicator is 0 or empty, no label is shown.
Comment 9 Katrin Fischer 2012-12-28 22:29:41 UTC
Mirko, you have persuaded me ;)

Could you please provide a follow-up to the intranet display to make it match with OPAC display?
Comment 10 Jared Camins-Esakov 2012-12-29 01:22:29 UTC
This patch has been pushed to master.