Bug 16171 - Show many media (856) in html5media tab
Summary: Show many media (856) in html5media tab
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Alex Arnaud
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-03-30 14:45 UTC by Alex Arnaud
Modified: 2016-12-05 21:25 UTC (History)
7 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 16171 - Show many media in html5media tabs (3.20 KB, patch)
2016-03-30 14:49 UTC, Alex Arnaud
Details | Diff | Splinter Review
Bug 16171 - Show many media in html5media tabs (3.25 KB, patch)
2016-03-31 14:31 UTC, Mirko Tietgen
Details | Diff | Splinter Review
[PASSED QA] Bug 16171 - Show many media in html5media tabs (3.32 KB, patch)
2016-04-03 21:11 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Arnaud 2016-03-30 14:45:54 UTC
If you have many 856 in a biblio, only one is displayed in the media tab (OPAC and staff). This patch allows to show all 856 in this tab.
Comment 1 Alex Arnaud 2016-03-30 14:49:36 UTC Comment hidden (obsolete)
Comment 2 Liz Rea 2016-03-31 02:11:35 UTC
Hi Alex,

May we have a test plan for this?

Cheers,
Liz
Comment 3 Mirko Tietgen 2016-03-31 14:31:50 UTC Comment hidden (obsolete)
Comment 4 Mirko Tietgen 2016-03-31 14:35:33 UTC
I have been meaning to fix this, but nobody complained, so it was not high on the list. Thanks Alex.

To test:
- Enable HTML5MediaEnabled
- Catalogue a record with two 856 fields, both having a video link in subfield u (and mime type in subfield q)

Without the patch, you get only one video in the detail view in OPAC and staff client. With patch applied, you see both videos.
Comment 5 Alex Arnaud 2016-03-31 15:26:14 UTC
Thanks Mirko for testing and test plan
Comment 6 Katrin Fischer 2016-04-03 21:11:27 UTC
Created attachment 49830 [details] [review]
[PASSED QA] Bug 16171 - Show many media in html5media tabs

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Brendan Gallagher 2016-04-06 22:12:05 UTC
Pushed to Master - Should be in the May 2016 Release.  Thanks!
Comment 8 Julian Maurice 2016-04-07 08:00:04 UTC
Patch pushed to 3.22.x, will be in 3.22.6
Comment 9 Frédéric Demians 2016-04-27 15:36:07 UTC
Pushed to 3.22.x, will be in 3.20.11.