Fixed some major sonar issues

[yangtools.git] / common / checkstyle-logging / src / main / java / org / opendaylight / yangtools / checkstyle / LoggerMustBeSlf4jCheck.java

diff --git a/common/checkstyle-logging/src/main/java/org/opendaylight/yangtools/checkstyle/LoggerMustBeSlf4jCheck.java b/common/checkstyle-logging/src/main/java/org/opendaylight/yangtools/checkstyle/LoggerMustBeSlf4jCheck.java
index 2871403f398213643377faa4128ae95f6d6d6536..2ddcdddfe4357d5758d34b462922f346fbc48c6d 100644 (file)
--- a/common/checkstyle-logging/src/main/java/org/opendaylight/yangtools/checkstyle/LoggerMustBeSlf4jCheck.java
+++ b/common/checkstyle-logging/src/main/java/org/opendaylight/yangtools/checkstyle/LoggerMustBeSlf4jCheck.java
@@ -27,20 +27,20 @@ public class LoggerMustBeSlf4jCheck extends Check {
 
     @Override
     public void visitToken(DetailAST aAST) {
-        if(aAST.getType() == TokenTypes.VARIABLE_DEF) {
-            final String typeName = CheckLoggingUtil.getTypeName(aAST);
-            if (CheckLoggingUtil.itsAFieldVariable(aAST) && typeName.contains("." + LOGGER_TYPE_NAME)) {
-                if(!typeName.equals(LOGGER_TYPE_FULL_NAME)) {
+        if (aAST.getType() == TokenTypes.VARIABLE_DEF) {
+            if (CheckLoggingUtil.isAFieldVariable(aAST)) {
+                final String typeName = CheckLoggingUtil.getTypeName(aAST);
+                if (typeName.contains("." + LOGGER_TYPE_NAME) && !typeName.equals(LOGGER_TYPE_FULL_NAME)) {
                     log(aAST.getLineNo(), LOG_MESSAGE);
                 }
             }
-        } else if(aAST.getType() == TokenTypes.IMPORT) {
+        } else if (aAST.getType() == TokenTypes.IMPORT) {
             final String importType = aAST.getFirstChild().findFirstToken(TokenTypes.IDENT).getText();
-            if(importType.equals(CheckLoggingUtil.LOGGER_TYPE_NAME)) {
-               final String importIdent = aAST.getFirstChild().getFirstChild().getLastChild().getText();
-                if(!importIdent.equals(SLF4J)) {
-                   log(aAST.getLineNo(), LOG_MESSAGE);
-               }
+            if (importType.equals(CheckLoggingUtil.LOGGER_TYPE_NAME)) {
+                final String importIdent = aAST.getFirstChild().getFirstChild().getLastChild().getText();
+                if (!importIdent.equals(SLF4J)) {
+                    log(aAST.getLineNo(), LOG_MESSAGE);
+                }
             }
         }
     }