Bug 18122 - Audio alerts: Add hint on where to enable sounds
authorMarc Véron <veron@veron.ch>
Wed, 15 Feb 2017 09:24:56 +0000 (10:24 +0100)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 3 Mar 2017 18:15:53 +0000 (18:15 +0000)
This patch adds a hint to page Home > Administration > Audio alerts
and related help page about where to enable the sound system
(syspref 'AudioAlerts').

To test:
- Apply patch
- Disable syspref 'AudioAlerts'
- Go to Home > Administration > Audio alerts
- Note hint below title 'Audio alerts'
- Enable syspref 'AudioAlerts'
- Go to  Home > Administration > Audio alerts
- Hint is no longer visible (since audios are enabled)
- Open Help page (top right)
- Verify that hint on help page makes sense.

Amended to add alert class as requested by comment #2 2017-02-21 mv

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/admin/audio_alerts.tt
koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/audio_alerts.tt

index 340198e..a1d3309 100644 (file)
@@ -1,3 +1,4 @@
+[% USE Koha %]
 [% INCLUDE 'doc-head-open.inc' %]
 <title>Koha &rsaquo; Administration &rsaquo; Audio alerts</title>
 [% INCLUDE 'doc-head-close.inc' %]
 
                 <form id="delete-alert-form" action="audio_alerts.pl" method="post">
                     <h3>Audio alerts</h3>
+                    [%IF !( Koha.Preference('AudioAlerts') ) %]
+                        <div class="dialog alert">
+                            <p><strong>Note: </strong></p>
+                            <p>Please enable system preference 'AudioAlerts' to activate sounds.</p>
+                        </div>
+                    [% END %]
                     <table id="audio-alerts-table">
                         <thead id="audio-alerts-table-head">
                             <tr>
index 01c07d7..5b824cf 100644 (file)
@@ -4,6 +4,8 @@
 
 <p>This section of Koha lets you specify a given sound to play when a given jQuery selector is matched.</p>
 
+<p><em>Note: Use system preference 'AudioAlerts' to enable/disable sounds.</em><p>
+
 <h2>Adding a new alert</h2>
 
 <p>To add a new alert:</p>