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 2012/09/03 02:30:18 UTC

svn commit: r1380100 - in /logging/log4j/log4j2/trunk: api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java src/changes/changes.xml

Author: rgoers
Date: Mon Sep  3 00:30:18 2012
New Revision: 1380100

URL: http://svn.apache.org/viewvc?rev=1380100&view=rev
Log:
Fix LOG4J2-74 - Logger.error(Marker, Message, Throwable) was internally using Level.TRACE.

Modified:
    logging/log4j/log4j2/trunk/api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java
    logging/log4j/log4j2/trunk/src/changes/changes.xml

Modified: logging/log4j/log4j2/trunk/api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java?rev=1380100&r1=1380099&r2=1380100&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java (original)
+++ logging/log4j/log4j2/trunk/api/src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java Mon Sep  3 00:30:18 2012
@@ -1147,8 +1147,8 @@ public abstract class AbstractLogger imp
      * @param t      A Throwable or null.
      */
     public void error(Marker marker, Message msg, Throwable t) {
-        if (isEnabled(Level.TRACE, marker, msg, t)) {
-            log(marker, FQCN, Level.TRACE, msg, t);
+        if (isEnabled(Level.ERROR, marker, msg, t)) {
+            log(marker, FQCN, Level.ERROR, msg, t);
         }
     }
 

Modified: logging/log4j/log4j2/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/src/changes/changes.xml?rev=1380100&r1=1380099&r2=1380100&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/src/changes/changes.xml (original)
+++ logging/log4j/log4j2/trunk/src/changes/changes.xml Mon Sep  3 00:30:18 2012
@@ -23,6 +23,9 @@
 
   <body>
     <release version="2.0-alpha3" date="TBD" description="">
+      <action issue="LOG4J2-74" dev="rgoers" type="fix">
+        Logger.error(Marker, Message, Throwable) was internally using Level.TRACE.
+      </action>
       <action issue="LOG4J2-75" dev="rgoers" type="fix">
         Enhanced Log4jContextListener to accept a configuration file location. Modified FileConfigurationMonitor
         to monitor files configured that way. Fixed other reconfiguration related bugs. Tested in JBoss and