Prevent E_USER_DEPRECATED errors from being fatal

The upcoming Twig v3 calls `trigger_error()` with a level of E_USER_DEPRECATED which would be caught by the exception handler and treated as a fatal error. This commit ensures that the error is reported but not shown.

Author: joelambert

src/Bootstrappers/RegisterExceptionHandler.php

@@ -94,7 +94,7 @@ public function handleError($level, $message, $file = '', $line = 0, $context =
     {
         $exception = new ErrorException($message, 0, $level, $file, $line);
 
-        if ($level === E_USER_NOTICE) {
+        if ($level === E_USER_NOTICE || $level === E_USER_DEPRECATED) {
             $this->getExceptionHandler()->report($exception);
             return;
         }

tests/Unit/Bootstrappers/RegisterExceptionHandlerTest.php

@@ -62,6 +62,26 @@ public function E_USER_NOTICE_errors_are_not_converted_to_exceptions()
         $bootstrapper->handleError(E_USER_NOTICE, 'Test Error');
     }
 
+    /**
+     * @test
+     */
+    public function E_USER_DEPRECATED_errors_are_not_converted_to_exceptions()
+    {
+        Functions\expect('is_admin')->once()->andReturn(false);
+
+        $app = new Application;
+        $handler = Mockery::mock(HandlerInterface::class);
+        $app->bind(HandlerInterface::class, $handler);
+
+        $handler->shouldReceive('report')->once()->with(Mockery::on(function ($e) {
+            return $e->getSeverity() === E_USER_DEPRECATED && $e->getMessage() === 'Test Error';
+        }));
+
+        $bootstrapper = new RegisterExceptionHandler();
+        $bootstrapper->bootstrap($app);
+        $bootstrapper->handleError(E_USER_DEPRECATED, 'Test Error');
+    }
+
     /** @test */
     public function handle_exception_should_call_handlers_report_and_render_methods()
     {