Attachment 85410 Details for Bug 22378 – Bug 22378: Fix audio alerts on SCO

[patch] Bug 22378: Fix audio alerts on SCO

Bug-22378-Fix-audio-alerts-on-SCO.patch (text/plain), 1.55 KB, created by Nick Clemens (kidclamp) on 2019-02-20 20:01:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2019-02-20 20:01:57 UTC

Size: 1.55 KB

patch

obsolete

>From 562faf4b2e019b20cea9633090ccf411bff36f1b Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Wed, 20 Feb 2019 20:00:01 +0000
>Subject: [PATCH] Bug 22378: Fix audio alerts on SCO
>
>Note: The SCO only supports sounds with a full URL, not KOha built in sounds
>
>To test:
>1 - Enable audio alerts
>2 -  Enable SCO
>3 - Note JS error on loading the alerts
>4 - Apply patch
>5 - No more error
>6 - Test with bug 18251
>---
> koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
>index d654de488d..317a8d3683 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/sco/sco-main.tt
>@@ -3,6 +3,7 @@
> [% USE Koha %]
> [% USE KohaDates %]
> [% USE AudioAlerts %]
>+[% USE To %]
> [% USE Price %]
> [% INCLUDE 'doc-head-open.inc' %]
> <title>[% IF ( LibraryNameTitle ) %][% LibraryNameTitle | html %][% ELSE %]Koha [% END %] &rsaquo; Self checkout </title>
>@@ -406,7 +407,7 @@
>             return true;
>         }
>         [% IF Koha.Preference('AudioAlerts') %]
>-            var AUDIO_ALERTS = JSON.parse( '[% AudioAlerts.AudioAlerts | replace( "'", "\\'" ) | replace( '"', '\\"' ) | html %]' );
>+            var AUDIO_ALERTS = JSON.parse( "[% To.json(AudioAlerts.AudioAlerts) | $raw %]" );
>             $( document ).ready(function() {
>                 if ( AUDIO_ALERTS ) {
>                     for ( var k in AUDIO_ALERTS ) {
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 22378: 85410 | 85615