Summary: | 655 display in XSLT | ||
---|---|---|---|
Product: | Koha | Reporter: | Nicole C. Engard <nengard> |
Component: | Templates | Assignee: | Winona Salesky <wsalesky> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | gwilliams, irma, mtj, nick, nicole, rgravel, tomascohen, wsalesky |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | |||
Bug Blocks: | 10344, 15047 | ||
Attachments: |
655 display in XSLT
Bug 13433 - 655 display in XSLT [SIGNED OFF] Added 655 display to OPAC and staff full views. [SIGNED OFF] Bug 13433 - 655 display in XSLT [PASSED QA] Bug 13433 - Added 655 display to OPAC and staff full views. [PASSED QA] Bug 13433 - 655 display in XSLT |
Description
Nicole C. Engard
2014-12-10 21:26:05 UTC
Any preferred location in the record for this display? My instinct is to place it either next to or below "Material Type" -Winona (In reply to Nicole C. Engard from comment #0) > 655: Display as Genre/Form (full display): Display corresponding subfields > a, v, x, y, z. This change would bring Koha’s display in line with the now > codified practice of separating about-ness (subject headings) from is-ness > (genre/form headings). > > If you're interested, for more info on that, see: > http://www.loc.gov/catdir/cpso/genre_form_faq.pdf I'm not exactly sure where "Material type" is landing now, however, I will note that Genre/Form headings are closely related to "Subject(s)," and so I think they should be displayed near them. I had originally contemplated them following "Subject(s)" but am open to other suggestions! Thanks so much, Winona! Thanks, I will have them following Subjects. -Winona (In reply to rgravel from comment #2) > I'm not exactly sure where "Material type" is landing now, however, I will > note that Genre/Form headings are closely related to "Subject(s)," and so I > think they should be displayed near them. I had originally contemplated them > following "Subject(s)" but am open to other suggestions! > > Thanks so much, Winona! Created attachment 36918 [details] [review] 655 display in XSLT Test Plan: 1) Apply this patch 2) Ensure you are using the default XSLT setting for the staff and opac search results and record details 3) Find or create a record with MARC tags 655 4) Perform an opac search that would show the record in the search results 5) Note this patch adds field 655 to display. Displays subfields a,v,x,y,z. uses a vertical bar to separate multiple titles. Vertical bar is wrapped in span class="separator" for easy suppression/customization of separator. 6) Repeat steps 4 and 5 for the staff interface Looks good in terms of displaying the correct fields, but two issues: 1 - 655 are now showing in both the Subject and Genre/Form Sections, they should only show as Genre/Form 2 - When displaying as Genre/Form the headings are not linked for searching, I think they should still be clickable. This is just my opinion though, willing to concede ts one. (In reply to Winona Salesky from comment #5) > Test Plan: > 1) Apply this patch > 2) Ensure you are using the default XSLT setting for the staff and opac > search results and record details > 3) Find or create a record with MARC tags 655 > 4) Perform an opac search that would show the record in the search results > 5) Note this patch adds field 655 to display. Displays subfields a,v,x,y,z. > uses a vertical bar to separate multiple titles. Vertical bar is wrapped in > span class="separator" for easy suppression/customization of separator. > 6) Repeat steps 4 and 5 for the staff interface Thanks Nick, I will take a look and fix these two issues. -Winona (In reply to Nick Clemens from comment #6) > Looks good in terms of displaying the correct fields, but two issues: > > 1 - 655 are now showing in both the Subject and Genre/Form Sections, they > should only show as Genre/Form > > 2 - When displaying as Genre/Form the headings are not linked for searching, > I think they should still be clickable. This is just my opinion though, > willing to concede ts one. > > (In reply to Winona Salesky from comment #5) > > Test Plan: > > 1) Apply this patch > > 2) Ensure you are using the default XSLT setting for the staff and opac > > search results and record details > > 3) Find or create a record with MARC tags 655 > > 4) Perform an opac search that would show the record in the search results > > 5) Note this patch adds field 655 to display. Displays subfields a,v,x,y,z. > > uses a vertical bar to separate multiple titles. Vertical bar is wrapped in > > span class="separator" for easy suppression/customization of separator. > > 6) Repeat steps 4 and 5 for the staff interface Created attachment 37937 [details] [review] Bug 13433 - 655 display in XSLT Test Plan: 1) Apply this patch 2) Ensure you are using the default XSLT setting for the staff and opac record details 3) Find or create a record with MARC tags 655 4) Perform an opac search and select records with 655 field that would show the record in the search results 5) Note this patch adds field 655 to display. Displays subfields a,v,x,y,z. uses a vertical bar to separate multiple titles. Vertical bar is wrapped in span class="separator" for easy suppression/customization of separator. 6) Repeat steps 4 and 5 for the staff interface Ideally, they should behave like subject headings and be clickable, too. (In reply to Nick Clemens from comment #6) > 2 - When displaying as Genre/Form the headings are not linked for searching, > I think they should still be clickable. This is just my opinion though, > willing to concede ts one. Created attachment 37986 [details] [review] [SIGNED OFF] Added 655 display to OPAC and staff full views. This patch field 655 to the full displays in the staff and OPAC views. Multiple 655 fields are separated with a span class=separator | . To test: * Search the opac for records with genre/field 655 * Click the title * Make sure the fields display properly * Repeat for a few more titles * Repeat in the Staff Client http://bugs.koha-community.org/show_bug.cgi?id=13433 Signed-off-by: Nick Clemens <nick@quecheelibrary.org> Created attachment 37987 [details] [review] [SIGNED OFF] Bug 13433 - 655 display in XSLT Test Plan: 1) Apply this patch 2) Ensure you are using the default XSLT setting for the staff and opac record details 3) Find or create a record with MARC tags 655 4) Perform an opac search and select records with 655 field that would show the record in the search results 5) Note this patch adds field 655 to display. Displays subfields a,v,x,y,z. uses a vertical bar to separate multiple titles. Vertical bar is wrapped in span class="separator" for easy suppression/customization of separator. 6) Repeat steps 4 and 5 for the staff interface Signed-off-by: Nick Clemens <nick@quecheelibrary.org> Created attachment 38102 [details] [review] [PASSED QA] Bug 13433 - Added 655 display to OPAC and staff full views. This patch field 655 to the full displays in the staff and OPAC views. Multiple 655 fields are separated with a span class=separator | . To test: * Search the opac for records with genre/field 655 * Click the title * Make sure the fields display properly * Repeat for a few more titles * Repeat in the Staff Client http://bugs.koha-community.org/show_bug.cgi?id=13433 Signed-off-by: Nick Clemens <nick@quecheelibrary.org> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Created attachment 38103 [details] [review] [PASSED QA] Bug 13433 - 655 display in XSLT Test Plan: 1) Apply this patch 2) Ensure you are using the default XSLT setting for the staff and opac record details 3) Find or create a record with MARC tags 655 4) Perform an opac search and select records with 655 field that would show the record in the search results 5) Note this patch adds field 655 to display. Displays subfields a,v,x,y,z. uses a vertical bar to separate multiple titles. Vertical bar is wrapped in span class="separator" for easy suppression/customization of separator. 6) Repeat steps 4 and 5 for the staff interface Signed-off-by: Nick Clemens <nick@quecheelibrary.org> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Patch pushed to master. Thanks Winona! |