feat: set charset to output of preview (#128)

tinect · frosh-automation[bot] · web-flow · commit 458a188e7001 · 2025-10-02T09:52:59.000+02:00
Co-authored-by: frosh-automation &lt;188718289+frosh-automation[bot]@users.noreply.github.com&gt;
diff --git a/src/Controller/Api/MailArchiveController.php b/src/Controller/Api/MailArchiveController.php
@@ -43,8 +43,7 @@ public function __construct(
         private readonly AbstractMailSender $mailSender,
         private readonly RequestStack $requestStack,
         private readonly EmlFileManager $emlFileManager,
-    ) {
-    }
+    ) {}
 
     #[Route(path: '/api/_action/frosh-mail-archive/resend-mail', name: 'api.action.frosh-mail-archive.resend-mail')]
     public function resend(Request $request, Context $context): JsonResponse
diff --git a/src/Extension/Checkout/Order/OrderExtension.php b/src/Extension/Checkout/Order/OrderExtension.php
@@ -1,4 +1,6 @@
-<?php declare(strict_types=1);
+<?php
+
+declare(strict_types=1);
 
 namespace Frosh\MailArchive\Extension\Checkout\Order;
 
diff --git a/src/Extension/Content/Flow/FlowExtension.php b/src/Extension/Content/Flow/FlowExtension.php
@@ -1,4 +1,6 @@
-<?php declare(strict_types=1);
+<?php
+
+declare(strict_types=1);
 
 namespace Frosh\MailArchive\Extension\Content\Flow;
 
diff --git a/src/Migration/Migration1598204175SenderToJson.php b/src/Migration/Migration1598204175SenderToJson.php
@@ -21,7 +21,5 @@ public function update(Connection $connection): void
 CHANGE `sender` `sender` json NOT NULL AFTER `id`;');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1690743548AddEmlPath.php b/src/Migration/Migration1690743548AddEmlPath.php
@@ -20,7 +20,5 @@ public function update(Connection $connection): void
                                             ADD `eml_path` varchar(2048) NULL;');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1691326842Attachment.php b/src/Migration/Migration1691326842Attachment.php
@@ -31,7 +31,5 @@ public function update(Connection $connection): void
 ');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1694604822AddSourceMailId.php b/src/Migration/Migration1694604822AddSourceMailId.php
@@ -19,7 +19,5 @@ public function update(Connection $connection): void
         $connection->executeStatement('ALTER TABLE `frosh_mail_archive` ADD `source_mail_id` BINARY(16) NULL;');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1694714751TransportInfo.php b/src/Migration/Migration1694714751TransportInfo.php
@@ -19,7 +19,5 @@ public function update(Connection $connection): void
         $connection->executeStatement('ALTER TABLE `frosh_mail_archive` ADD `transport_state` VARCHAR(255) NULL;');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1707002823DropEmlFiled.php b/src/Migration/Migration1707002823DropEmlFiled.php
@@ -19,7 +19,5 @@ public function update(Connection $connection): void
         $connection->executeStatement('ALTER TABLE `frosh_mail_archive` DROP COLUMN `eml`');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1713775973AlterSubjectFieldLength.php b/src/Migration/Migration1713775973AlterSubjectFieldLength.php
@@ -19,7 +19,5 @@ public function update(Connection $connection): void
         $connection->executeStatement('ALTER TABLE `frosh_mail_archive` MODIFY `subject` VARCHAR(998) NOT NULL;');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1739273849MigrateResentState.php b/src/Migration/Migration1739273849MigrateResentState.php
@@ -34,7 +34,5 @@ public function update(Connection $connection): void
         }
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1739730285AddOrderId.php b/src/Migration/Migration1739730285AddOrderId.php
@@ -24,7 +24,5 @@ public function update(Connection $connection): void
         ');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1739731953DropCustomerFK.php b/src/Migration/Migration1739731953DropCustomerFK.php
@@ -22,7 +22,5 @@ public function update(Connection $connection): void
         ');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Migration/Migration1739741754AddFlowId.php b/src/Migration/Migration1739741754AddFlowId.php
@@ -21,7 +21,5 @@ public function update(Connection $connection): void
             ADD COLUMN `flow_id` BINARY(16) NULL;');
     }
 
-    public function updateDestructive(Connection $connection): void
-    {
-    }
+    public function updateDestructive(Connection $connection): void {}
 }
diff --git a/src/Resources/app/administration/src/module/frosh-mail-archive/page/frosh-mail-archive-detail/index.js b/src/Resources/app/administration/src/module/frosh-mail-archive/page/frosh-mail-archive-detail/index.js
@@ -123,17 +123,12 @@ Component.register('frosh-mail-archive-detail', {
                 });
         },
         getContent(html) {
+            const binary = new TextEncoder().encode(html);
+            let result = '';
+            binary.forEach(b => result += String.fromCharCode(b));
+
             return (
-                'data:text/html;base64,' +
-                btoa(
-                    unescape(
-                        encodeURIComponent(
-                            html.replace(/[\u00A0-\u2666]/g, function (c) {
-                                return '&#' + c.charCodeAt(0) + ';';
-                            })
-                        )
-                    )
-                )
+                'data:text/html;base64;charset=utf-8,' + btoa(result)
             );
         },
         openCustomer() {
diff --git a/src/Services/EmlFileManager.php b/src/Services/EmlFileManager.php
@@ -17,8 +17,7 @@ class EmlFileManager
     public function __construct(
         #[Autowire(service: 'frosh_platform_mail_archive.filesystem.private')]
         private readonly FilesystemOperator $filesystem,
-    ) {
-    }
+    ) {}
 
     /**
      * @return string Path to the eml file
diff --git a/src/Services/MailSender.php b/src/Services/MailSender.php
@@ -46,8 +46,7 @@ public function __construct(
         private readonly EntityRepository $froshMailArchiveRepository,
         private readonly EntityRepository $customerRepository,
         private readonly EmlFileManager $emlFileManager,
-    ) {
-    }
+    ) {}
 
     public function send(Email $email, ?Envelope $envelope = null): void
     {
@@ -141,7 +140,7 @@ private function getCustomerIdByMail(array $to): ?string
         $criteria = new Criteria();
 
         /** @var list<string> $addresses */
-        $addresses = \array_map(fn (Address $mail) => $mail->getAddress(), $to);
+        $addresses = \array_map(fn(Address $mail) => $mail->getAddress(), $to);
 
         $criteria->addFilter(new EqualsAnyFilter('email', $addresses));
 
diff --git a/src/Subscriber/MailArchiveDeleteSubscriber.php b/src/Subscriber/MailArchiveDeleteSubscriber.php
@@ -21,8 +21,7 @@ class MailArchiveDeleteSubscriber implements EventSubscriberInterface
     public function __construct(
         private readonly EntityRepository $froshMailArchiveRepository,
         private readonly EmlFileManager $emlFileManager,
-    ) {
-    }
+    ) {}
 
     public static function getSubscribedEvents(): array
     {
diff --git a/src/Subscriber/MailBeforeSentSubscriber.php b/src/Subscriber/MailBeforeSentSubscriber.php
@@ -1,4 +1,6 @@
-<?php declare(strict_types=1);
+<?php
+
+declare(strict_types=1);
 
 namespace Frosh\MailArchive\Subscriber;
 
diff --git a/src/Subscriber/MailTransportSubscriber.php b/src/Subscriber/MailTransportSubscriber.php
@@ -1,4 +1,6 @@
-<?php declare(strict_types=1);
+<?php
+
+declare(strict_types=1);
 
 namespace Frosh\MailArchive\Subscriber;
 
@@ -24,8 +26,7 @@
     public function __construct(
         private EntityRepository $froshMailArchiveRepository,
         private EmlFileManager $emlFileManager,
-    ) {
-    }
+    ) {}
 
     public static function getSubscribedEvents(): array
     {
@@ -77,7 +78,7 @@ private function getAttachments(Email $message): array
     {
         $attachments = $message->getAttachments();
 
-        return array_map(static fn (DataPart $attachment) => [
+        return array_map(static fn(DataPart $attachment) => [
             'fileName' => $attachment->getFilename() ?? 'attachment',
             'contentType' => $attachment->getContentType(),
             'fileSize' => \strlen($attachment->getBody()),
diff --git a/src/Subscriber/SendMailActionSubscriber.php b/src/Subscriber/SendMailActionSubscriber.php
@@ -1,4 +1,6 @@
-<?php declare(strict_types=1);
+<?php
+
+declare(strict_types=1);
 
 namespace Frosh\MailArchive\Subscriber;
 

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,5 @@ public function update(Connection $connection): void`
`21`	`21`	CHANGE `sender` `sender` json NOT NULL AFTER `id`;');
`22`	`22`	`}`
`23`	`23`
`24`		`- public function updateDestructive(Connection $connection): void`
`25`		`- {`
`26`		`- }`
	`24`	`+ public function updateDestructive(Connection $connection): void {}`
`27`	`25`	`}`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,5 @@ public function update(Connection $connection): void`
`20`	`20`	ADD `eml_path` varchar(2048) NULL;');
`21`	`21`	`}`
`22`	`22`
`23`		`- public function updateDestructive(Connection $connection): void`
`24`		`- {`
`25`		`- }`
	`23`	`+ public function updateDestructive(Connection $connection): void {}`
`26`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,5 @@ public function update(Connection $connection): void`
`31`	`31`	`');`
`32`	`32`	`}`
`33`	`33`
`34`		`- public function updateDestructive(Connection $connection): void`
`35`		`- {`
`36`		`- }`
	`34`	`+ public function updateDestructive(Connection $connection): void {}`
`37`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,5 @@ public function update(Connection $connection): void`
`19`	`19`	$connection->executeStatement('ALTER TABLE `frosh_mail_archive` ADD `source_mail_id` BINARY(16) NULL;');
`20`	`20`	`}`
`21`	`21`
`22`		`- public function updateDestructive(Connection $connection): void`
`23`		`- {`
`24`		`- }`
	`22`	`+ public function updateDestructive(Connection $connection): void {}`
`25`	`23`	`}`