Fix (?) for radio buttons in repeat groups, in certain templates, wer…

…en't correctly picking up defaults, and targetting the wrong repeat.
Fabrik · Nov 18, 2017 · 93068f7 · 93068f7
1 parent 978cd6e
commit 93068f7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/plugins/fabrik_element/radiobutton/radiobutton-min.js b/plugins/fabrik_element/radiobutton/radiobutton-min.js
diff --git a/plugins/fabrik_element/radiobutton/radiobutton.js b/plugins/fabrik_element/radiobutton/radiobutton.js
@@ -148,7 +148,10 @@ define(['jquery', 'fab/elementlist'], function (jQuery, FbElementList) {
             }
             this._getSubElements().each(function (sub) {
                 if (sub.value === v) {
-                    sub.checked = 'checked';
+                    sub.set('checked', true);
+                }
+                else {
+                    sub.set('checked', false);
                 }
             });
         },
@@ -183,6 +186,13 @@ define(['jquery', 'fab/elementlist'], function (jQuery, FbElementList) {
                 this.watchAddToggle();
                 this.watchAdd();
             }
+            this._getSubElements().each(function (sub, i) {
+                sub.id = this.options.element + '_input_' + i;
+                var label = sub.getParent('label');
+                if (label) {
+                    label.htmlFor = sub.id;
+                }
+            }.bind(this));
             this.parent(c);
             this.btnGroup();
         },