fix method names, quotes, duplication in forms.js

This commit is contained in:
msquare 2016-08-21 20:14:09 +02:00
parent 53ad1b5110
commit 191328d703
2 changed files with 20 additions and 16 deletions

View File

@ -856,8 +856,8 @@ function make_select($items, $selected, $name, $title = null) {
$html = '<div id="selection_' . $name . '" class="selection ' . $name . '">' . "\n";
$html .= implode("\n", $html_items);
$html .= buttons(array(
button("javascript: check_all('selection_" . $name . "')", _("All"), ""),
button("javascript: uncheck_all('selection_" . $name . "')", _("None"), "")
button("javascript: checkAll('selection_" . $name . "', true)", _("All"), ""),
button("javascript: checkAll('selection_" . $name . "', false)", _("None"), "")
));
$html .= '</div>' . "\n";
return $html;

View File

@ -1,24 +1,28 @@
function check_all(id) {
/**
* Runs through the DOM under the element with the given id, finds all
* checkboxes and sets them to the wanted state.
*
* @param String
* id Id of the element containing all the checkboxes
* @param Boolean
* checked True if the checkboxes should be checked
*/
function checkAll(id, checked) {
var obj = document.getElementById(id);
var boxes = obj.getElementsByTagName("input");
for (var i = 0; i < boxes.length; i++) {
if (boxes[i].type == "checkbox" && !boxes[i].disabled)
if (boxes[i].type === "checkbox" && !boxes[i].disabled) {
boxes[i].checked = true;
}
}
function uncheck_all(id) {
var obj = document.getElementById(id);
var boxes = obj.getElementsByTagName("input");
for ( var i = 0; i < boxes.length; i++) {
if (boxes[i].type == "checkbox")
boxes[i].checked = false;
}
}
$(function() {
$('form').submit(function(ev) {
$('input[type="submit"]').prop("readonly", true).addClass("disabled");
/**
* Disable every submit button after clicking (to prevent double-clicking)
*/
$("form").submit(function(ev) {
$("input[type='submit']").prop("readonly", true).addClass("disabled");
return true;
});
});