maketank/Froxlor
2
0
Fork 0
Code Pull Requests Releases Activity

Add missing variable declarations

Browse Source 
This fixes warnings from LGTM:

    Variable i is used like a local variable,
    but is missing a declaration.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
This commit is contained in:
Stefan Weil
2018-10-17 11:49:13 +02:00
parent 20177e06fc
commit 8cf797374d
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
templates/Sparkle/assets/js/customers.js vendored
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;
}

2
templates/Sparkle/assets/js/domains.js vendored
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");
}