fix: keep localized mail subject unchanged

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>

Author: vitormattos

lib/Service/MailService.php

@@ -42,41 +42,13 @@ private function getFileById(int $fileId): File {
 		return $this->files[$fileId];
 	}
 
-	private function encodeMailSubject(string $subject): string {
-		if (preg_match('/^[\x20-\x7E]+$/', $subject) === 1) {
-			return $subject;
-		}
-
-		if (function_exists('mb_encode_mimeheader')) {
-			$encoded = mb_encode_mimeheader($subject, 'UTF-8', 'B', "\r\n");
-			if (is_string($encoded) && $encoded !== '') {
-				return str_replace(["\r", "\n"], '', $encoded);
-			}
-		}
-
-		if (function_exists('iconv_mime_encode')) {
-			$encoded = iconv_mime_encode('Subject', $subject, [
-				'scheme' => 'B',
-				'input-charset' => 'UTF-8',
-				'output-charset' => 'UTF-8',
-				'line-length' => 76,
-				'line-break-chars' => "\r\n",
-			]);
-			if (is_string($encoded) && str_starts_with($encoded, 'Subject: ')) {
-				return str_replace(["\r", "\n"], '', substr($encoded, 9));
-			}
-		}
-
-		return $subject;
-	}
-
 	/**
 	 * @psalm-suppress MixedMethodCall
 	 */
 	public function notifySignDataUpdated(SignRequest $data, string $email, ?string $description = null): void {
 		$emailTemplate = $this->mailer->createEMailTemplate('settings.TestEmail');
 		// TRANSLATORS The subject of the email that is sent after changes are made to the signature request that may affect something for the signer who will sign the document. Some possible reasons: URL for signature changed (when the URL expires), the person who requested the signature sent a notification
-		$emailTemplate->setSubject($this->encodeMailSubject($this->l10n->t('LibreSign: Changes into a file for you to sign')));
+		$emailTemplate->setSubject($this->l10n->t('LibreSign: Changes into a file for you to sign'));
 		$emailTemplate->addHeader();
 		$emailTemplate->addHeading($this->l10n->t('File to sign'), false);
 
@@ -112,7 +84,7 @@ public function notifySignDataUpdated(SignRequest $data, string $email, ?string
 	 */
 	public function notifyUnsignedUser(SignRequest $data, string $email, ?string $description = null): void {
 		$emailTemplate = $this->mailer->createEMailTemplate('settings.TestEmail');
-		$emailTemplate->setSubject($this->encodeMailSubject($this->l10n->t('LibreSign: There is a file for you to sign')));
+		$emailTemplate->setSubject($this->l10n->t('LibreSign: There is a file for you to sign'));
 		$emailTemplate->addHeader();
 		$emailTemplate->addHeading($this->l10n->t('File to sign'), false);
 
@@ -146,7 +118,7 @@ public function notifyUnsignedUser(SignRequest $data, string $email, ?string $de
 	public function notifySignedUser(SignRequest $signRequest, string $email, File $libreSignFile, string $displayName): void {
 		$emailTemplate = $this->mailer->createEMailTemplate('settings.TestEmail');
 		// TRANSLATORS The subject of the email that is sent after a document has been signed by a user. This email is sent to the person who requested the signature.
-		$emailTemplate->setSubject($this->encodeMailSubject($this->l10n->t('LibreSign: A file has been signed')));
+		$emailTemplate->setSubject($this->l10n->t('LibreSign: A file has been signed'));
 		$emailTemplate->addHeader();
 		$emailTemplate->addHeading($this->l10n->t('File signed'), false);
 		// TRANSLATORS The text in the email that is sent after a document has been signed by a user. %s will be replaced with the name of the user who signed the document.
@@ -175,7 +147,7 @@ public function notifySignedUser(SignRequest $signRequest, string $email, File $
 	public function notifyCanceledRequest(SignRequest $signRequest, string $email, File $libreSignFile): void {
 		$emailTemplate = $this->mailer->createEMailTemplate('settings.TestEmail');
 		// TRANSLATORS The subject of the email that is sent when a signature request has been canceled.
-		$emailTemplate->setSubject($this->encodeMailSubject($this->l10n->t('LibreSign: A signature request has been canceled')));
+		$emailTemplate->setSubject($this->l10n->t('LibreSign: A signature request has been canceled'));
 		$emailTemplate->addHeader();
 		$emailTemplate->addHeading($this->l10n->t('Signature request canceled'), false);
 		// TRANSLATORS The text in the email that is sent when a signature request has been canceled. %s will be replaced with the name of the file.
@@ -197,7 +169,7 @@ public function notifyCanceledRequest(SignRequest $signRequest, string $email, F
 
 	public function sendCodeToSign(string $email, string $name, string $code): void {
 		$emailTemplate = $this->mailer->createEMailTemplate('settings.TestEmail');
-		$emailTemplate->setSubject($this->encodeMailSubject($this->l10n->t('LibreSign: Code to sign file')));
+		$emailTemplate->setSubject($this->l10n->t('LibreSign: Code to sign file'));
 		$emailTemplate->addHeader();
 		$emailTemplate->addBodyText($this->l10n->t('Use this code to sign the document:'));
 		$emailTemplate->addBodyText($code);

tests/php/Unit/Service/MailServiceTest.php

@@ -116,7 +116,7 @@ public function testFailToSendMailToUnsignedUser():void {
 		$this->assertNull($actual);
 	}
 
-	public function testSendCodeToSignEncodesAccentedSubjectAsMimeHeader(): void {
+	public function testSendCodeToSignUsesLocalizedSubjectWithoutMutation(): void {
 		$l10n = $this->createMock(IL10N::class);
 		$l10n
 			->method('t')
@@ -139,12 +139,7 @@ public function testSendCodeToSignEncodesAccentedSubjectAsMimeHeader(): void {
 		$emailTemplate
 			->expects($this->once())
 			->method('setSubject')
-			->with($this->callback(static function (string $subject): bool {
-				if (preg_match('/^[\x20-\x7E]+$/', $subject) !== 1) {
-					return false;
-				}
-				return str_contains($subject, '=?UTF-8?B?') || str_contains($subject, '=?UTF-8?Q?');
-			}));
+			->with('LibreSign : Code nécessaire à la signature du fichier');
 		$emailTemplate
 			->expects($this->once())
 			->method('addHeader');