diff --git a/java-checks/src/main/java/org/sonar/java/checks/HardcodedURICheck.java b/java-checks/src/main/java/org/sonar/java/checks/HardcodedURICheck.java index 9407d0c4db6..c4d68f9e955 100644 --- a/java-checks/src/main/java/org/sonar/java/checks/HardcodedURICheck.java +++ b/java-checks/src/main/java/org/sonar/java/checks/HardcodedURICheck.java @@ -117,17 +117,17 @@ public void leaveFile(JavaFileScannerContext context) { try { for (VariableData v : hardCodedUri) { - // equals to an identifier with unknown semantic, we cannot compare their symbols - if (idNamesWithoutSemantic.contains(v.identifier())) { - continue; - } - - // idNamesWithSemantic is used to only compare the symbols when their string identifier are the same - // as comparing symbols is costly - if (idNamesWithSemantic.contains(v.identifier()) && idSymbols.contains(v.symbol())) { - continue; - } - reportHardcodedURI(v.initializer()); + // equals to an identifier with unknown semantic, we cannot compare their symbols + if (idNamesWithoutSemantic.contains(v.identifier())) { + continue; + } + + // idNamesWithSemantic is used to only compare the symbols when their string identifier are the same + // as comparing symbols is costly + if (idNamesWithSemantic.contains(v.identifier()) && idSymbols.contains(v.symbol())) { + continue; + } + reportHardcodedURI(v.initializer()); } } finally { super.leaveFile(context);