From 1161fcef20b482868b1b37e07cf6217c53c3622c Mon Sep 17 00:00:00 2001
From: Pavel Dohnal <pavel.dohnal@automattic.com>
Date: Wed, 18 Dec 2024 11:25:22 +0100
Subject: [PATCH] Display personalization tags properly in listings

[MAILPOET-6385]
---
 .../NewslettersResponseBuilder.php            | 24 ++++++++++++++++++-
 .../NewslettersResponseBuilderTest.php        | 18 ++++++++++++++
 2 files changed, 41 insertions(+), 1 deletion(-)

diff --git a/mailpoet/lib/API/JSON/ResponseBuilders/NewslettersResponseBuilder.php b/mailpoet/lib/API/JSON/ResponseBuilders/NewslettersResponseBuilder.php
index 1bbd235207..cf4c72e399 100644
--- a/mailpoet/lib/API/JSON/ResponseBuilders/NewslettersResponseBuilder.php
+++ b/mailpoet/lib/API/JSON/ResponseBuilders/NewslettersResponseBuilder.php
@@ -113,6 +113,28 @@ class NewslettersResponseBuilder {
     return $data;
   }
 
+  private function processPersonalizationTags(string $content): string {
+    if (strpos($content, '<!--') === false) {
+      // we don't need to parse anything if there are no personalization tags
+      return $content;
+    }
+    if (!class_exists('\MailPoet\EmailEditor\Engine\PersonalizationTags\HTML_Tag_Processor')) {
+      // editor is not active, we cannot process personalization tags
+      return $content;
+    }
+
+    $content_processor = new \MailPoet\EmailEditor\Engine\PersonalizationTags\HTML_Tag_Processor($content);
+    while ($content_processor->next_token()) {
+      $type = $content_processor->get_token_type();
+      if ($type === '#comment') {
+        $token = $content_processor->get_modifiable_text();
+        $content_processor->replace_token($token);
+      }
+    }
+    $content_processor->flush_updates();
+    return $content_processor->get_updated_html();
+  }
+
   /**
    * @param NewsletterEntity[] $newsletters
    * @return mixed[]
@@ -138,7 +160,7 @@ class NewslettersResponseBuilder {
     $data = [
       'id' => (string)$newsletter->getId(), // (string) for BC
       'hash' => $newsletter->getHash(),
-      'subject' => $newsletter->getSubject(),
+      'subject' => $this->processPersonalizationTags($newsletter->getSubject()),
       'type' => $newsletter->getType(),
       'status' => $newsletter->getStatus(),
       'sent_at' => ($sentAt = $newsletter->getSentAt()) ? $sentAt->format(self::DATE_FORMAT) : null,
diff --git a/mailpoet/tests/integration/API/JSON/ResponseBuilders/NewslettersResponseBuilderTest.php b/mailpoet/tests/integration/API/JSON/ResponseBuilders/NewslettersResponseBuilderTest.php
index 2e507f39b2..7ac6bd2629 100644
--- a/mailpoet/tests/integration/API/JSON/ResponseBuilders/NewslettersResponseBuilderTest.php
+++ b/mailpoet/tests/integration/API/JSON/ResponseBuilders/NewslettersResponseBuilderTest.php
@@ -57,4 +57,22 @@ class NewslettersResponseBuilderTest extends \MailPoetTest {
     $em->remove($newsletter);
     $em->flush();
   }
+
+  public function testItReplacesPersonalizationTags() {
+    $em = $this->diContainer->get(EntityManager::class);
+    $responseBuilder = $this->diContainer->get(NewslettersResponseBuilder::class);
+    $em->persist($newsletter = new NewsletterEntity);
+    $newsletter->setType(NewsletterEntity::TYPE_STANDARD);
+    $newsletter->setStatus(NewsletterEntity::STATUS_SENT);
+    $newsletter->setSubject('Subject');
+    $em->flush();
+
+    $newsletter->setSubject('Subject');
+    $response = $responseBuilder->buildForListing([$newsletter]);
+    verify($response[0]['subject'])->equals('Subject');
+
+    $newsletter->setSubject('Hello <!--[mailpoet/subscriber-firstname default="subscriber"]-->!');
+    $response = $responseBuilder->buildForListing([$newsletter]);
+    verify($response[0]['subject'])->equals('Hello [mailpoet/subscriber-firstname default="subscriber"]!');
+  }
 }