From a4fe3fe8ba72dab2ca637cb2233b5da6ad2a9628 Mon Sep 17 00:00:00 2001
From: Julian Maurice <julian.maurice@biblibre.com>
Date: Fri, 20 Dec 2013 11:01:09 +0100
Subject: [PATCH] Bug 8064: Fix issues with control fields and non-repeatable
 check

Fields < 010 were systematically unchecked during target rebuild because
they have no subfields (unchecking manually all subfields of a field
automatically uncheck the field)
When user check a field or a subfield, we have to uncheck it before
calling (sub)field_can_be_added. Otherwise, field is treated as if it
was already in target record, and this is a problem for non-repeatable
fields.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
---
 koha-tmpl/intranet-tmpl/prog/en/js/merge-record.js | 26 ++++++++++++++++------
 1 file changed, 19 insertions(+), 7 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/merge-record.js b/koha-tmpl/intranet-tmpl/prog/en/js/merge-record.js
index ab74e1a..9c3f492 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/js/merge-record.js
+++ b/koha-tmpl/intranet-tmpl/prog/en/js/merge-record.js
@@ -124,7 +124,11 @@ function rebuild_target($sources, $target) {
     var fields = target_record[tag];
     for (i in fields) {
       var field = fields[i];
-      if (field.subfields.length > 0) {
+      if (parseInt(tag) < 10) {
+        var $field_clone = $('#' + field.id).clone();
+        $field_clone.find('.fieldpick').remove();
+        $target.append($field_clone);
+      } else if (field.subfields.length > 0) {
         var $field_clone = $('#' + field.id).clone();
         $field_clone.find('ul').empty();
         $field_clone.find('.fieldpick').remove();
@@ -137,7 +141,7 @@ function rebuild_target($sources, $target) {
           $field_clone.find('ul').append($subfield_clone);
         }
       } else {
-        $('#' + field.id).find('input.fieldpick').attr('checked', false);
+        $('#' + field.id).find('input.fieldpick').removeAttr('checked');
       }
     }
   }
@@ -149,12 +153,16 @@ function rebuild_target($sources, $target) {
 $(document).ready(function(){
     // When a field is checked / unchecked
     $('input.fieldpick').click(function() {
-        // (un)check all subfields
         var ischecked = this.checked;
-        if (ischecked && !field_can_be_added($('#tabs'), $(this).parent())) {
-          return false;
+        if (ischecked) {
+          $(this).removeAttr('checked');
+          if (!field_can_be_added($('#tabs'), $(this).parent())) {
+            return false;
+          }
+          $(this).attr('checked', 'checked');
         }
 
+        // (un)check all subfields
         $(this).parent().find("input.subfieldpick").each(function() {
             this.checked = ischecked;
         });
@@ -164,8 +172,12 @@ $(document).ready(function(){
     // When a field or subfield is checked / unchecked
     $("input.subfieldpick").click(function() {
       var ischecked = this.checked;
-      if (ischecked && !subfield_can_be_added($('#tabs'), $(this).parent())) {
-        return false;
+      if (ischecked) {
+        $(this).removeAttr('checked');
+        if (!subfield_can_be_added($('#tabs'), $(this).parent())) {
+          return false;
+        }
+        $(this).attr('checked', 'checked');
       }
       rebuild_target($('#tabs'), $('#resultul'));
     });
-- 
1.9.1