Fix serialization implementation

Seldaek · Feb 6, 2023 · c8b1e0e · c8b1e0e
1 parent 3e92e9d
commit c8b1e0e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Monolog/Logger.php b/src/Monolog/Logger.php
@@ -741,7 +741,7 @@ public function __unserialize(array $data): void
     {
         foreach (['name', 'handlers', 'processors', 'microsecondTimestamps', 'timezone', 'exceptionHandler', 'logDepth', 'detectCycles'] as $property) {
             if (isset($data[$property])) {
-                $this->$property = $data;
+                $this->$property = $data[$property];
             }
         }
 

diff --git a/tests/Monolog/LoggerTest.php b/tests/Monolog/LoggerTest.php
@@ -700,7 +700,11 @@ public function testCustomHandleException()
     public function testSerializable()
     {
         $logger = new Logger(__METHOD__);
-        self::assertInstanceOf(Logger::class, unserialize(serialize($logger)));
+        $copy = unserialize(serialize($logger));
+        self::assertInstanceOf(Logger::class, $copy);
+        self::assertSame($logger->getName(), $copy->getName());
+        self::assertSame($logger->getTimezone()->getName(), $copy->getTimezone()->getName());
+        self::assertSame($logger->getHandlers(), $copy->getHandlers());
     }
 
     public function testReset()