Bug 10987

Summary: Too many 'see also' and 'usedfor/see from' labels occurences
Product: Koha Reporter: Tomás Cohen Arazi <tomascohen>
Component: TemplatesAssignee: Tomás Cohen Arazi <tomascohen>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: enhancement    
Priority: P5 - low CC: bgkriegel, gitbot, gmcharlt, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: Sponsored Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 10691    
Bug Blocks:    
Attachments: Bug 10987: Too many 'see also' and 'see from' labels occurences
[SIGNED-OFF] Bug 10987: Too many 'see also' and 'see from' labels occurences
Bug 10987: Too many 'see also' and 'see from' labels occurences
Bug 10987: Too many 'see also' and 'see from' labels occurences (rebased+bootstrap)
Bug 10987: Too many 'see also' and 'see from' labels occurences (rebased+bootstrap)

Description Tomás Cohen Arazi 2013-10-02 18:51:41 UTC
If you do an authority search and the results contain authorities with more than one 4xx or 5xx the 'see also' and 'used for/see from' labels are unnecesarilly repeated, affecting readability.
Comment 1 Tomás Cohen Arazi 2013-10-02 18:53:03 UTC Comment hidden (obsolete)
Comment 2 I'm just a bot 2013-10-03 07:02:56 UTC
Patch applied cleanly, go forth and signoff
Comment 3 Bernardo Gonzalez Kriegel 2013-10-09 12:12:51 UTC Comment hidden (obsolete)
Comment 4 Jonathan Druart 2013-11-07 13:41:12 UTC
Tomás, the patch does not apply anymore, could you rebase it please?

fatal: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc).
Comment 5 Jonathan Druart 2013-11-07 13:41:52 UTC
I forgot the dependency! Sorry.
Comment 6 Jonathan Druart 2013-11-07 16:11:40 UTC
Tested with bug 10691. Works great.
Marked as Passed QA.
Comment 7 Jonathan Druart 2013-11-07 16:12:42 UTC Comment hidden (obsolete)
Comment 8 Tomás Cohen Arazi 2013-12-11 15:48:19 UTC Comment hidden (obsolete)
Comment 9 Jonathan Druart 2014-01-16 15:57:31 UTC
Tested on top of bug 10691. All works great. Maybe it would be better to split patches for the OPAC.

Marked as Passed QA.
Comment 10 Jonathan Druart 2014-01-16 15:59:40 UTC
Created attachment 24464 [details] [review]
Bug 10987: Too many 'see also' and 'see from' labels occurences (rebased+bootstrap)

If an authority record has more than one 4xx$a (or 5xx$a) entries the
corresponding labels 'see also' and 'used for/see from' are repeated.

To test:
- Have authority records with more than one entry on the 4xx (or 5xx) fields.
- Do a search, check "see also:" (or 'used for/see from') appear more than once.
- Apply the patch
- Reload and check it looks nicer :-D
- Repeat for: staff auth search, opac using ccsr, opac using prog
- Signoff

Regards
To+

Sponsored-by: Universidad Nacional de Cordoba

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>

Work as decribed. No koha-qa errors
Nice view on staff and both opac

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 11 Galen Charlton 2014-01-16 23:10:35 UTC
Pushed to master, along with a follow-up that repairs breakage in the display of an authority record that has only one 4XX, or one 5XX.

Thanks, Tomás!