roller-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From adityasha...@apache.org
Subject [roller] 04/05: Fixed: sonarqube issue - 'checked' is already defined Variables and functions should not be redeclared
Date Mon, 04 Jan 2021 13:09:20 GMT
This is an automated email from the ASF dual-hosted git repository.

adityasharma pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/roller.git

commit 82c5a5b53650cb24b83417308374c8b3297121bc
Author: Aditya Sharma <adityasharma@apache.org>
AuthorDate: Mon Jan 4 18:27:51 2021 +0530

    Fixed: sonarqube issue - 'checked' is already defined
    Variables and functions should not be redeclared
---
 app/src/main/webapp/theme/scripts/roller.js | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/app/src/main/webapp/theme/scripts/roller.js b/app/src/main/webapp/theme/scripts/roller.js
index 298a799..ef85d10 100644
--- a/app/src/main/webapp/theme/scripts/roller.js
+++ b/app/src/main/webapp/theme/scripts/roller.js
@@ -127,11 +127,12 @@ function folderPreference(folderId) {
 }
 
 function toggleNextRow(e) {
+    var checked;
     if (e.type === "checkbox") {
-        var checked = e.checked;
+        checked = e.checked;
     } else if (e.type === "radio") {
         var v = e.value;
-        var checked = (v === "1" || v === "y" || v === "true");
+        checked = (v === "1" || v === "y" || v === "true");
     }
     // var nextRow = e.parentNode.parentNode.nextSibling;
     // the above doesn't work on Mozilla since it treats white space as nodes


Mime
View raw message