You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by ca...@apache.org on 2007/08/22 07:32:19 UTC

svn commit: r568426 - /logging/log4j/companions/extras/trunk/src/main/java/org/apache/log4j/rolling/FilterBasedTriggeringPolicy.java

Author: carnold
Date: Tue Aug 21 22:32:19 2007
New Revision: 568426

URL: http://svn.apache.org/viewvc?rev=568426&view=rev
Log:
Bug 32572: FilterBasedTriggeringPolicy now usable with SMTPAppender

Modified:
    logging/log4j/companions/extras/trunk/src/main/java/org/apache/log4j/rolling/FilterBasedTriggeringPolicy.java

Modified: logging/log4j/companions/extras/trunk/src/main/java/org/apache/log4j/rolling/FilterBasedTriggeringPolicy.java
URL: http://svn.apache.org/viewvc/logging/log4j/companions/extras/trunk/src/main/java/org/apache/log4j/rolling/FilterBasedTriggeringPolicy.java?rev=568426&r1=568425&r2=568426&view=diff
==============================================================================
--- logging/log4j/companions/extras/trunk/src/main/java/org/apache/log4j/rolling/FilterBasedTriggeringPolicy.java (original)
+++ logging/log4j/companions/extras/trunk/src/main/java/org/apache/log4j/rolling/FilterBasedTriggeringPolicy.java Tue Aug 21 22:32:19 2007
@@ -20,6 +20,7 @@
 import org.apache.log4j.spi.Filter;
 import org.apache.log4j.spi.LoggingEvent;
 import org.apache.log4j.spi.OptionHandler;
+import org.apache.log4j.spi.TriggeringEventEvaluator;
 import org.apache.log4j.xml.UnrecognizedElementHandler;
 import org.w3c.dom.Element;
 
@@ -35,7 +36,7 @@
  *
  */
 public final class FilterBasedTriggeringPolicy
-        implements TriggeringPolicy, UnrecognizedElementHandler {
+        implements TriggeringPolicy, TriggeringEventEvaluator, UnrecognizedElementHandler {
   /**
    * The first filter in the filter chain. Set to <code>null</code> initially.
    */
@@ -52,16 +53,13 @@
   public FilterBasedTriggeringPolicy() {
   }
 
-  /**
-   * {@inheritDoc}
-   *
-   */
-  public boolean isTriggeringEvent(
-    final Appender appender, final LoggingEvent event, final String file,
-    final long fileLength) {
+    /**
+     * {@inheritDoc}
+     */
+  public boolean isTriggeringEvent(LoggingEvent event) {
     //
     //   in the abnormal case of no contained filters
-    //     always return true to avoid each logging event 
+    //     always return true to avoid each logging event
     //     from having its own file.
     if (headFilter == null) {
       return false;
@@ -81,6 +79,17 @@
     }
 
     return true;
+   }
+
+
+  /**
+   * {@inheritDoc}
+   *
+   */
+  public boolean isTriggeringEvent(
+    final Appender appender, final LoggingEvent event, final String file,
+    final long fileLength) {
+    return isTriggeringEvent(event);
   }
 
   /**



---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org