bug #48314 [Mime] Fix MessagePart serialization (Amunak)

nicolas-grekas · nicolas-grekas · commit ed1ab3f454df · 2022-12-13T11:06:53.000+01:00
This PR was merged into the 5.4 branch. Discussion ---------- [Mime] Fix MessagePart serialization | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #48313 | License | MIT | Doc PR | n/a Fixes serialization of Symfony\Component\Mime\Part\MessagePart and adds test coverage. Commits ------- 8cb094c bug #48313 [Mime] Fix MessagePart serialization
diff --git a/src/Symfony/Component/Mime/Part/MessagePart.php b/src/Symfony/Component/Mime/Part/MessagePart.php
@@ -59,4 +59,17 @@ public function bodyToIterable(): iterable
     {
         return $this->message->toIterable();
     }
+
+    /**
+     * @return array
+     */
+    public function __sleep()
+    {
+        return ['message'];
+    }
+
+    public function __wakeup()
+    {
+        $this->__construct($this->message);
+    }
 }
diff --git a/src/Symfony/Component/Mime/Tests/Part/MessagePartTest.php b/src/Symfony/Component/Mime/Tests/Part/MessagePartTest.php
@@ -39,4 +39,14 @@ public function testHeaders()
             new ParameterizedHeader('Content-Disposition', 'attachment', ['name' => 'Subject.eml', 'filename' => 'Subject.eml'])
         ), $p->getPreparedHeaders());
     }
+
+    public function testSerialize()
+    {
+        $email = (new Email())->from('fabien@symfony.com')->to('you@example.com')->text('content');
+        $email->getHeaders()->addIdHeader('Message-ID', $email->generateMessageId());
+
+        $p = new MessagePart($email);
+        $expected = clone $p;
+        $this->assertEquals($expected->toString(), unserialize(serialize($p))->toString());
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -59,4 +59,17 @@ public function bodyToIterable(): iterable`
`59`	`59`	`{`
`60`	`60`	`return $this->message->toIterable();`
`61`	`61`	`}`
	`62`	`+`
	`63`	`+ /**`
	`64`	`+ * @return array`
	`65`	`+ */`
	`66`	`+ public function __sleep()`
	`67`	`+ {`
	`68`	`+ return ['message'];`
	`69`	`+ }`
	`70`	`+`
	`71`	`+ public function __wakeup()`
	`72`	`+ {`
	`73`	`+ $this->__construct($this->message);`
	`74`	`+ }`
`62`	`75`	`}`