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 <[email protected]> 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
