| Summary: | MARC21: Display of $d for 7xx and 1xx fields should be optional | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
| Component: | OPAC | Assignee: | Katrin Fischer <katrin.fischer> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | minor | ||
| Priority: | P5 - low | CC: | brendan, f.demians, hector.hecaxmmx, julian.maurice, mtj |
| 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: | ||
| Circulation function: | |||
| Bug Depends on: | |||
| Bug Blocks: | 16343 | ||
| Attachments: |
Bug 15100 - MARC21: Display of $d for 7xx and 1xx fields should be optional
Bug 15100 - MARC21: Display of $d for 7xx and 1xx fields should be optional Bug 15100: MARC21 - make display of "Dates associated with a name" optional [SIGNED-OFF]Bug 15100: MARC21 - make display of "Dates associated with a name" optional Bug 15100: MARC21 - make display of "Dates associated with a name" optional |
||
|
Description
Katrin Fischer
2015-11-01 20:23:15 UTC
Created attachment 44614 [details] [review] Bug 15100 - MARC21: Display of $d for 7xx and 1xx fields should be optional Remove $d - Dates associated with a name for Main entries (1XX) and Added entries (7XX) To test: -Download and stage marc records with $d subfield for 1XX and 7XX or download from this attchment <http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36515> -Reindex zebra -Search whichever or all records -Notice that subfield $d is present in all 1XX and 7XX -Apply patch -Search again and see that the subfield $d is not shown Created attachment 44615 [details] [review] Bug 15100 - MARC21: Display of $d for 7xx and 1xx fields should be optional Remove $d - Dates associated with a name for Main entries (1XX) and Added entries (7XX) To test: -Download and stage marc records with $d subfield for 1XX and 7XX or download from this attchment <http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36515> -Reindex zebra -Search whichever or all records -Notice that subfield $d is present in all 1XX and 7XX in results and detail record page -Apply patch -Search again and see that the subfield $d is not shown in results and detail record page This is not what Katrin is asking for. Some libraries may want to display author dates, some others may not want. Your patch just remove $d. The idea is to display the dates in an identified html <span> in order to allow libraries to hide author dates easily with css. (In reply to Frédéric Demians from comment #3) > This is not what Katrin is asking for. Some libraries may want to display > author dates, some others may not want. Your patch just remove $d. The idea > is to display the dates in an identified html <span> in order to allow > libraries to hide author dates easily with css. Does this need a syspref? or how to control the class to hide or show the span tag? Please explain to me? Exist information about or one bug to take as example Regards > Does this need a syspref? or how to control the class to hide or show the
> span tag?
No syspref. You'd just need to enclose 100/70x date subfield into html <span>
with a class identifying it precisely. For example:
<span class="authordates">1802-1885</span>
You have an example with author $4 subfield.
Created attachment 45072 [details] [review] Bug 15100: MARC21 - make display of "Dates associated with a name" optional With the changes for RDA now more subfields for the fields 100, 110, 111, and 700, 710, 711 are displayed. One of them is $d - Dates associated with a name This patch adds separate treatment of the $d subfield and wraps it in a span with the class "authordates". This makes it possible to hide the information with CSS. To test: - Make sure you have several records with $d catalogued - Check the display in result list and detail page with and without the page. - Check, if there is a difference in display - Check if the dates are correctly wrapped in a span - Try to hide the information, using the following entry in your *UserCSS preferences: .authordates { display: none; } Hi Héctor, thx for taking a look at this! Hope it's ok, I have attached another patch that treats $d separately from the other subfields. Created attachment 45093 [details] [review] [SIGNED-OFF]Bug 15100: MARC21 - make display of "Dates associated with a name" optional With the changes for RDA now more subfields for the fields 100, 110, 111, and 700, 710, 711 are displayed. One of them is $d - Dates associated with a name This patch adds separate treatment of the $d subfield and wraps it in a span with the class "authordates". This makes it possible to hide the information with CSS. To test: - Make sure you have several records with $d catalogued - Check the display in result list and detail page with and without the page. - Check, if there is a difference in display - Check if the dates are correctly wrapped in a span - Try to hide the information, using the following entry in your *UserCSS preferences: .authordates { display: none; } Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised Created attachment 46412 [details] [review] Bug 15100: MARC21 - make display of "Dates associated with a name" optional With the changes for RDA now more subfields for the fields 100, 110, 111, and 700, 710, 711 are displayed. One of them is $d - Dates associated with a name This patch adds separate treatment of the $d subfield and wraps it in a span with the class "authordates". This makes it possible to hide the information with CSS. To test: - Make sure you have several records with $d catalogued - Check the display in result list and detail page with and without the page. - Check, if there is a difference in display - Check if the dates are correctly wrapped in a span - Try to hide the information, using the following entry in your *UserCSS preferences: .authordates { display: none; } Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as advertised Signed-off-by: Jesse Weaver <jweaver@bywatersolutions.com> Pushed to Master - Should be in the May 2016 release. Thanks! Patch pushed to 3.22.x, will be in 3.22.3 |