Summary: | Show same information for format, material type etc. on the detail page as in the result list | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | MARC Bibliographic data support | Assignee: | Lucas Gass (lukeg) <lucas> |
Status: | Failed QA --- | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | lucas |
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: | |||
Attachments: |
Bug 20161: add format, material type, etc to staff details page
Bug 20161: add format, material type, etc to staff/OPAC details page |
Description
Katrin Fischer
2018-02-08 10:10:43 UTC
Created attachment 131294 [details] [review] Bug 20161: add format, material type, etc to staff details page 1. Make sure DisplayIconsXSLT is on. 2. Look at the results page from the staff client. 3. Notice the results_material_type contains 'Material type', 'Format', 'Literary form', 'Type of visual material' 4. Look at the staff detail page and notice that only 'Material type' will show. 5. Apply patch and restart everything 6. Now look at the detail page and it should match what you see on the results page. 7. Turn off DisplayIconsXSLT 8. Make sure nothing on the results_material_type now displays. This first patch deals with the staff client. A second one to deal with the OPAC will be incoming ASAP. Created attachment 131303 [details] [review] Bug 20161: add format, material type, etc to staff/OPAC details page For staff side: 1. Make sure DisplayIconsXSLT is on. 2. Look at the results page from the staff client. 3. Notice the results_material_type contains 'Material type', 'Format', 'Literary form', 'Type of visual material' 4. Look at the staff detail page and notice that only 'Material type' will show. 5. Apply patch and restart everything 6. Now look at the detail page and it should match what you see on the results page. 7. Turn off DisplayIconsXSLT 8. Make sure nothing on the results_material_type now displays. For OPAC: 1. Make sure DisplayOPACiconsXSLT is on. 2. Look at the results page from the OPAC. 3. Notice the results_material_type contains 'Material type', 'Format', 'Literary form', 'Type of visual material' 4. Look at the OPAC detail page and notice that only 'Material type' will show. 5. Apply patch and restart everything 6. Now look at the detail page and it should match what you see on the results page. 7. Turn off DisplayOPACiconsXSLT 8. Make sure nothing on the results_material_type now displays. Hi Lucas, if we want to display the same information everywhere, maybe a new include file (like for the languages) would be a good idea? It would save some copy and paste and make it easier to keep things aligned in the future. If we make it a template, it would also be easy to pass on some specifics like different class names. What do you think? (In reply to Katrin Fischer from comment #4) > Hi Lucas, > > if we want to display the same information everywhere, maybe a new include > file (like for the languages) would be a good idea? It would save some copy > and paste and make it easier to keep things aligned in the future. > > If we make it a template, it would also be easy to pass on some specifics > like different class names. > > What do you think? I was curious about this, it is A LOT of repeated code. I think it's a good idea and I'll mark this one as FQA now because of that. |