Attachment 80932 Details for Bug 21476 – Bug 21476: Incorrect filter prevents HTML5 media from playing in the OPAC

[patch] Bug 21476: Incorrect filter prevents HTML5 media from playing in the OPAC

Bug-21476-Incorrect-filter-prevents-HTML5-media-fr.patch (text/plain), 1.98 KB, created by Owen Leonard on 2018-10-19 12:39:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2018-10-19 12:39:04 UTC

Size: 1.98 KB

patch

obsolete

>From 1dad23fdac0485e7d22faa0fb97b3767fe5ad507 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 19 Oct 2018 12:36:12 +0000
>Subject: [PATCH] Bug 21476: Incorrect filter prevents HTML5 media from playing
> in the OPAC
>
>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.
>---
> 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 c451d8d..f13ffde 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 _ ' src="' _  HTML5MediaSet.srcblock _'"' _ HTML5MediaSet.typeblock | $raw %] />
>                                     <span>[[% HTML5MediaParent | html %] tag not supported by your browser.]</span>
>                                   </[% HTML5MediaParent | html %]>
>                                 </p>
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

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