Bug 21742: Fix YouTube variable propagation
authorNick Clemens <nick@bywatersolutions.com>
Wed, 31 Oct 2018 12:18:58 +0000 (12:18 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Fri, 9 Nov 2018 16:11:46 +0000 (16:11 +0000)
commitec7e569d55575f26ba481684ff157c8ec3985002
treeb33a59e728394fa87c9f2c071cb1b312629b3f79
parent776423df10e6237be78420ef63e1a53694cb884d
Bug 21742: Fix YouTube variable propagation

The loop to find all media has an $isyoutube variable. This was declared
outside the loop and never reset. Once a YouTube video was encountered
all subsequent videos were considered as YouTube. This patch reduces the
scope of the variable and resets per loop.

To test:
1 - Enable HTML5Media and HTML5MediaYouTube
2 - Add 856$u to record for a youtube vido and a pdf (sample record
        attached to bug)
3 - View the record, the Play media tab has two video boxes
4 - Play the second, it fails
5 - Apply patch
6 - View record, Play media should include only one video link

Signed-off-by: Devinim <kohadevinim@devinim.com.tr>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit 8ceba54c8dcfd9b598375b8af432c0111ec63d04)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
C4/HTML5Media.pm