Attachment 81780 Details for Bug 21476 – Bug 21476: Fix HTML5 media from playing in the OPAC - incorrect filters

[patch] Bug 21476: Fix HTML5 media from playing in the OPAC - incorrect filters

Bug-21476-Fix-HTML5-media-from-playing-in-the-OPAC.patch (text/plain), 2.21 KB, created by Jonathan Druart on 2018-10-31 20:25:48 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2018-10-31 20:25:48 UTC

Size: 2.21 KB

patch

obsolete

>From cff85173f851b399b8bf4c645d816279059c3c95 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Tue, 23 Oct 2018 12:30:25 +0000
>Subject: [PATCH] Bug 21476: Fix HTML5 media from playing in the OPAC -
> incorrect filters
>
>This patch makes a correction to a template filter to prevent
>incorrectly-encoded HTML in media source links which were preventing
>media from embedding correctly in the OPAC.
>
>To test, apply the patch and enable the HTML5MediaEnabled and
>HTML5MediaYouTube system preferences.
>
>- Edit a record to include links to videos in 856$u. For example:
>  - https://www.youtube.com/watch?v=grTwH6Evdfc
>    - https://archive.org/download/POPEYEMeetsSindbadTheSailor1936/POPEYE%20meets%20Sindbad%20The%20Sailor%20%281936%29.ogv
>
>    View the record in the OPAC and confirm that the videos display in
>    the
>    "Play media" tab and that they play correctly.
>
>Tested and works as described, also matches the intranet code.
>
>Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>index 3bfa6f4ede..3896a73d6b 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>@@ -1005,7 +1005,7 @@
>                                 <p>
>                                   [% SET ctrl_preload = ' controls preload=none' #translatability %]
>                                   <[% HTML5MediaParent _ ctrl_preload | html %] >
>-                                    <[% HTML5MediaSet.child _ ' src="' _  HTML5MediaSet.srcblock _'"' _ HTML5MediaSet.typeblock | html %] />
>+                                    <[% HTML5MediaSet.child | html %] src="[% HTML5MediaSet.srcblock | html %]" [% HTML5MediaSet.typeblock | html %] />
>                                     <span>[[% HTML5MediaParent | html %] tag not supported by your browser.]</span>
>                                   </[% HTML5MediaParent | html %]>
>                                 </p>
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 21476: 80932 | 81013 | 81047 | 81713 | 81714 | 81721 | 81780