Merge pull request #22 from stackkit/development

Skip files in attach and attachData if their data is empty or null

Author: marickvantuil

src/EmailComposer.php

@@ -218,6 +218,12 @@ public function mailable(Mailable $mailable)
      */
     public function attach($file, $options = [])
     {
+        $validFileName = (is_string($file) && strlen($file) > 0);
+
+        if (! $validFileName) {
+            return $this;
+        }
+
         $attachments = $this->hasData('attachments') ? $this->getData('attachments') : [];
 
         $attachments[] = compact('file', 'options');
@@ -235,6 +241,12 @@ public function attach($file, $options = [])
      */
     public function attachData($data, $name, array $options = [])
     {
+        $validData = (is_string($data) && strlen($data) > 0);
+
+        if (! $validData) {
+            return $this;
+        }
+
         $attachments = $this->hasData('rawAttachments') ? $this->getData('rawAttachments') : [];
 
         $attachments[] = compact('data', 'name', 'options');

tests/SenderTest.php

@@ -167,6 +167,28 @@ public function attachments_are_added_to_the_email()
         $this->assertEquals('application/pdf', $attachment->getContentType());
     }
 
+    /** @test */
+    public function attachments_are_not_added_if_the_data_is_not_valid()
+    {
+        $this->sent = [];
+        $this->composeEmail()->attach(null)->send();
+        $this->artisan('email:send');
+        $attachments = reset($this->sent)->getMessage()->getChildren();
+        $this->assertCount(0, $attachments);
+
+        $this->sent = [];
+        $this->composeEmail()->attach(false)->send();
+        $this->artisan('email:send');
+        $attachments = reset($this->sent)->getMessage()->getChildren();
+        $this->assertCount(0, $attachments);
+
+        $this->sent = [];
+        $this->composeEmail()->attach('')->send();
+        $this->artisan('email:send');
+        $attachments = reset($this->sent)->getMessage()->getChildren();
+        $this->assertCount(0, $attachments);
+    }
+
     /** @test */
     public function raw_attachments_are_added_to_the_email()
     {
@@ -189,4 +211,26 @@ public function raw_attachments_are_added_to_the_email()
         $this->assertEquals('application/pdf', $attachment->getContentType());
         $this->assertContains('Hello CI!', $attachment->getBody());
     }
+
+    /** @test */
+    public function raw_attachments_are_not_added_if_the_data_is_not_valid()
+    {
+        $this->sent = [];
+        $this->composeEmail()->attachData(null, 'test.png')->send();
+        $this->artisan('email:send');
+        $attachments = reset($this->sent)->getMessage()->getChildren();
+        $this->assertCount(0, $attachments);
+
+        $this->sent = [];
+        $this->composeEmail()->attachData(false, 'test.png')->send();
+        $this->artisan('email:send');
+        $attachments = reset($this->sent)->getMessage()->getChildren();
+        $this->assertCount(0, $attachments);
+
+        $this->sent = [];
+        $this->composeEmail()->attachData('', 'test.png')->send();
+        $this->artisan('email:send');
+        $attachments = reset($this->sent)->getMessage()->getChildren();
+        $this->assertCount(0, $attachments);
+    }
 }