You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4php-dev@logging.apache.org by ih...@apache.org on 2011/06/18 14:24:12 UTC

svn commit: r1137177 - /logging/log4php/trunk/src/main/php/appenders/LoggerAppenderMailEvent.php

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);
 	}
 }
-