Bug 34613: Replace onclick attributes in midiplayer.js
authorDavid Cook <dcook@prosentient.com.au>
Fri, 25 Aug 2023 01:26:01 +0000 (01:26 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Mon, 18 Sep 2023 18:31:47 +0000 (15:31 -0300)
commita4bdd27fd9d953a8ae086a6f3e8b7960f9b7160b
treebda10aed4304e8c52f06866a4759329f7f1ddf5d
parent8eb27c40e6fd771e8d4c2bff28cc1a45838af712
Bug 34613: Replace onclick attributes in midiplayer.js

This patch replaces the onclick attributes in midiplayer.js
with Javascript click handlers.

To test:
0. Apply patch
1. Clear browser cache (it may be necessary to go into the
   Network tab and explicitly disable cache)
2. Add 031 subfields 2gopnu to Default framework
3. Create a catalogue record with a 031 like follows:
2: pe
g: G-2
o: 4/4
p: 4bB''C2bE/2F4GbB/'bB2''C4D/F2.bE/4GG2bB/4'''C2C4''bB/4bE2G4bB/4bE2.F/
n: xFCGD
4. Enable the following sysprefs:
   OPACShowMusicalInscripts
   OPACPlayMusicalInscripts
5. Go to OPAC record view and click "Play this sample"
6. Play with the "Pause/Start" and "Stop" buttons
7. If the buttons work, it means the patch worked

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/opac-tmpl/lib/verovio/midiplayer.js