Просмотр исходного кода

Merge pull request #1402 from dmaicher/issue-1401

FilterHandler: fix handle empty batch of filtered records
Jordi Boggiano 6 лет назад
Родитель
Сommit
ea216b0e2f

+ 3 - 1
src/Monolog/Handler/FilterHandler.php

@@ -128,7 +128,9 @@ class FilterHandler extends AbstractHandler
             }
         }
 
-        $this->getHandler($filtered[count($filtered) - 1])->handleBatch($filtered);
+        if (count($filtered) > 0) {
+            $this->getHandler($filtered[count($filtered) - 1])->handleBatch($filtered);
+        }
     }
 
     /**

+ 8 - 0
tests/Monolog/Handler/FilterHandlerTest.php

@@ -167,4 +167,12 @@ class FilterHandlerTest extends TestCase
         );
         $handler->handle($this->getRecord(Logger::WARNING));
     }
+
+    public function testHandleEmptyBatch()
+    {
+        $test = new TestHandler();
+        $handler = new FilterHandler($test);
+        $handler->handleBatch(array());
+        $this->assertSame(array(), $test->getRecords());
+    }
 }