Merge pull request #578 from stweil/lgtm

Fix warnings from LGTM
This commit is contained in:
Michael Kaufmann
2018-10-17 11:55:36 +02:00
committed by GitHub
3 changed files with 7 additions and 7 deletions

View File

@@ -75,7 +75,7 @@ $(document).ready(function() {
circularCircle(canvas, 42, 0, 270, 4, usedColor);
circularCircle(canvas, 38, 0, 270, 4, assiColor);
}
circularText(canvas, 60, 42, 26, "∞")
circularText(canvas, 60, 42, 26, "∞");
} else if (!isNaN(assigned)) {
// Limited ressources but assigned
available = parseFloat(available);
@@ -90,7 +90,7 @@ $(document).ready(function() {
circularText(canvas, 60, 42, 22, usedP + "%");
} else if (available == "∞") {
circularCircle(canvas, 40, 0, 270, 8, unliColor);
circularText(canvas, 60, 42, 26, "∞")
circularText(canvas, 60, 42, 26, "∞");
} else {
// Limited ressources
available = parseFloat(available);
@@ -111,7 +111,7 @@ $(document).ready(function() {
if (usedP > 100) {
circularText(canvas, 60, 42, 22, usedP + "%", overColor);
} else {
circularText(canvas, 60, 42, 22, usedP + "%")
circularText(canvas, 60, 42, 22, usedP + "%");
}

View File

@@ -30,7 +30,7 @@ $(document).ready(function() {
},
dataType: "json",
success: function(json) {
for (i in json) {
for (var i in json) {
if (i == 'email_imap' || i == 'email_pop3' || i == 'perlenabled' || i == 'phpenabled' || i == 'dnsenabled') {
/** handle checkboxes **/
if (json[i] == 1) {
@@ -42,8 +42,8 @@ $(document).ready(function() {
/** handle array of values **/
$("input[name='allowed_phpconfigs[]']").each(function(index) {
$(this).prop('checked', false);
for (j in json[i]) {
if ($(this).val() == json[i][j]) {
for (var j in json[i]) {
if ($(this).val() == json[i][j]) {
$(this).prop('checked', true);
break;
}

View File

@@ -35,7 +35,7 @@ $(document).ready(function() {
$('#phpsettingid option').each(function() {
var pid = $(this).val();
$(this).attr("disabled", "disabled");
for (i in json) {
for (var i in json) {
if (pid == json[i]) {
$(this).removeAttr("disabled");
}