Added __toString magic method to LoggerLevel.
authorIvan Habunek <ihabunek@apache.org>
Sat, 18 Sep 2010 07:46:03 +0000 (07:46 +0000)
committerIvan Habunek <ihabunek@apache.org>
Sat, 18 Sep 2010 07:46:03 +0000 (07:46 +0000)
git-svn-id: https://svn.apache.org/repos/asf/logging/log4php/trunk@998420 13f79535-47bb-0310-9956-ffa450edef68

src/changes/changes.xml
src/main/php/LoggerLevel.php
src/main/php/layouts/LoggerLayoutHtml.php
src/main/php/layouts/LoggerLayoutSimple.php
src/main/php/layouts/LoggerLayoutTTCC.php

index cc7be3f..6c47a73 100644 (file)
@@ -24,6 +24,7 @@
   </properties>
   <body>
        <release version="2.1" description="Stabilizing">
+               <action type="update" by="Ivan Habunek">Added __toString magic method to LoggerLevel.</action>
                <action type="fix" issue="LOG4PHP-117" by="Maciej Mazur, Ivan Habunek">LoggerConfiguratorIni::configure() and unexptected results from error_get_last()</action>
                <action type="fix" issue="LOG4PHP-113" by="Ivan Habunek">Milliseconds do not change when using LoggerLayoutPattern</action>
                <action type="fix" issue="LOG4PHP-115" by="Vaceletm">Instanciate LoggerAppenderPDO by hand throw exception</action>
index 837262f..81e42eb 100644 (file)
@@ -213,13 +213,20 @@ class LoggerLevel {
        }
 
        /**
-        * Returns the string representation of this priority.
+        * Returns the string representation of this level.
         * @return string
-        * @final
         */
        public function toString() {
                return $this->levelStr;
        }
+       
+       /**
+        * Returns the string representation of this level.
+        * @return string
+        */
+       public function __toString() {
+               return $this->toString();
+       }
 
        /**
         * Returns the integer representation of this level.
index fe94afa..b8a56f9 100644 (file)
@@ -145,15 +145,11 @@ class LoggerLayoutHtml extends LoggerLayout {
         $level = $event->getLevel();
         
         if ($level->equals(LoggerLevel::getLevelDebug())) {
-          $sbuf .= "<font color=\"#339933\">";
-          $sbuf .= $level->toString();
-          $sbuf .= "</font>";
+          $sbuf .= "<font color=\"#339933\">$level</font>";
         } else if ($level->equals(LoggerLevel::getLevelWarn())) {
-          $sbuf .= "<font color=\"#993300\"><strong>";
-          $sbuf .= $level->toString();
-          $sbuf .= "</strong></font>";
+          $sbuf .= "<font color=\"#993300\"><strong>$level</strong></font>";
         } else {
-          $sbuf .= $level->toString();
+          $sbuf .= $level;
         }
         $sbuf .= "</td>" . PHP_EOL;
     
index 4cc0351..18f7a61 100644 (file)
@@ -56,6 +56,7 @@ class LoggerLayoutSimple extends LoggerLayout {
      */
     public function format(LoggerLoggingEvent $event) {
         $level = $event->getLevel();
-        return $level->toString() . ' - ' . $event->getRenderedMessage(). PHP_EOL;
+        $message = $event->getRenderedMessage();
+        return "$level - $message" . PHP_EOL;
     }
 }
index 0fce38d..35fdea1 100644 (file)
@@ -190,7 +190,7 @@ class LoggerLayoutTTCC extends LoggerLayout {
         }
         
         $level = $event->getLevel();
-        $format .= $level->toString().' ';
+        $format .= $level.' ';
         
         if($this->categoryPrefixing) {
             $format .= $event->getLoggerName().' ';