Added setting to store html

Author: samnotsowise

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+### 1.2.6
+
+* Added: Setting to enable storage of HTML instead of Text
+
 ### 1.2.5
 
 * Fixed: Added no search HTML for performance reasons.

src/Processors/CommunicationProcessor.php

@@ -13,8 +13,7 @@
 use Rhubarb\Scaffolds\Communications\Models\Communication;
 use Rhubarb\Scaffolds\Communications\Models\CommunicationItem;
 use Rhubarb\Scaffolds\Communications\Models\CommunicationItemSendAttempt;
-use Rhubarb\Scaffolds\Communications\Tests\Models\CommunicationItemTest;
-use Rhubarb\Stem\Filters\Equals;
+use Rhubarb\Scaffolds\Communications\Settings\CommunicationProcessorSettings;
 use Rhubarb\Stem\Schema\SolutionSchema;
 
 final class CommunicationProcessor
@@ -182,6 +181,8 @@ public static function draftPackage(CommunicationPackage $package)
         }
         $communication->save();
 
+        $storeHtml = CommunicationProcessorSettings::singleton()->storeHtml;
+
         foreach ($package->getSendables() as $sendable) {
             foreach ($sendable->getRecipients() as $recipient) {
                 $clone = clone $sendable;
@@ -190,7 +191,7 @@ public static function draftPackage(CommunicationPackage $package)
 
                 $item = SolutionSchema::getModel("CommunicationItem");
                 $item->Recipient = (string)$recipient;
-                $item->Text = $clone->getText();
+                $item->Text = $storeHtml ? $clone->getHtml() : $clone->getText();
                 $item->Type = $clone->getSendableType();
                 $item->SendableClassName = get_class($clone);
                 $data = $clone->toArray();

src/Settings/CommunicationProcessorSettings.php

@@ -10,4 +10,6 @@ class CommunicationProcessorSettings extends Settings
      * @var int Number of emails to send max per second.
      */
     public $throttle = 20;
-}
+
+    public $storeHtml = false;
+}

tests/unit/Processors/CommunicationProcessorTest.php

@@ -11,6 +11,7 @@
 use Rhubarb\Scaffolds\Communications\Models\CommunicationItem;
 use Rhubarb\Scaffolds\Communications\Models\CommunicationItemSendAttempt;
 use Rhubarb\Scaffolds\Communications\Processors\CommunicationProcessor;
+use Rhubarb\Scaffolds\Communications\Settings\CommunicationProcessorSettings;
 use Rhubarb\Scaffolds\Communications\Tests\Fixtures\CommunicationTestCase;
 
 class CommunicationProcessorTest extends CommunicationTestCase
@@ -122,6 +123,7 @@ private function createCommunication($dateToSend)
     {
         $email = new SimpleEmail();
         $email->setText("Test Message Body");
+        $email->setHtml('<p>Test Message Body</p>');
         $email->setSender("sender@gcdtech.com");
         $email->addRecipientByEmail("test@test.com");
 
@@ -157,4 +159,15 @@ private function createCommunicationWithMultipleRecipients($dateToSend, $noOfRec
 
         return $communication;
     }
+
+    public function testStoreHtmlUponRequest()
+    {
+        $this->createCommunication(null);
+        self::assertNotContains('<p>', CommunicationItem::findLast()->Text);
+
+        CommunicationProcessorSettings::singleton()->storeHtml = true;
+
+        $this->createCommunication(null);
+        self::assertContains('<p>', CommunicationItem::findLast()->Text);
+    }
 }