Summary: | 490 not displaying without indicator 1 | ||
---|---|---|---|
Product: | Koha | Reporter: | Jared Camins-Esakov <jcamins> |
Component: | OPAC | Assignee: | Jared Camins-Esakov <jcamins> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | major | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, katrin.fischer, koha.sekjal |
Version: | 3.6 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: |
Bug 7007: 490 series statements should appear when not traced
[SIGNED-OFF] Bug 7007: 490 series statements should appear when not traced |
Description
Jared Camins-Esakov
2011-10-11 19:10:27 UTC
Created attachment 5843 [details] [review] Bug 7007: 490 series statements should appear when not traced Although the MARC21 standard requires libraries to set the first indicator to '0' when a 490 field is untraced, many libraries leave it blank. Previously, this resulted in a blank line with the label "Series:" but no actual series information. This patch fixes that by showing untraced series headings for 490s whenever the first indicator is not set to '1' Created attachment 5865 [details] [review] [SIGNED-OFF] Bug 7007: 490 series statements should appear when not traced Although the MARC21 standard requires libraries to set the first indicator to '0' when a 490 field is untraced, many libraries leave it blank. Previously, this resulted in a blank line with the label "Series:" but no actual series information. This patch fixes that by showing untraced series headings for 490s whenever the first indicator is not set to '1' Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Checked that series shows correctly when first indicator is 0, 1 or not set. Since MARC21 defines first indicator as only 0 or 1, we don't need to worry about other possibilities creeping in. This patch is .xsl only, and just redefines the logic from "is 0" to "is not 1", which will be a little more lenient on sloppy cataloging. Marking Passed QA Pushed, please test Closing, since this now works. If anyone objects to the patch author closing the bug, please feel free to test independently (and reopen if I was mistaken). |