Comments

palantir · Aug 3, 2022 · fd93165 · fd93165
1 parent 36b55d0
commit fd93165
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/baseline-error-prone/src/main/java/com/palantir/baseline/errorprone/DefaultLocale.java b/baseline-error-prone/src/main/java/com/palantir/baseline/errorprone/DefaultLocale.java
@@ -16,6 +16,8 @@
 
 package com.palantir.baseline.errorprone;
 
+import static com.google.errorprone.fixes.SuggestedFixes.qualifyType;
+
 import com.google.auto.service.AutoService;
 import com.google.errorprone.BugPattern;
 import com.google.errorprone.BugPattern.SeverityLevel;
@@ -34,7 +36,7 @@
         linkType = BugPattern.LinkType.CUSTOM,
         summary = "Implicit use of the platform default locale, which can result in differing behaviour between JVM"
                 + " executions.",
-        severity = SeverityLevel.WARNING)
+        severity = SeverityLevel.SUGGESTION)
 public final class DefaultLocale extends BugChecker implements BugChecker.MethodInvocationTreeMatcher {
 
     private static final long serialVersionUID = 1L;
@@ -47,13 +49,12 @@ public final class DefaultLocale extends BugChecker implements BugChecker.Method
     @Override
     public Description matchMethodInvocation(MethodInvocationTree tree, VisitorState state) {
         if (MATCHER.matches(tree, state)) {
+            SuggestedFix.Builder fix = SuggestedFix.builder();
             return buildDescription(tree)
-                    .addFix(SuggestedFix.builder()
-                            .addImport("java.util.Locale")
-                            .replace(
+                    .addFix(fix.replace(
                                     state.getEndPosition(tree.getMethodSelect()),
                                     state.getEndPosition(tree),
-                                    "(Locale.ROOT)")
+                                    String.format("(%s.ROOT)", qualifyType(state, fix, "java.util.Locale")))
                             .build())
                     .build();
         }

diff --git a/...ne-java/src/main/groovy/com/palantir/baseline/extensions/BaselineErrorProneExtension.java b/...ne-java/src/main/groovy/com/palantir/baseline/extensions/BaselineErrorProneExtension.java
@@ -38,7 +38,6 @@ public class BaselineErrorProneExtension {
             "CompileTimeConstantViolatesLiskovSubstitution",
             "ConsistentLoggerName",
             "ConsistentOverrides",
-            "DefaultLocale",
             "DeprecatedGuavaObjects",
             "ExecutorSubmitRunnableFutureIgnored",
             "ExtendsErrorOrThrowable",