From 10cbf64b24be481d3c668c0d7015dfec32d485b5 Mon Sep 17 00:00:00 2001 From: Mirko Tietgen Date: Thu, 3 Mar 2016 07:58:55 +0100 Subject: [PATCH] Bug 14168: (followup) require WWW::YouTube::Download only when syspref enabled This patch fixes the problem of missing WWW::YouTube::Download breaking record detail views. Instead of a general 'use', the module is only required if the related syspref is enabled. Signed-off-by: Jesse Weaver Signed-off-by: Brendan A Gallagher --- C4/HTML5Media.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/C4/HTML5Media.pm b/C4/HTML5Media.pm index 7c2b174d92..b4389332a7 100644 --- a/C4/HTML5Media.pm +++ b/C4/HTML5Media.pm @@ -23,7 +23,6 @@ use warnings; use C4::Context; use MARC::Field; use Koha::Upload; -use WWW::YouTube::Download qw(playback_url); =head1 HTML5Media @@ -101,6 +100,8 @@ sub gethtml5media { $HTML5Media{srcblock} = $HTML5Media_field->subfield('u'); if (grep /youtube/, $HTML5Media_field->subfield('u') ) { # TODO is there an official YT URL shortener? Can we use that too? if ($HTML5MediaYouTube == 1) { + require WWW::YouTube::Download; + import WWW::YouTube::Download qw(playback_url); my $youtube = WWW::YouTube::Download->new; $HTML5Media{srcblock} = $youtube->playback_url( $HTML5Media_field->subfield('u'), { -- 2.39.5