Bug 21742: Fix YouTube variable propagation
authorNick Clemens <nick@bywatersolutions.com>
Wed, 31 Oct 2018 12:18:58 +0000 (12:18 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Thu, 1 Nov 2018 18:59:33 +0000 (18:59 +0000)
commit8ceba54c8dcfd9b598375b8af432c0111ec63d04
tree48f4a167aef1ceff99408b834b28c80c6f0d6886
parent8f6bd9415096924e2648a1a0a5866733f6839760
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>
C4/HTML5Media.pm