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>
