Fix AppenderRef

Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/f07465bd
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/f07465bd
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/f07465bd

Branch: 
refs/heads/LOG4J2-1010&LOG4J2-1447-injectable-contextdata&better-datastructure
Commit: f07465bd57ff31f943e26774b0d1aabf21e82289
Parents: 0728abc
Author: Mikael Ståldal <mikael.stal...@magine.com>
Authored: Thu Aug 18 14:44:48 2016 +0200
Committer: Mikael Ståldal <mikael.stal...@magine.com>
Committed: Thu Aug 18 14:44:48 2016 +0200

----------------------------------------------------------------------
 .../org/apache/log4j/config/Log4j1ConfigurationFactory.java   | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/f07465bd/log4j-1.2-api/src/main/java/org/apache/log4j/config/Log4j1ConfigurationFactory.java
----------------------------------------------------------------------
diff --git 
a/log4j-1.2-api/src/main/java/org/apache/log4j/config/Log4j1ConfigurationFactory.java
 
b/log4j-1.2-api/src/main/java/org/apache/log4j/config/Log4j1ConfigurationFactory.java
index f70468b..f8067e5 100644
--- 
a/log4j-1.2-api/src/main/java/org/apache/log4j/config/Log4j1ConfigurationFactory.java
+++ 
b/log4j-1.2-api/src/main/java/org/apache/log4j/config/Log4j1ConfigurationFactory.java
@@ -34,6 +34,7 @@ import 
org.apache.logging.log4j.core.config.ConfigurationSource;
 import 
org.apache.logging.log4j.core.config.builder.api.AppenderComponentBuilder;
 import org.apache.logging.log4j.core.config.builder.api.ConfigurationBuilder;
 import org.apache.logging.log4j.core.config.builder.api.LayoutComponentBuilder;
+import 
org.apache.logging.log4j.core.config.builder.api.RootLoggerComponentBuilder;
 import org.apache.logging.log4j.core.config.builder.impl.BuiltConfiguration;
 import org.apache.logging.log4j.status.StatusLogger;
 
@@ -239,9 +240,13 @@ public class Log4j1ConfigurationFactory extends 
ConfigurationFactory {
         }
         final String[] rootLoggerParts = rootLoggerValue.split("\\s*,\\s*");
         final Level rootLoggerLevel = rootLoggerParts.length > 0 ? 
Level.valueOf(rootLoggerParts[0]) : Level.ERROR;
-        builder.add(builder.newRootLogger(rootLoggerLevel));
         final String[] sortedAppenderNames = 
Arrays.copyOfRange(rootLoggerParts, 1, rootLoggerParts.length);
         Arrays.sort(sortedAppenderNames);
+        RootLoggerComponentBuilder loggerBuilder = 
builder.newRootLogger(rootLoggerLevel);
+        for (String appender : sortedAppenderNames) {
+            loggerBuilder.add(builder.newAppenderRef(appender));
+        }
+        builder.add(loggerBuilder);
         return sortedAppenderNames;
     }
 

Reply via email to