code cleanup

Author: marickvantuil

src/Email.php

@@ -3,9 +3,6 @@
 namespace Buildcode\LaravelDatabaseEmails;
 
 use Illuminate\Database\Eloquent\Model;
-use Illuminate\Mail\Message;
-use Illuminate\Support\Facades\Event;
-use Illuminate\Support\Facades\Mail;
 use Carbon\Carbon;
 use Exception;
 
@@ -31,9 +28,6 @@
  */
 class Email extends Model
 {
-    /**
-     * Make sure all encrypted attributes are decrypted.
-     */
     use HasEncryptedAttributes;
 
     /**
@@ -97,9 +91,9 @@ public function getRecipient()
      */
     public function getRecipientsAsString()
     {
-        $glue = ',';
+        $glue = ', ';
 
-        return implode($glue, (array) $this->recipient);
+        return implode($glue, (array)$this->recipient);
     }
 
     /**
@@ -183,7 +177,6 @@ public function getAttempts()
         return $this->attempts;
     }
 
-
     /**
      * Get the scheduled date.
      *
@@ -268,7 +261,6 @@ public function isScheduled()
         return !is_null($this->getScheduledDate());
     }
 
-
     /**
      * Determine if the e-mail is encrypted.
      *
@@ -340,7 +332,7 @@ public function markAsFailed(Exception $exception)
         $this->update([
             'sending' => 0,
             'failed'  => 1,
-            'error'   => (string) $exception,
+            'error'   => (string)$exception,
         ]);
     }
 

src/EmailComposer.php

@@ -44,7 +44,7 @@ public function getEmail()
      * Set a data value.
      *
      * @param string $key
-     * @param mixed $value
+     * @param mixed  $value
      * @return static
      */
     public function setData($key, $value)
@@ -58,7 +58,7 @@ public function setData($key, $value)
      * Get a data value.
      *
      * @param string $key
-     * @param mixed $default
+     * @param mixed  $default
      * @return mixed
      */
     public function getData($key, $default = null)
@@ -201,7 +201,7 @@ public function mailable(Mailable $mailable)
      * Attach a file to the e-mail.
      *
      * @param string $file
-     * @param array $options
+     * @param array  $options
      * @return static
      */
     public function attach($file, $options = [])
@@ -216,8 +216,8 @@ public function attach($file, $options = [])
     /**
      * Attach in-memory data as an attachment.
      *
-     * @param  string  $data
-     * @param  string  $name
+     * @param  string $data
+     * @param  string $name
      * @param  array  $options
      * @return $this
      */

src/MailableReader.php

@@ -4,7 +4,6 @@
 
 use function call_user_func_array;
 use Exception;
-use Illuminate\Mail\Mailable;
 
 class MailableReader
 {

src/Store.php

@@ -34,7 +34,7 @@ public function getQueue()
      * Get all e-mails that failed to be sent.
      *
      * @param int $id
-     * @return Collection
+     * @return Collection|Email[]
      */
     public function getFailed($id = null)
     {

src/Validator.php

@@ -144,7 +144,7 @@ private function validateView(EmailComposer $composer)
         $view = $composer->getData('view');
 
         if (!view()->exists($view)) {
-            throw new InvalidArgumentException('View [' . $view .'] does not exist');
+            throw new InvalidArgumentException('View [' . $view . '] does not exist');
         }
     }
 

tests/EncryptionTest.php

@@ -2,8 +2,6 @@
 
 namespace Tests;
 
-use Buildcode\LaravelDatabaseEmails\Email;
-
 class EncryptionTest extends TestCase
 {
     function setUp()

tests/RetryFailedEmailsCommandTest.php

@@ -2,9 +2,7 @@
 
 namespace Tests;
 
-use Buildcode\LaravelDatabaseEmails\Store;
 use Illuminate\Support\Facades\DB;
-use Illuminate\Support\Facades\Event;
 
 class RetryFailedEmailsCommandTest extends TestCase
 {

tests/SendEmailsCommandTest.php

@@ -5,7 +5,6 @@
 use Buildcode\LaravelDatabaseEmails\Store;
 use Carbon\Carbon;
 use Illuminate\Support\Facades\DB;
-use Illuminate\Support\Facades\Event;
 
 class SendEmailsCommandTest extends TestCase
 {