diff --git a/src/main/java/org/openrewrite/java/logging/LoggingFramework.java b/src/main/java/org/openrewrite/java/logging/LoggingFramework.java index cb589e6..029f7ec 100644 --- a/src/main/java/org/openrewrite/java/logging/LoggingFramework.java +++ b/src/main/java/org/openrewrite/java/logging/LoggingFramework.java @@ -43,7 +43,7 @@ public String getFactoryType() { } public static LoggingFramework fromOption(@Nullable String option) { - if(option != null) { + if (option != null) { for (LoggingFramework value : values()) { if (value.toString().equalsIgnoreCase(option)) { return value; diff --git a/src/main/java/org/openrewrite/java/logging/ParameterizedLogging.java b/src/main/java/org/openrewrite/java/logging/ParameterizedLogging.java index 9965ea9..79c4107 100644 --- a/src/main/java/org/openrewrite/java/logging/ParameterizedLogging.java +++ b/src/main/java/org/openrewrite/java/logging/ParameterizedLogging.java @@ -183,7 +183,7 @@ private static Object getLiteralValue(J.Literal literal) { } return literal .getValueSource() - .substring(1, literal.getValueSource().length() -1) + .substring(1, literal.getValueSource().length() - 1) .replace("\\", "\\\\"); } } diff --git a/src/main/java/org/openrewrite/java/logging/slf4j/ChangeLogLevel.java b/src/main/java/org/openrewrite/java/logging/slf4j/ChangeLogLevel.java index 7c994cc..ac78100 100644 --- a/src/main/java/org/openrewrite/java/logging/slf4j/ChangeLogLevel.java +++ b/src/main/java/org/openrewrite/java/logging/slf4j/ChangeLogLevel.java @@ -71,23 +71,23 @@ public enum Level { @Override public TreeVisitor getVisitor() { String methodPattern = "org.slf4j.Logger " + from.name().toLowerCase() + "(..)"; - return Preconditions.check(new UsesMethod<>(methodPattern), new JavaIsoVisitor() { + return Preconditions.check(new UsesMethod<>(methodPattern), new JavaIsoVisitor() { final MethodMatcher logMatcher = new MethodMatcher(methodPattern); @Override public J.MethodInvocation visitMethodInvocation(J.MethodInvocation method, ExecutionContext ctx) { J.MethodInvocation m = super.visitMethodInvocation(method, ctx); - if(!logMatcher.matches(m)) { + if (!logMatcher.matches(m)) { return m; } List args = m.getArguments(); - if(args.size() == 0) { + if (args.size() == 0) { return m; } J.Literal lit = leftMostLiteral(args.get(0)); - if(lit == null || lit.getValue() == null) { + if (lit == null || lit.getValue() == null) { return m; } - if(!StringUtils.isBlank(startsWith) && !lit.getValue().toString().startsWith(startsWith)) { + if (!StringUtils.isBlank(startsWith) && !lit.getValue().toString().startsWith(startsWith)) { return m; } m = (J.MethodInvocation) new ChangeMethodName(methodPattern, to.name().toLowerCase(), true, null) @@ -100,10 +100,10 @@ public J.MethodInvocation visitMethodInvocation(J.MethodInvocation method, Execu @Nullable J.Literal leftMostLiteral(Expression arg) { - if(arg instanceof J.Literal) { + if (arg instanceof J.Literal) { return (J.Literal) arg; } - if(arg instanceof J.Binary) { + if (arg instanceof J.Binary) { return leftMostLiteral(((J.Binary) arg).getLeft()); } return null; diff --git a/src/main/java/org/openrewrite/java/logging/slf4j/CompleteExceptionLogging.java b/src/main/java/org/openrewrite/java/logging/slf4j/CompleteExceptionLogging.java index 8e8f306..ba1bce7 100644 --- a/src/main/java/org/openrewrite/java/logging/slf4j/CompleteExceptionLogging.java +++ b/src/main/java/org/openrewrite/java/logging/slf4j/CompleteExceptionLogging.java @@ -96,7 +96,7 @@ public J.MethodInvocation visitMethodInvocation(J.MethodInvocation method, Expression lastParameter = args.get(args.size() - 1); boolean isLastParameterAnException = lastParameter instanceof J.Identifier && - TypeUtils.isAssignableTo("java.lang.Throwable",lastParameter.getType()); + TypeUtils.isAssignableTo("java.lang.Throwable", lastParameter.getType()); if (isLastParameterAnException) { return method; } diff --git a/src/test/java/org/openrewrite/java/logging/slf4j/LoggersNamedForEnclosingClassTest.java b/src/test/java/org/openrewrite/java/logging/slf4j/LoggersNamedForEnclosingClassTest.java index d5954d4..100952d 100644 --- a/src/test/java/org/openrewrite/java/logging/slf4j/LoggersNamedForEnclosingClassTest.java +++ b/src/test/java/org/openrewrite/java/logging/slf4j/LoggersNamedForEnclosingClassTest.java @@ -121,7 +121,7 @@ void method() { @Issue("https://github.com/openrewrite/rewrite-logging-frameworks/issues/101") @Test - void doNotChangeGetClassOnNonFinalClass(){ + void doNotChangeGetClassOnNonFinalClass() { //language=java rewriteRun( java(