From a82bac49f24516fc7a11cbcc1c20fd4b18b95a74 Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Wed, 30 Jan 2013 19:26:25 +0100
Subject: [PATCH 01/10] Added ZendMonitor Handler

---
 src/Monolog/Handler/ZendMonitorHandler.php    | 86 +++++++++++++++++++
 .../Handler/ZendMonitorHandlerTest.php        | 62 +++++++++++++
 2 files changed, 148 insertions(+)
 create mode 100644 src/Monolog/Handler/ZendMonitorHandler.php
 create mode 100644 tests/Monolog/Handler/ZendMonitorHandlerTest.php

diff --git a/src/Monolog/Handler/ZendMonitorHandler.php b/src/Monolog/Handler/ZendMonitorHandler.php
new file mode 100644
index 00000000..0f0fe2b4
--- /dev/null
+++ b/src/Monolog/Handler/ZendMonitorHandler.php
@@ -0,0 +1,86 @@
+<?php
+/*
+ * This file is part of the Monolog package.
+ *
+ * (c) Jordi Boggiano <j.boggiano@seld.be>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Monolog\Handler;
+
+use Monolog\Logger;
+
+/**
+ * Handler sending logs to Zend Monitor
+ *
+ * @author  Christian Bergau <cbergau86@gmail.com>
+ */
+class ZendMonitorHandler extends AbstractProcessingHandler
+{
+    /**
+     * Monolog level / ZendMonitor Custom Event priority map
+     *
+     * @var array
+     */
+    protected $levelMap = array(
+        Logger::DEBUG     => 1,
+        Logger::INFO      => 2,
+        Logger::NOTICE    => 3,
+        Logger::WARNING   => 4,
+        Logger::ERROR     => 5,
+        Logger::CRITICAL  => 6,
+        Logger::ALERT     => 7,
+        Logger::EMERGENCY => 0,
+    );
+
+    /**
+     * Is application running on a zend server?
+     *
+     * @var bool
+     */
+    protected $isZendServer = false;
+
+    /**
+     * Construct
+     *
+     * @param   int     $level
+     * @param   bool    $bubble
+     */
+    public function __construct($level = Logger::DEBUG, $bubble = true)
+    {
+        $this->isZendServer = function_exists('zend_monitor_custom_event');
+        parent::__construct($level, $bubble);
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    protected function write(array $record)
+    {
+        if ($this->isZendServer()) {
+            $this->writeZendMonitorCustomEvent($record);
+        }
+    }
+
+    /**
+     * Write a record to Zend Monitor
+     *
+     * @param   array   $record
+     */
+    protected function writeZendMonitorCustomEvent(array $record)
+    {
+        zend_monitor_custom_event($this->levelMap[$record['level']], $record['message'], $record);
+    }
+
+    /**
+     * Is Zend Server?
+     *
+     * @return bool
+     */
+    public function isZendServer()
+    {
+        return $this->isZendServer;
+    }
+}
diff --git a/tests/Monolog/Handler/ZendMonitorHandlerTest.php b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
new file mode 100644
index 00000000..ed8bdac7
--- /dev/null
+++ b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
@@ -0,0 +1,62 @@
+<?php
+/*
+ * This file is part of the Monolog package.
+ *
+ * (c) Jordi Boggiano <j.boggiano@seld.be>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Monolog\Handler;
+
+use Monolog\Logger;
+use Monolog\TestCase;
+
+class ZendMonitorHandlerTest extends TestCase
+{
+    protected $zendMonitorHandler;
+
+    public function setUp()
+    {
+        if (!function_exists('zend_monitor_custom_event')) {
+            $this->markTestSkipped('ZendServer is not installed');
+        }
+    }
+
+    /**
+     * @covers  \Monolog\Handler\ZendMonitor::__construct
+     * @covers  \Monolog\Handler\ZendMonitor::isZendServer
+     */
+    public function testIsZendServerReturnsTrue()
+    {
+        $zendMonitor = $this->getMockBuilder('Monolog\Handler\ZendMonitorHandler')
+            ->setMethods(null)
+            ->getMock();
+        $this->assertTrue($zendMonitor->isZendServer());
+    }
+
+    /**
+     * @covers  \Monolog\Handler\ZendMonitor::write
+     */
+    public function testWrite()
+    {
+        $zendMonitor = $this->getMockBuilder('Monolog\Handler\ZendMonitorHandler')
+            ->setMethods(array('writeZendMonitorCustomEvent'))
+            ->getMock();
+        $zendMonitor->expects($this->once())
+            ->method('writeZendMonitorCustomEvent');
+        $zendMonitor->handle(
+            array(
+                'message'    => 'addDebug Message',
+                'context'    => array(),
+                'level'      => Logger::DEBUG,
+                'level_name' => 'DEBUG',
+                'channel'    => 'zendmonitor',
+                'extra'      => array(),
+                'formatted'  => '[2013-01-30 19:07:32] zendmonitor.DEBUG: addDebug Message [] []'
+            )
+        );
+    }
+
+}

From 32145bc49e3c78cd7b09eb77d91d14a70935e959 Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Wed, 30 Jan 2013 19:37:42 +0100
Subject: [PATCH 02/10] Removed testing a mock instead of the real handler

---
 .../Handler/ZendMonitorHandlerTest.php        | 122 +++++++++---------
 1 file changed, 60 insertions(+), 62 deletions(-)

diff --git a/tests/Monolog/Handler/ZendMonitorHandlerTest.php b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
index ed8bdac7..70ec9724 100644
--- a/tests/Monolog/Handler/ZendMonitorHandlerTest.php
+++ b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
@@ -1,62 +1,60 @@
-<?php
-/*
- * This file is part of the Monolog package.
- *
- * (c) Jordi Boggiano <j.boggiano@seld.be>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
-
-namespace Monolog\Handler;
-
-use Monolog\Logger;
-use Monolog\TestCase;
-
-class ZendMonitorHandlerTest extends TestCase
-{
-    protected $zendMonitorHandler;
-
-    public function setUp()
-    {
-        if (!function_exists('zend_monitor_custom_event')) {
-            $this->markTestSkipped('ZendServer is not installed');
-        }
-    }
-
-    /**
-     * @covers  \Monolog\Handler\ZendMonitor::__construct
-     * @covers  \Monolog\Handler\ZendMonitor::isZendServer
-     */
-    public function testIsZendServerReturnsTrue()
-    {
-        $zendMonitor = $this->getMockBuilder('Monolog\Handler\ZendMonitorHandler')
-            ->setMethods(null)
-            ->getMock();
-        $this->assertTrue($zendMonitor->isZendServer());
-    }
-
-    /**
-     * @covers  \Monolog\Handler\ZendMonitor::write
-     */
-    public function testWrite()
-    {
-        $zendMonitor = $this->getMockBuilder('Monolog\Handler\ZendMonitorHandler')
-            ->setMethods(array('writeZendMonitorCustomEvent'))
-            ->getMock();
-        $zendMonitor->expects($this->once())
-            ->method('writeZendMonitorCustomEvent');
-        $zendMonitor->handle(
-            array(
-                'message'    => 'addDebug Message',
-                'context'    => array(),
-                'level'      => Logger::DEBUG,
-                'level_name' => 'DEBUG',
-                'channel'    => 'zendmonitor',
-                'extra'      => array(),
-                'formatted'  => '[2013-01-30 19:07:32] zendmonitor.DEBUG: addDebug Message [] []'
-            )
-        );
-    }
-
-}
+<?php
+/*
+ * This file is part of the Monolog package.
+ *
+ * (c) Jordi Boggiano <j.boggiano@seld.be>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Monolog\Handler;
+
+use Monolog\Logger;
+use Monolog\TestCase;
+
+class ZendMonitorHandlerTest extends TestCase
+{
+    protected $zendMonitorHandler;
+
+    public function setUp()
+    {
+        if (!function_exists('zend_monitor_custom_event')) {
+            $this->markTestSkipped('ZendServer is not installed');
+        }
+    }
+
+    /**
+     * @covers  \Monolog\Handler\ZendMonitor::__construct
+     * @covers  \Monolog\Handler\ZendMonitor::isZendServer
+     */
+    public function testIsZendServerReturnsTrue()
+    {
+        $zendMonitor = new ZendMonitorHandler();
+        $this->assertTrue($zendMonitor->isZendServer());
+    }
+
+    /**
+     * @covers  \Monolog\Handler\ZendMonitor::write
+     */
+    public function testWrite()
+    {
+        $zendMonitor = $this->getMockBuilder('Monolog\Handler\ZendMonitorHandler')
+            ->setMethods(array('writeZendMonitorCustomEvent'))
+            ->getMock();
+        $zendMonitor->expects($this->once())
+            ->method('writeZendMonitorCustomEvent');
+        $zendMonitor->handle(
+            array(
+                'message'    => 'addDebug Message',
+                'context'    => array(),
+                'level'      => Logger::DEBUG,
+                'level_name' => 'DEBUG',
+                'channel'    => 'zendmonitor',
+                'extra'      => array(),
+                'formatted'  => '[2013-01-30 19:07:32] zendmonitor.DEBUG: addDebug Message [] []'
+            )
+        );
+    }
+
+}

From 6875c5869a34e773547bb172903ffdf33f425f7d Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Wed, 30 Jan 2013 19:42:13 +0100
Subject: [PATCH 03/10] Added ZendMonitorHandler to the readme

---
 README.mdown | 1 +
 1 file changed, 1 insertion(+)

diff --git a/README.mdown b/README.mdown
index 4489b4f7..ac0ad65c 100644
--- a/README.mdown
+++ b/README.mdown
@@ -126,6 +126,7 @@ Handlers
 - _DoctrineCouchDBHandler_: Logs records to a CouchDB server via the Doctrine CouchDB ODM.
 - _RavenHandler_: Logs records to a [Sentry](http://getsentry.com/) server using
   [raven](https://packagist.org/packages/raven/raven).
+- _ZendMonitorHandler_: Logs records to the ZendMonitor.
 
 Wrappers / Special Handlers
 ---------------------------

From ea009c091b57705a971104839aaabb797b7684c4 Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Wed, 30 Jan 2013 19:45:54 +0100
Subject: [PATCH 04/10] Format record with NormalizerFormatter  before writing
 to ZendMonitor.

---
 src/Monolog/Handler/ZendMonitorHandler.php | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/Monolog/Handler/ZendMonitorHandler.php b/src/Monolog/Handler/ZendMonitorHandler.php
index 0f0fe2b4..d8892c93 100644
--- a/src/Monolog/Handler/ZendMonitorHandler.php
+++ b/src/Monolog/Handler/ZendMonitorHandler.php
@@ -10,6 +10,7 @@
 
 namespace Monolog\Handler;
 
+use Monolog\Formatter\NormalizerFormatter;
 use Monolog\Logger;
 
 /**
@@ -60,7 +61,8 @@ class ZendMonitorHandler extends AbstractProcessingHandler
     protected function write(array $record)
     {
         if ($this->isZendServer()) {
-            $this->writeZendMonitorCustomEvent($record);
+            $formatter = new NormalizerFormatter();
+            $this->writeZendMonitorCustomEvent($formatter->format($record));
         }
     }
 
@@ -71,7 +73,7 @@ class ZendMonitorHandler extends AbstractProcessingHandler
      */
     protected function writeZendMonitorCustomEvent(array $record)
     {
-        zend_monitor_custom_event($this->levelMap[$record['level']], $record['message'], $record);
+        zend_monitor_custom_event($this->levelMap[$record['level']], $record['message'], $record['formatted']);
     }
 
     /**

From 4eb6011dc1e387e55d1467aad3df4e2b5174c302 Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Wed, 30 Jan 2013 20:17:14 +0100
Subject: [PATCH 05/10] Use getRecord to get a record in unit test

---
 tests/Monolog/Handler/ZendMonitorHandlerTest.php | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/tests/Monolog/Handler/ZendMonitorHandlerTest.php b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
index 70ec9724..2b249237 100644
--- a/tests/Monolog/Handler/ZendMonitorHandlerTest.php
+++ b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
@@ -44,17 +44,7 @@ class ZendMonitorHandlerTest extends TestCase
             ->getMock();
         $zendMonitor->expects($this->once())
             ->method('writeZendMonitorCustomEvent');
-        $zendMonitor->handle(
-            array(
-                'message'    => 'addDebug Message',
-                'context'    => array(),
-                'level'      => Logger::DEBUG,
-                'level_name' => 'DEBUG',
-                'channel'    => 'zendmonitor',
-                'extra'      => array(),
-                'formatted'  => '[2013-01-30 19:07:32] zendmonitor.DEBUG: addDebug Message [] []'
-            )
-        );
+        $zendMonitor->handle($this->getRecord());
     }
 
 }

From b4b2d91c670a6daa4e2a79cb80ed0e7b8837844e Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Wed, 30 Jan 2013 21:42:59 +0100
Subject: [PATCH 06/10] Refactored writeZendMonitorCustomEvent signature to
 provide better testability

---
 src/Monolog/Handler/ZendMonitorHandler.php       | 10 ++++++----
 tests/Monolog/Handler/ZendMonitorHandlerTest.php | 13 +++++++++----
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/src/Monolog/Handler/ZendMonitorHandler.php b/src/Monolog/Handler/ZendMonitorHandler.php
index d8892c93..71962f19 100644
--- a/src/Monolog/Handler/ZendMonitorHandler.php
+++ b/src/Monolog/Handler/ZendMonitorHandler.php
@@ -62,18 +62,20 @@ class ZendMonitorHandler extends AbstractProcessingHandler
     {
         if ($this->isZendServer()) {
             $formatter = new NormalizerFormatter();
-            $this->writeZendMonitorCustomEvent($formatter->format($record));
+            $record = $formatter->format($record);
+            $this->writeZendMonitorCustomEvent($record['level'], $record['message']);
         }
     }
 
     /**
      * Write a record to Zend Monitor
      *
-     * @param   array   $record
+     * @param   int     $level
+     * @param   string  $message
      */
-    protected function writeZendMonitorCustomEvent(array $record)
+    protected function writeZendMonitorCustomEvent($level, $message)
     {
-        zend_monitor_custom_event($this->levelMap[$record['level']], $record['message'], $record['formatted']);
+        zend_monitor_custom_event($level, $message);
     }
 
     /**
diff --git a/tests/Monolog/Handler/ZendMonitorHandlerTest.php b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
index 2b249237..2e827825 100644
--- a/tests/Monolog/Handler/ZendMonitorHandlerTest.php
+++ b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
@@ -39,12 +39,17 @@ class ZendMonitorHandlerTest extends TestCase
      */
     public function testWrite()
     {
+        $record = $this->getRecord();
+
         $zendMonitor = $this->getMockBuilder('Monolog\Handler\ZendMonitorHandler')
-            ->setMethods(array('writeZendMonitorCustomEvent'))
-            ->getMock();
+                            ->setMethods(array('writeZendMonitorCustomEvent'))
+                            ->getMock();
+
         $zendMonitor->expects($this->once())
-            ->method('writeZendMonitorCustomEvent');
-        $zendMonitor->handle($this->getRecord());
+                    ->method('writeZendMonitorCustomEvent')
+                    ->with($record['level'], $record['message']);
+
+        $zendMonitor->handle($record);
     }
 
 }

From a2162c1787f9c5ed4e2b2ef9cba793d095514a9c Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Thu, 31 Jan 2013 07:22:52 +0100
Subject: [PATCH 07/10] Override getDefaultFormatter instead of creating a
 formatter with 'new' in method 'write'. Using levelMap again. Refactored
 test.

---
 src/Monolog/Handler/ZendMonitorHandler.php    | 22 +++++++++++++++++--
 .../Handler/ZendMonitorHandlerTest.php        |  4 +++-
 2 files changed, 23 insertions(+), 3 deletions(-)

diff --git a/src/Monolog/Handler/ZendMonitorHandler.php b/src/Monolog/Handler/ZendMonitorHandler.php
index 71962f19..1e9741e8 100644
--- a/src/Monolog/Handler/ZendMonitorHandler.php
+++ b/src/Monolog/Handler/ZendMonitorHandler.php
@@ -61,9 +61,9 @@ class ZendMonitorHandler extends AbstractProcessingHandler
     protected function write(array $record)
     {
         if ($this->isZendServer()) {
-            $formatter = new NormalizerFormatter();
+            $formatter = $this->getDefaultFormatter();
             $record = $formatter->format($record);
-            $this->writeZendMonitorCustomEvent($record['level'], $record['message']);
+            $this->writeZendMonitorCustomEvent($this->levelMap[$record['level']], $record['message']);
         }
     }
 
@@ -78,6 +78,24 @@ class ZendMonitorHandler extends AbstractProcessingHandler
         zend_monitor_custom_event($level, $message);
     }
 
+    /**
+     * {@inheritdoc}
+     */
+    public function getDefaultFormatter()
+    {
+        return new NormalizerFormatter();
+    }
+
+    /**
+     * Get the level map
+     *
+     * @return array
+     */
+    public function getLevelMap()
+    {
+        return $this->levelMap;
+    }
+
     /**
      * Is Zend Server?
      *
diff --git a/tests/Monolog/Handler/ZendMonitorHandlerTest.php b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
index 2e827825..a599e65e 100644
--- a/tests/Monolog/Handler/ZendMonitorHandlerTest.php
+++ b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
@@ -45,9 +45,11 @@ class ZendMonitorHandlerTest extends TestCase
                             ->setMethods(array('writeZendMonitorCustomEvent'))
                             ->getMock();
 
+        $levelMap = $zendMonitor->getLevelMap();
+
         $zendMonitor->expects($this->once())
                     ->method('writeZendMonitorCustomEvent')
-                    ->with($record['level'], $record['message']);
+                    ->with($levelMap[$record['level']], $record['message']);
 
         $zendMonitor->handle($record);
     }

From 5a2a9129d4b99214404d16afb9baf0878d5ee2b7 Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Thu, 31 Jan 2013 07:24:14 +0100
Subject: [PATCH 08/10] Added test for getDefaultFormatter

---
 tests/Monolog/Handler/ZendMonitorHandlerTest.php | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/tests/Monolog/Handler/ZendMonitorHandlerTest.php b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
index a599e65e..e88aacf2 100644
--- a/tests/Monolog/Handler/ZendMonitorHandlerTest.php
+++ b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
@@ -54,4 +54,9 @@ class ZendMonitorHandlerTest extends TestCase
         $zendMonitor->handle($record);
     }
 
+    public function testGetDefaultFormatterReturnsNormalizerFormatter()
+    {
+        $zendMonitor = new ZendMonitorHandler();
+        $this->assertInstanceOf('Monolog\Formatter\NormalizerFormatter', $zendMonitor->getDefaultFormatter());
+    }
 }

From 83b84ff8188d9af5fa6d903f21cc8b0a25df6b4b Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Thu, 31 Jan 2013 10:39:11 +0100
Subject: [PATCH 09/10] Pass $record['formatted'] as the user parameter for
 zend monitor. Refactored test.

---
 src/Monolog/Handler/ZendMonitorHandler.php    | 13 +++++++----
 .../Handler/ZendMonitorHandlerTest.php        | 23 +++++++++++++++----
 2 files changed, 27 insertions(+), 9 deletions(-)

diff --git a/src/Monolog/Handler/ZendMonitorHandler.php b/src/Monolog/Handler/ZendMonitorHandler.php
index 1e9741e8..770cb13a 100644
--- a/src/Monolog/Handler/ZendMonitorHandler.php
+++ b/src/Monolog/Handler/ZendMonitorHandler.php
@@ -61,9 +61,11 @@ class ZendMonitorHandler extends AbstractProcessingHandler
     protected function write(array $record)
     {
         if ($this->isZendServer()) {
-            $formatter = $this->getDefaultFormatter();
-            $record = $formatter->format($record);
-            $this->writeZendMonitorCustomEvent($this->levelMap[$record['level']], $record['message']);
+            $this->writeZendMonitorCustomEvent(
+                $this->levelMap[$record['level']],
+                $record['message'],
+                $record['formatted']
+            );
         }
     }
 
@@ -72,10 +74,11 @@ class ZendMonitorHandler extends AbstractProcessingHandler
      *
      * @param   int     $level
      * @param   string  $message
+     * @param   array   $formatted
      */
-    protected function writeZendMonitorCustomEvent($level, $message)
+    protected function writeZendMonitorCustomEvent($level, $message, $formatted)
     {
-        zend_monitor_custom_event($level, $message);
+        zend_monitor_custom_event($level, $message, $formatted);
     }
 
     /**
diff --git a/tests/Monolog/Handler/ZendMonitorHandlerTest.php b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
index e88aacf2..6d441c6b 100644
--- a/tests/Monolog/Handler/ZendMonitorHandlerTest.php
+++ b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
@@ -40,16 +40,31 @@ class ZendMonitorHandlerTest extends TestCase
     public function testWrite()
     {
         $record = $this->getRecord();
+        $formatterResult = array(
+            'message' => $record['message']
+        );
 
         $zendMonitor = $this->getMockBuilder('Monolog\Handler\ZendMonitorHandler')
-                            ->setMethods(array('writeZendMonitorCustomEvent'))
-                            ->getMock();
+            ->setMethods(array('writeZendMonitorCustomEvent', 'getDefaultFormatter'))
+            ->getMock();
+
+        $formatterMock = $this->getMockBuilder('Monolog\Formatter\NormalizerFormatter')
+            ->disableOriginalConstructor()
+            ->getMock();
+
+        $formatterMock->expects($this->once())
+            ->method('format')
+            ->will($this->returnValue($formatterResult));
+
+        $zendMonitor->expects($this->once())
+            ->method('getDefaultFormatter')
+            ->will($this->returnValue($formatterMock));
 
         $levelMap = $zendMonitor->getLevelMap();
 
         $zendMonitor->expects($this->once())
-                    ->method('writeZendMonitorCustomEvent')
-                    ->with($levelMap[$record['level']], $record['message']);
+            ->method('writeZendMonitorCustomEvent')
+            ->with($levelMap[$record['level']], $record['message'], $formatterResult);
 
         $zendMonitor->handle($record);
     }

From f0ba64ad4b8b9ad235f422c8d30591d0705e3146 Mon Sep 17 00:00:00 2001
From: ChristianB <cbergau86@gmail.com>
Date: Thu, 31 Jan 2013 12:16:16 +0100
Subject: [PATCH 10/10] Exception will be thrown if ZendMonitorHandler is
 instantiated without having Zend Server installed.

---
 .../Handler/MissingExtensionException.php     | 22 ++++++++++++
 src/Monolog/Handler/ZendMonitorHandler.php    | 34 +++++--------------
 .../Handler/ZendMonitorHandlerTest.php        | 15 +++-----
 3 files changed, 35 insertions(+), 36 deletions(-)
 create mode 100644 src/Monolog/Handler/MissingExtensionException.php

diff --git a/src/Monolog/Handler/MissingExtensionException.php b/src/Monolog/Handler/MissingExtensionException.php
new file mode 100644
index 00000000..0cb21cd2
--- /dev/null
+++ b/src/Monolog/Handler/MissingExtensionException.php
@@ -0,0 +1,22 @@
+<?php
+
+/*
+ * This file is part of the Monolog package.
+ *
+ * (c) Jordi Boggiano <j.boggiano@seld.be>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Monolog\Handler;
+
+/**
+ * Exception can be thrown if an extension for an handler is missing
+ *
+ * @author  Christian Bergau <cbergau86@gmail.com>
+ */
+class MissingExtensionException extends \Exception
+{
+
+}
diff --git a/src/Monolog/Handler/ZendMonitorHandler.php b/src/Monolog/Handler/ZendMonitorHandler.php
index 770cb13a..0a264231 100644
--- a/src/Monolog/Handler/ZendMonitorHandler.php
+++ b/src/Monolog/Handler/ZendMonitorHandler.php
@@ -36,22 +36,18 @@ class ZendMonitorHandler extends AbstractProcessingHandler
         Logger::EMERGENCY => 0,
     );
 
-    /**
-     * Is application running on a zend server?
-     *
-     * @var bool
-     */
-    protected $isZendServer = false;
-
     /**
      * Construct
      *
      * @param   int     $level
      * @param   bool    $bubble
+     * @throws  MissingExtensionException
      */
     public function __construct($level = Logger::DEBUG, $bubble = true)
     {
-        $this->isZendServer = function_exists('zend_monitor_custom_event');
+        if (!function_exists('zend_monitor_custom_event')) {
+            throw new MissingExtensionException('You must have Zend Server installed in order to use this handler');
+        }
         parent::__construct($level, $bubble);
     }
 
@@ -60,13 +56,11 @@ class ZendMonitorHandler extends AbstractProcessingHandler
      */
     protected function write(array $record)
     {
-        if ($this->isZendServer()) {
-            $this->writeZendMonitorCustomEvent(
-                $this->levelMap[$record['level']],
-                $record['message'],
-                $record['formatted']
-            );
-        }
+        $this->writeZendMonitorCustomEvent(
+            $this->levelMap[$record['level']],
+            $record['message'],
+            $record['formatted']
+        );
     }
 
     /**
@@ -98,14 +92,4 @@ class ZendMonitorHandler extends AbstractProcessingHandler
     {
         return $this->levelMap;
     }
-
-    /**
-     * Is Zend Server?
-     *
-     * @return bool
-     */
-    public function isZendServer()
-    {
-        return $this->isZendServer;
-    }
 }
diff --git a/tests/Monolog/Handler/ZendMonitorHandlerTest.php b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
index 6d441c6b..ea2d90ce 100644
--- a/tests/Monolog/Handler/ZendMonitorHandlerTest.php
+++ b/tests/Monolog/Handler/ZendMonitorHandlerTest.php
@@ -25,17 +25,7 @@ class ZendMonitorHandlerTest extends TestCase
     }
 
     /**
-     * @covers  \Monolog\Handler\ZendMonitor::__construct
-     * @covers  \Monolog\Handler\ZendMonitor::isZendServer
-     */
-    public function testIsZendServerReturnsTrue()
-    {
-        $zendMonitor = new ZendMonitorHandler();
-        $this->assertTrue($zendMonitor->isZendServer());
-    }
-
-    /**
-     * @covers  \Monolog\Handler\ZendMonitor::write
+     * @covers  Monolog\Handler\ZendMonitorHandler::write
      */
     public function testWrite()
     {
@@ -69,6 +59,9 @@ class ZendMonitorHandlerTest extends TestCase
         $zendMonitor->handle($record);
     }
 
+    /**
+     * @covers Monolog\Handler\ZendMonitorHandler::getDefaultFormatter
+     */
     public function testGetDefaultFormatterReturnsNormalizerFormatter()
     {
         $zendMonitor = new ZendMonitorHandler();