You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by rg...@apache.org on 2013/04/02 08:40:46 UTC
svn commit: r1463400 - in /logging/log4j/log4j2/trunk:
core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
src/changes/changes.xml
Author: rgoers
Date: Tue Apr 2 06:40:46 2013
New Revision: 1463400
URL: http://svn.apache.org/r1463400
Log:
LOG4J2-190 - BaseConfiguration addLoggerAppender saved appender using the Logger name.
Modified:
logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
logging/log4j/log4j2/trunk/src/changes/changes.xml
Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java?rev=1463400&r1=1463399&r2=1463400&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java (original)
+++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java Tue Apr 2 06:40:46 2013
@@ -236,7 +236,7 @@ public class BaseConfiguration extends A
}
protected PluginManager getPluginManager() {
- //don't cache a pluginmanager instance - packages may be updated, requiring
+ //don't cache a pluginmanager instance - packages may be updated, requiring
// re-discovery of plugins
PluginManager mgr = new PluginManager("Core");
mgr.collectPlugins();
@@ -307,7 +307,7 @@ public class BaseConfiguration extends A
public void setConfigurationMonitor(ConfigurationMonitor monitor) {
this.monitor = monitor;
}
-
+
public ConfigurationMonitor getConfigurationMonitor() {
return monitor;
}
@@ -315,7 +315,7 @@ public class BaseConfiguration extends A
public void setAdvertiser(Advertiser advertiser) {
this.advertiser = advertiser;
}
-
+
public Advertiser getAdvertiser() {
return advertiser;
}
@@ -332,7 +332,7 @@ public class BaseConfiguration extends A
public synchronized void addLoggerAppender(final org.apache.logging.log4j.core.Logger logger,
final Appender appender) {
final String name = logger.getName();
- appenders.putIfAbsent(name, appender);
+ appenders.putIfAbsent(appender.getName(), appender);
final LoggerConfig lc = getLoggerConfig(name);
if (lc.getName().equals(name)) {
lc.addAppender(appender, null, null);
Modified: logging/log4j/log4j2/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/src/changes/changes.xml?rev=1463400&r1=1463399&r2=1463400&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/src/changes/changes.xml (original)
+++ logging/log4j/log4j2/trunk/src/changes/changes.xml Tue Apr 2 06:40:46 2013
@@ -23,6 +23,9 @@
<body>
<release version="2.0-beta5" date="@TBD@" description="Bug fixes and enhancements">
+ <action issue="LOG4J2-190" dev="rgoers" type="fix" due-to="Werner">
+ BaseConfiguration addLoggerAppender saved appender using the Logger name.
+ </action>
<action issue="LOG4J2-160" dev="rgoers" type="update" due-to="Joanne Polsky">
Move Throwable pattern converter options processing to ThrowableFormatOptions class.
</action>