Bug 18706 - subfields to delete not disabled anymore in batch item modification

In Tools > Batch item modification, one can click a checkbox to delete a subfield.
This click use to disable the input to change the field value.

This comes from the patch : Bug 13501: Move "Required" and checkbox after input/select
Because input is found using JQuery next().

This patch replaces with $(this).parent().find("[name='field_value']").

Test plan :
- Go to Tools > Batch item modification
- Enter a barcode and submit
- Click on a checkbox on a non-mandatory subfield
=> Without patch the input or select is not disabled
=> With patch the input or select is disabled
- Submit
=> Check that subfield is deleted

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit 7c9273be16)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit dc1f51959a)
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
This commit is contained in:
Fridolin Somers 2017-05-31 16:02:44 +02:00 committed by Katrin Fischer
parent b64e0dfdbc
commit f82009d10e
1 changed files with 4 additions and 4 deletions

View File

@ -22,13 +22,13 @@ $(document).ready(function(){
$("input[name='disable_input']").click(function() {
var row = $(this).attr("id");
row = row.replace("row","hint");
var subfield = $(this).next();
if ($(this).is(":checked")){
var editor = $(this).parent().find("[name='field_value']");
if ($(this).is(":checked")) {
$(this).val($(this).siblings("[name='subfield']").val());
$(subfield).prop('disabled', true);
$(editor).prop('disabled', true);
$("#"+row).html(_("This subfield will be deleted"));
} else {
$(subfield).prop('disabled', false);
$(editor).prop('disabled', false);
$("#"+row).html("");
}
});