- ".$frm->checkbox('verify_table[]', $x,false,'label='.$x)." |
+ ".$frm->checkbox('verify_table['.$t.']', $x, false, array('label'=>$x))." |
";
}
diff --git a/e107_handlers/form_handler.php b/e107_handlers/form_handler.php
index 0d60d2a51..6afc4d6ff 100644
--- a/e107_handlers/form_handler.php
+++ b/e107_handlers/form_handler.php
@@ -1504,11 +1504,11 @@ class e_form
return $this->checkbox($name, $value, $checked).$this->label($label ? $label : LAN_ENABLED, $name, $value);
}
- function checkbox_toggle($name, $selector = 'multitoggle', $id = false, $label='')
+ function checkbox_toggle($name, $selector = 'multitoggle', $id = false, $label='') //TODO Fixme - labels will break this. Don't use checkbox, use html.
{
$selector = 'jstarget:'.$selector;
if($id) $id = $this->name2id($id);
-
+
return $this->checkbox($name, $selector, false, array('id' => $id,'class' => 'checkbox checkbox-inline toggle-all','label'=>$label));
}
diff --git a/e107_web/js/core/admin.jquery.js b/e107_web/js/core/admin.jquery.js
index d17e99f0f..c8b3fcae3 100644
--- a/e107_web/js/core/admin.jquery.js
+++ b/e107_web/js/core/admin.jquery.js
@@ -539,15 +539,18 @@ $(document).ready(function()
// Check-All checkbox toggle
$("input.toggle-all").click(function(evt) {
var selector = 'input[type="checkbox"].checkbox';
+
if($(this).val().indexOf('jstarget:') === 0) {
selector = 'input[type="checkbox"][name^="' + $(this).val().split(/jstarget\:/)[1] + '"]';
}
-
- if($(this).is(":checked")){
- $(selector).attr("checked", "checked");
+
+ if($(this).is(":checked")){
+ //$(selector).attr("checked", "checked");
+ $(selector).prop('checked', true);
}
else{
- $(selector).removeAttr("checked");
+ $(selector).prop('checked',false);
+ // $(selector).removeAttr("checked");
}
});