Author: ihabunek
Date: Sat Jun 18 12:24:12 2011
New Revision: 1137177

URL: http://svn.apache.org/viewvc?rev=1137177&view=rev
Log:
Modified LoggerAppenderMailEvent not to throw an exception when an email cannot 
be sent.

Modified:
    logging/log4php/trunk/src/main/php/appenders/LoggerAppenderMailEvent.php

Modified: 
logging/log4php/trunk/src/main/php/appenders/LoggerAppenderMailEvent.php
URL: 
http://svn.apache.org/viewvc/logging/log4php/trunk/src/main/php/appenders/LoggerAppenderMailEvent.php?rev=1137177&r1=1137176&r2=1137177&view=diff
==============================================================================
--- logging/log4php/trunk/src/main/php/appenders/LoggerAppenderMailEvent.php 
(original)
+++ logging/log4php/trunk/src/main/php/appenders/LoggerAppenderMailEvent.php 
Sat Jun 18 12:24:12 2011
@@ -156,12 +156,7 @@ class LoggerAppenderMailEvent extends Lo
                $addHeader = empty($this->from) ? '' : "From: 
{$this->from}\r\n";
                
                if(!$this->dry) {
-                       $result = mail($this->to, $this->subject, 
-                               $this->layout->getHeader() . 
$this->layout->format($event) . $this->layout->getFooter($event), $addHeader);  
                   
-                       if ($result === false) {
-                               // The error message is only printed to stderr 
as warning. Any idea how to get it?
-                               throw new LoggerException("Error sending mail 
to '".$this->to."'!");
-                       }
+                       $result = mail($this->to, $this->subject, 
$this->layout->getHeader() . $this->layout->format($event) . 
$this->layout->getFooter($event), $addHeader);                   
                } else {
                        echo "DRY MODE OF MAIL APP.: Send mail to: 
".$this->to." with additional headers '".trim($addHeader)."' and content: 
".$this->layout->format($event);
                }
@@ -170,4 +165,3 @@ class LoggerAppenderMailEvent extends Lo
                ini_set('smtp_port', $prevSmtpPort);
        }
 }
-


Reply via email to