Merge pull request #404 from adrium/hotfix/404

Register only one error handler in `swallowDeprecationNotices()`

Author: weierophinney

src/Application.php

@@ -677,24 +677,16 @@ private function checkForDuplicateRoute($path, $methods = null)
      */
     private function swallowDeprecationNotices()
     {
-        $handler = function ($errno, $errstr) {
-            if ($errno !== E_USER_DEPRECATED) {
-                return false;
+        $handler = function ($errno, $errstr, $errfile, $errline, $errcontext) use (&$previous) {
+            $swallow = $errno === E_USER_DEPRECATED && false !== strstr($errstr, 'error middleware is deprecated');
+
+            if ($swallow || $previous === null) {
+                return $swallow;
             }
-            return false !== strstr($errstr, 'error middleware is deprecated');
+
+            $previous($errno, $errstr, $errfile, $errline, $errcontext);
         };
 
         $previous = set_error_handler($handler);
-        if (! $previous) {
-            return;
-        }
-
-        set_error_handler(function ($errno, $errstr, $errfile, $errline, $errcontext) use ($handler, $previous) {
-            if ($handler($errno, $errstr)) {
-                return true;
-            }
-
-            return $previous($errno, $errstr, $errfile, $errline, $errcontext);
-        });
     }
 }

test/IntegrationTest.php

@@ -84,9 +84,9 @@ public function testInjectedFinalHandlerCanEmitA404WhenNoMiddlewareMatches()
      */
     public function testErrorMiddlewareDeprecationErrorHandlerWillNotOverridePreviouslyRegisteredErrorHandler()
     {
-        $triggered = false;
+        $triggered = 0;
         $this->errorHandler = set_error_handler(function ($errno, $errstr) use (&$triggered) {
-            $triggered = true;
+            $triggered++;
             return true;
         });
 
@@ -104,6 +104,10 @@ public function testErrorMiddlewareDeprecationErrorHandlerWillNotOverridePreviou
 
         $app->run($request, $response);
 
-        $this->assertTrue($triggered);
+        trigger_error('Triggered', E_USER_NOTICE);
+
+        restore_error_handler();
+
+        $this->assertSame(2, $triggered);
     }
 }