Fixing Import custom email template functionality

mautic · Aug 25, 2023 · 8af114b · 8af114b
1 parent 32bdaae
commit 8af114b
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 3 deletions.
diff --git a/app/bundles/EmailBundle/Helper/MailHelper.php b/app/bundles/EmailBundle/Helper/MailHelper.php
@@ -1328,13 +1328,11 @@ public function setEmail(Email $email, $allowBcc = true, $slots = [], $assetAtta
             $this->setPlainText($plainText);
         }
 
-        $BCcontent  = $email->getContent();
         $customHtml = $email->getCustomHtml();
         // Process emails created by Mautic v1
-        if (empty($customHtml) && !empty($BCcontent)) {
+        if (empty($customHtml)) {
             $template = $email->getTemplate();
             if (empty($slots)) {
-                $template = $email->getTemplate();
                 $slots    = $this->factory->getTheme($template)->getSlots('email');
             }
 

diff --git a/app/bundles/EmailBundle/Tests/Controller/EmailControllerFunctionalTest.php b/app/bundles/EmailBundle/Tests/Controller/EmailControllerFunctionalTest.php
@@ -387,4 +387,35 @@ public function testEmailDetailsPageShouldNotHavePendingCount(): void
 
         $this->assertCount(0, $pendingCountQuery);
     }
+
+    public function testSendEmailForImportCustomEmailTemplate(): void
+    {
+        $email = new Email();
+        $email->setName('Test Email C');
+        $email->setSubject('Test Email C Subject');
+        $email->setTemplate('blank');
+        $email->setEmailType('template');
+
+        $contact = new Lead();
+        $contact->setEmail('john@doe.email');
+
+        $this->em->persist($email);
+        $this->em->persist($contact);
+        $this->em->flush();
+
+        // Create the member now.
+        $payload = [
+            'action'   => 'lead:getEmailTemplate',
+            'template' => $email->getId(),
+        ];
+
+        $this->client->request('GET', '/s/ajax', $payload, [], $this->createAjaxHeaders());
+        $clientResponse = $this->client->getResponse();
+        $response       = json_decode($clientResponse->getContent(), true);
+
+        $this->assertSame(1, $response['success']);
+        $this->assertNotEmpty($response['subject']);
+        $this->assertEquals($email->getSubject(), $response['subject']);
+        $this->assertNotEmpty($response['body']);
+    }
 }