[LOG4PHP-16] Patch for LoggerAppenderSyslog to use Layouts

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.0.0
• Component/s: Code
• Assignee: Knut Urdalen
• Reporter: Gary Richardson
• Created: 17/Jul/08 16:31
• Updated: 01/Dec/13 10:56
• Resolved: 29/Apr/09 20:35

Description

I noticed that LoggerAppenderSyslog seems to ignore layouts. I'm not sure if this is by design.

The following patch causes LoggerAppenderSyslog to use layouts:

Index: src/main/php/appenders/LoggerAppenderSyslog.php
===================================================================
— src/main/php/appenders/LoggerAppenderSyslog.php (revision 677439)
+++ src/main/php/appenders/LoggerAppenderSyslog.php (working copy)
@@ -86,6 +86,11 @@
*/
private $_overridePriority;

+ public function __construct($name)

{ + parent::__construct($name); + $this->requiresLayout = true; + }

+
/**

• Set the ident of the syslog message.
  *
  @@ -156,7 +161,12 @@
  openlog($this->_ident, $this->_option, $this->_facility);

$level = $event->getLevel();

• $message = $event->getRenderedMessage();
  + if(null == $this->layout) { + $message = $event->getRenderedMessage(); + }

  + else

  { + $message = $this->layout->format($event); + }

// If the priority of a syslog message can be overridden by a value defined in the properties-file,
// use that value, else use the one that is defined in the code.