summaryrefslogtreecommitdiff
path: root/public/js
diff options
context:
space:
mode:
authormsquare <msquare@notrademark.de>2016-08-21 20:14:09 +0200
committermsquare <msquare@notrademark.de>2016-08-21 20:14:09 +0200
commit191328d7034a229901cac4af1240ab1885713129 (patch)
tree04a5037dcfde6495c3ab821a8d82fc51ba5b5812 /public/js
parent53ad1b5110e22bdbb57960aec7d29b824dda7b5b (diff)
fix method names, quotes, duplication in forms.js
Diffstat (limited to 'public/js')
-rw-r--r--public/js/forms.js32
1 files changed, 18 insertions, 14 deletions
diff --git a/public/js/forms.js b/public/js/forms.js
index a8c7f6ce..896cbb35 100644
--- a/public/js/forms.js
+++ b/public/js/forms.js
@@ -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)
+ for (var i = 0; i < boxes.length; i++) {
+ 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;
});
});