Uploaded image for project: 'Commons Logging'
  1. Commons Logging
  2. LOGGING-9

[logging] Log.trace() doesn't use log4j 1.3 trace methods

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.4
    • Fix Version/s: 1.1.0
    • Labels:
      None
    • Environment:

      Operating System: other
      Platform: Other

    • Bugzilla Id:
      34437

      Description

      Jakarta Commons Logging 1.0.4 detects log4j 1.3 and alters its behaviour
      accordingly but continues to map its trace methods to log4j debug methods
      while log4j 1.3 now has its own trace methods. The following patch fixes this
      bug:

      Index: commons-
      logging/src/java/org/apache/commons/logging/impl/Log4JLogger.java
      ===================================================================
      — commons-logging/src/java/org/apache/commons/logging/impl/Log4JLogger.java
      (revision 161137)
      +++ commons-logging/src/java/org/apache/commons/logging/impl/Log4JLogger.java
      (working copy)
      @@ -84,7 +84,7 @@
      if(is12)

      { getLogger().log(FQCN, (Priority) Level.DEBUG, message, null ); }

      else

      { - getLogger().log(FQCN, Level.DEBUG, message, null ); + getLogger().log(FQCN, Level.TRACE, message, null ); }

      }

      @@ -97,7 +97,7 @@
      if(is12)

      { getLogger().log(FQCN, (Priority) Level.DEBUG, message, t ); }

      else

      { - getLogger().log(FQCN, Level.DEBUG, message, t ); + getLogger().log(FQCN, Level.TRACE, message, t ); }

      }

      @@ -277,7 +277,11 @@

      • For Log4J, this returns the value of <code>isDebugEnabled()</code>
        */
        public boolean isTraceEnabled() {
      • return getLogger().isDebugEnabled();
        + if(is12) { + return getLogger().isDebugEnabled(); + }

        else

        { + return getLogger().isTraceEnabled(); + }

        }

      /**

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                pdegregorio@comcast.net Peter DeGregorio
              • Votes:
                2 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: