Fix typo in LoggerConfig.RootLogger.Builder#withtFilter (#3372)

In `LoggerConfig.RootLogger.Builder`:

- Deprecated method with typo in name `withtFilter`.
- Added correct method `setFilter`.


Co-authored-by: Piotr P. Karwasz <[email protected]>

Author: JWT007

log4j-core-test/src/test/java/org/apache/logging/log4j/core/async/AsyncLoggerConfigTest.java

@@ -100,7 +100,7 @@ void testSingleFilterInvocation() {
                 .withLoggerName(FQCN)
                 .withConfig(configuration)
                 .withLevel(Level.INFO)
-                .withFilter(filter)
+                .setFilter(filter)
                 .build();
         final Appender appender = mock(Appender.class);
         when(appender.isStarted()).thenReturn(true);

log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/CustomConfigurationTest.java

@@ -63,20 +63,26 @@ void testConfig(final LoggerContext ctx) {
                 .withConfiguration(config)
                 .build();
         final FileAppender appender = FileAppender.newBuilder()
-                .withFileName(logFile.toString())
-                .withAppend(false)
-                .setName("File")
+                .setBufferedIo(false)
                 .setIgnoreExceptions(false)
-                .withBufferedIo(false)
+                .setName("File")
                 .setLayout(layout)
+                .withAppend(false)
+                .withFileName(logFile.toString())
                 .build();
         appender.start();
         config.addAppender(appender);
         final AppenderRef ref = AppenderRef.createAppenderRef("File", null, null);
         final AppenderRef[] refs = new AppenderRef[] {ref};
 
-        final LoggerConfig loggerConfig = LoggerConfig.createLogger(
-                false, Level.INFO, "org.apache.logging.log4j", "true", refs, null, config, null);
+        final LoggerConfig loggerConfig = LoggerConfig.newBuilder()
+                .withConfig(config)
+                .withAdditivity(false)
+                .withIncludeLocation("true")
+                .withLevel(Level.INFO)
+                .withLoggerName("org.apache.logging.log4j")
+                .withRefs(refs)
+                .build();
         loggerConfig.addAppender(appender, null, null);
         config.addLogger("org.apache.logging.log4j", loggerConfig);
         ctx.updateLoggers();

log4j-core-test/src/test/java/org/apache/logging/log4j/core/config/LoggerConfigTest.java

@@ -47,11 +47,16 @@ class LoggerConfigTest {
     private static final String FQCN = LoggerConfigTest.class.getName();
 
     private static LoggerConfig createForProperties(final Property[] properties) {
-        return LoggerConfig.createLogger(
-                true, Level.INFO, "name", "false", new AppenderRef[0], properties, new NullConfiguration(), null);
+        return LoggerConfig.newBuilder()
+                .withConfig(new NullConfiguration())
+                .withAdditivity(true)
+                .withLevel(Level.INFO)
+                .withLoggerName("name")
+                .withIncludeLocation("false")
+                .withProperties(properties)
+                .build();
     }
 
-    @SuppressWarnings({"deprecation"})
     @Test
     void testPropertiesWithoutSubstitution() {
         assertNull(createForProperties(null).getPropertyList(), "null propertiesList");
@@ -129,7 +134,7 @@ void testSingleFilterInvocation() {
                 .withLoggerName(FQCN)
                 .withConfig(configuration)
                 .withLevel(Level.INFO)
-                .withFilter(filter)
+                .setFilter(filter)
                 .build();
         final Appender appender = mock(Appender.class);
         when(appender.isStarted()).thenReturn(true);

log4j-core/src/main/java/org/apache/logging/log4j/core/async/package-info.java

@@ -18,7 +18,7 @@
  * Provides Asynchronous Logger classes and interfaces for low-latency logging.
  */
 @Export
-@Version("2.24.1")
+@Version("2.25.0")
 package org.apache.logging.log4j.core.async;
 
 import org.osgi.annotation.bundle.Export;

log4j-core/src/main/java/org/apache/logging/log4j/core/config/LoggerConfig.java

@@ -213,15 +213,21 @@ public Filter getFilter() {
         }
 
         /**
-         * @deprecated Use {@link #withFilter(Filter)} instead
+         * @deprecated Use {@link #setFilter(Filter)} instead
          */
         @Deprecated
         public B withtFilter(final Filter filter) {
-            this.filter = filter;
-            return asBuilder();
+            return setFilter(filter);
         }
 
+        /** @deprecated since 2.25.0. Use {@link #setFilter(Filter)} instead. */
+        @Deprecated
         public B withFilter(final Filter filter) {
+            return setFilter(filter);
+        }
+
+        /** @since 2.25.0 */
+        public B setFilter(final Filter filter) {
             this.filter = filter;
             return asBuilder();
         }
@@ -936,7 +942,16 @@ public Filter getFilter() {
                 return filter;
             }
 
+            /**
+             * @deprecated since 2.25.0. Use {@link #setFilter(Filter)} instead.
+             */
+            @Deprecated
             public B withtFilter(final Filter filter) {
+                return setFilter(filter);
+            }
+
+            /** @since 2.25.0 */
+            public B setFilter(final Filter filter) {
                 this.filter = filter;
                 return asBuilder();
             }

log4j-core/src/main/java/org/apache/logging/log4j/core/config/package-info.java

@@ -18,7 +18,7 @@
  * Configuration of Log4j 2.
  */
 @Export
-@Version("2.24.1")
+@Version("2.25.0")
 package org.apache.logging.log4j.core.config;
 
 import org.osgi.annotation.bundle.Export;

src/changelog/.2.x.x/3369_fix_typo_in_RootLogger_Builder_method.xml

@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<entry xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+       xmlns="https://logging.apache.org/xml/ns"
+       xsi:schemaLocation="https://logging.apache.org/xml/ns https://logging.apache.org/xml/ns/log4j-changelog-0.xsd"
+       type="added">
+  <issue id="3369" link="https://github.com/apache/logging-log4j2/issues/3369"/>
+  <description format="asciidoc">
+    Fixed typo in "LoggerConfig.RootLogger.Builder#withtFilter(...)" method.
+    Created new corrected 'withFilter()' method and deprecated old method.
+  </description>
+</entry>