Attachment 82552 Details for Bug 21819 – Bug 21819: (QA follow-up) Replace onchange() in touched code

[patch] Bug 21819: (QA follow-up) Replace onchange() in touched code

Bug-21819-QA-follow-up-Replace-onchange-in-touched.patch (text/plain), 2.10 KB, created by Marcel de Rooy on 2018-11-20 15:41:45 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2018-11-20 15:41:45 UTC

Size: 2.10 KB

patch

obsolete

>From 1244ce82c708b1be33ca2deec572c914e3a53fa1 Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Tue, 20 Nov 2018 16:32:46 +0100
>Subject: [PATCH] Bug 21819: (QA follow-up) Replace onchange() in touched code
>Content-Type: text/plain; charset=utf-8
>
>AFAIK we should not call document...onchange() and assume that it works.
>Apparently, it does at least in some browsers. But we are not sure.
>We could easily replace them here by the documented jQuery counterpart.
>
>Note: If you try git grep -l -E "onchange\(" you will find some
>occurrences in two libraries and only in one js file (this one).
>We should replace all occurrences here; will open a new report for that.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Verified that the change event was triggered: span should expand/collapse.
>---
> koha-tmpl/intranet-tmpl/prog/js/marc_modification_templates.js | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/marc_modification_templates.js b/koha-tmpl/intranet-tmpl/prog/js/marc_modification_templates.js
>index 6a23270..d6c7bfe 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/marc_modification_templates.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/marc_modification_templates.js
>@@ -256,11 +256,9 @@ function editAction( mmta_id, ordering, action, field_number, from_field, from_s
>     document.getElementById('to_field').value = to_field;
>     document.getElementById('to_subfield').value = to_subfield;
>     if ( to_regex_search == '' && to_regex_replace == '' && to_regex_modifiers == '' ) {
>-        document.getElementById('to_field_regex').checked = false;
>-        document.getElementById('to_field_regex').onchange();
>+        $('#to_field_regex').prop('checked', false).change();
>     } else {
>-        document.getElementById('to_field_regex').checked = true;
>-        document.getElementById('to_field_regex').onchange();
>+        $('#to_field_regex').prop('checked', true).change();
>         $("#to_regex_search").val(to_regex_search);
>         $("#to_regex_replace").val(to_regex_replace);
>         $("#to_regex_modifiers").val(to_regex_modifiers);
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 21819: 82424 | 82427 | 82551 | 82552 | 82555