Bug 30709: 'Insert' button in notices editor not adding selected placeholders to...
authorOwen Leonard <oleonard@myacpl.org>
Fri, 6 May 2022 12:05:12 +0000 (12:05 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Fri, 6 May 2022 20:33:10 +0000 (10:33 -1000)
commitcbc3b19cdb8d993b555fe831faf5b039b44160e8
treee7463b4d59d3e9a9d7766797ef5e8ff142cc293d
parent0d196ae7fb0f2b977e9986eb58c8081f61808178
Bug 30709: 'Insert' button in notices editor not adding selected placeholders to notice

This patch updates letter.js so that it uses .prop("selected") to look
for selected <option>s instead of .attr("selected"). This is necessary
because of the jQuery upgrade.

To test, apply the patch and edit any notice. Test that you can select
one or more database columns and insert them into the body of a notice
by clicking "Insert." Confirm that your selections are inserted where
you left the cursor in the message textarea.

Test with multiple message transports and with multiple languages
installed (with TranslateNotices enabled).

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
koha-tmpl/intranet-tmpl/prog/js/letter.js