fix: replace var with let in swagger-initializer.js

[sonarqube-agent]

Replace 'var' with 'let' for the 'baseUrl' variable declaration to comply with SonarQube rule S3504. This change improves code quality by using block-level scoping instead of function-level scoping, preventing potential hoisting-related bugs and following modern JavaScript best practices.

View Project in SonarCloud

---

Fixed Issues

javascript:S3504 - Unexpected var, use let or const instead. • CRITICAL • View issue

Location: swagger/src/main/resources/com/webcohesion/enunciate/modules/swagger/swagger-initializer.js:11

Why is this an issue?

Variables declared with var are function-scoped, meaning they are accessible within the entire function in which they are defined. If a variable is declared using var outside of any function, it becomes a global variable and is accessible throughout the entire JavaScript program.

What changed

Replaces 'var' with 'let' for the 'baseUrl' variable declaration. The static analysis rule flags the use of 'var' because it has function-level scoping and hoisting behavior that can lead to unexpected bugs. Since 'baseUrl' is reassigned later in the code (e.g., trimming the hash or query string), 'let' is the appropriate modern replacement, providing block-level scoping and avoiding the pitfalls of 'var'.

--- a/swagger/src/main/resources/com/webcohesion/enunciate/modules/swagger/swagger-initializer.js
+++ b/swagger/src/main/resources/com/webcohesion/enunciate/modules/swagger/swagger-initializer.js
@@ -11,1 +11,1 @@ window.onload = function() {
-    var baseUrl = document.location.href;
+    let baseUrl = document.location.href;

Have a suggestion or found an issue? Share your feedback here.

---

SonarQube Remediation Agent uses AI. Check for mistakes.

[sonarqubecloud]

SonarQube reviewer guide

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud