From 4422cdc2b6ef4ff688aebf042401f9dabba8ff1f Mon Sep 17 00:00:00 2001 From: Diego Saint Esteben Date: Fri, 18 Dec 2015 17:08:31 -0300 Subject: [PATCH] Unset the "datetime" parameter in LogglyFormatter --- src/Monolog/Formatter/LogglyFormatter.php | 2 +- tests/Monolog/Formatter/LogglyFormatterTest.php | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/Monolog/Formatter/LogglyFormatter.php b/src/Monolog/Formatter/LogglyFormatter.php index 401859bb..e8debf37 100644 --- a/src/Monolog/Formatter/LogglyFormatter.php +++ b/src/Monolog/Formatter/LogglyFormatter.php @@ -39,7 +39,7 @@ class LogglyFormatter extends JsonFormatter { if (isset($record["datetime"]) && ($record["datetime"] instanceof \DateTime)) { $record["timestamp"] = $record["datetime"]->format("Y-m-d\TH:i:s.uO"); - // TODO 2.0 unset the 'datetime' parameter, retained for BC + unset($record["datetime"]); } return parent::format($record); diff --git a/tests/Monolog/Formatter/LogglyFormatterTest.php b/tests/Monolog/Formatter/LogglyFormatterTest.php index 6d59b3f3..fc22334c 100644 --- a/tests/Monolog/Formatter/LogglyFormatterTest.php +++ b/tests/Monolog/Formatter/LogglyFormatterTest.php @@ -34,6 +34,7 @@ class LogglyFormatterTest extends TestCase $formatter = new LogglyFormatter(); $record = $this->getRecord(); $formatted_decoded = json_decode($formatter->format($record), true); + $this->assertArrayNotHasKey("datetime", $formatted_decoded); $this->assertArrayHasKey("timestamp", $formatted_decoded); $this->assertEquals(new \DateTime($formatted_decoded["timestamp"]), $record["datetime"]); }