removed unused code inherited from version 4.0

config/mail.global.php.dist

@@ -2,20 +2,20 @@
 
 declare(strict_types=1);
 
+use Symfony\Component\Mailer\Transport\SendmailTransport;
+
 return [
 
     /**
      * Dotkernel mail module configuration
-     * Note that many of these options can be set programmatically too, when sending mail messages
-     * actually that is what you'll usually do, these config provide just default and options that remain the same for all mails
+     * Note that many of these options can be set programmatically too, when sending mail messages actually that is
+     * what you'll usually do, these config provide just default and options that remain the same for all mails
      */
-
     'dot_mail' => [
         //the key is the mail service name, this is the default one, which does not extends any configuration
         'default' => [
             //tells which other mail service configuration to extend
             'extends' => null,
-
             /**
              * the mail transport to use
              * can be any class implementing Symfony\Component\Mailer\Transport\TransportInterface
@@ -26,88 +26,57 @@ return [
              *
              * defaults to sendmail
              **/
-
-            'transport' => Symfony\Component\Mailer\Transport\SendmailTransport::class,
-
+            'transport' => SendmailTransport::class,
             //message configuration
             'message_options' => [
-
                 //from email address of the email
                 'from' => '',
-
                 //from name to be displayed instead of from address
                 'from_name' => '',
-
                 //reply-to email address of the email
                 'reply_to' => '',
-
                 //replyTo name to be displayed instead of the address
                 'reply_to_name' => '',
-
                 //destination email address as string or a list of email addresses
                 'to' => [],
-
                 //copy destination addresses
                 'cc' => [],
-
                 //hidden copy destination addresses
                 'bcc' => [],
-
                 //email subject
                 'subject' => '',
-
                 //body options - content can be plain text, HTML
                 'body' => [
                     'content' => '',
                     'charset' => 'utf-8',
                 ],
-
                 //attachments config
                 'attachments' => [
                     'files' => [],
                     'dir'   => [
                         'iterate'   => false,
                         'path'      => 'data/mail/attachments',
                         'recursive' => false,
-                    ]
+                    ],
                 ],
             ],
-
             //options that will be used only if Symfony\Component\Mailer\Transport\Smtp\SmtpTransport adapter is used
             'smtp_options' => [
-
                 //hostname or IP address of the mail server
                 'host' => '',
-
                 //port of the mail server - 587 or 465 for secure connections
-                'port' => 587,
-
-                //connection class used for authentication
-                //the value can be one of smtp, plain, login or crammd5
-                'connection_class' => 'login',
-
+                'port'              => 587,
                 'connection_config' => [
-
                     //the smtp authentication identity
                     'username' => '',
-
                     //the smtp authentication credential
                     'password' => '',
-
-                    //the encryption type to be used, ssl or tls
-                    //null should be used to disable SSL
-                    'ssl' => 'tls',
-                ]
+                ],
             ],
         ],
         // option to log the SENT emails
         'log' => [
-          'sent' => getcwd() . '/log/mail/sent.log'
+            'sent' => getcwd() . '/log/mail/sent.log',
         ],
-
-        /**
-         * You can define other mail services here, with the same structure as the default block
-         * you can even extend from the default block, and overwrite only the differences
-         */
     ],
 ];

src/Options/MailOptions.php

@@ -21,7 +21,6 @@
 class MailOptions extends AbstractOptions
 {
     protected array $eventListeners                = [];
-    protected array $saveSentMessageFolder         = [];
     protected TransportInterface|string $transport = SmtpTransport::class;
     protected array $transportMap                  = [
         'smtp'     => [SmtpTransport::class],
@@ -89,14 +88,4 @@ public function setEventListeners(array $eventListeners): void
     {
         $this->eventListeners = $eventListeners;
     }
-
-    public function getSaveSentMessageFolder(): array
-    {
-        return $this->saveSentMessageFolder;
-    }
-
-    public function setSaveSentMessageFolder(array $saveSentMessageFolder): void
-    {
-        $this->saveSentMessageFolder = $saveSentMessageFolder;
-    }
 }

src/Options/SmtpOptions.php

@@ -15,7 +15,6 @@
 class SmtpOptions extends AbstractOptions
 {
     protected string $name            = 'localhost';
-    protected string $connectionClass = 'smtp';
     protected array $connectionConfig = [];
     protected string $host            = '127.0.0.1';
     protected int $port               = 25;
@@ -36,23 +35,6 @@ public function setName(string $name): static
         return $this;
     }
 
-    /**
-     * Get connection class
-     */
-    public function getConnectionClass(): string
-    {
-        return $this->connectionClass;
-    }
-
-    /**
-     * Set connection class
-     */
-    public function setConnectionClass(string $connectionClass): static
-    {
-        $this->connectionClass = $connectionClass;
-        return $this;
-    }
-
     /**
      * Get connection configuration array
      */

test/CommonTrait.php

@@ -58,11 +58,8 @@ private function generateConfig(): array
                      *
                      * defaults to sendmail
                      **/
-                    'transport' => SmtpTransport::class,
-
-                    // Valid only if the Transport is SMTP
-                    'save_sent_message_folder' => ['INBOX.Sent'],
-                    'message_options'          => [
+                    'transport'       => SmtpTransport::class,
+                    'message_options' => [
                         'from'          => '',
                         'from_name'     => '',
                         'reply_to'      => '',
@@ -84,21 +81,16 @@ private function generateConfig(): array
                             ],
                         ],
                     ],
-
                     //options that will be used only if Symfony\Component\Mailer\Transport\Smtp\SmtpTransport
                     // adapter is used
                     'smtp_options' => [
                         'host'              => 'testHost',
                         'port'              => 587,
-                        'connection_class'  => 'login',
                         'connection_config' => [
-
                             //the smtp authentication identity
                             'username' => 'test',
-
                             //the smtp authentication credential
                             'password' => 'testPassword',
-                            'ssl'      => 'tls',
                         ],
                     ],
                 ],
@@ -108,11 +100,6 @@ private function generateConfig(): array
                 'log' => [
                     'sent' => $this->fileSystem->url() . '/log/mail/sent.log',
                 ],
-
-                /**
-                 * You can define other mail services here, with the same structure as the default block
-                 * you can even extend from the default block, and overwrite only the differences
-                 */
             ],
         ];
     }

test/Options/MailOptionsTest.php

@@ -17,25 +17,22 @@ public function testGettersAndSetters(): void
     {
         $subject = new MailOptions();
 
-        $transport             = 'smtp';
-        $transportMap          = ['test' => 'array'];
-        $messageOptions        = ['from' => '', 'to' => []];
-        $smtpOptions           = ['host' => '', 'port' => 587];
-        $eventListeners        = [AbstractMailEventListener::class];
-        $saveSentMessageFolder = ['INBOX.Sent'];
+        $transport      = 'smtp';
+        $transportMap   = ['test' => 'array'];
+        $messageOptions = ['from' => '', 'to' => []];
+        $smtpOptions    = ['host' => '', 'port' => 587];
+        $eventListeners = [AbstractMailEventListener::class];
 
         $subject->setTransport($transport);
         $subject->setTransportMap($transportMap);
         $subject->setMessageOptions($messageOptions);
         $subject->setSmtpOptions($smtpOptions);
         $subject->setEventListeners($eventListeners);
-        $subject->setSaveSentMessageFolder($saveSentMessageFolder);
 
         $this->assertSame(SmtpTransport::class, $subject->getTransport());
         $this->assertSame($transportMap, $subject->getTransportMap());
         $this->assertInstanceOf(MessageOptions::class, $subject->getMessageOptions());
         $this->assertInstanceOf(SmtpOptions::class, $subject->getSmtpOptions());
         $this->assertSame($eventListeners, $subject->getEventListeners());
-        $this->assertSame($saveSentMessageFolder, $subject->getSaveSentMessageFolder());
     }
 }

test/Options/SmtpOptionsTest.php

@@ -23,7 +23,6 @@ public function testSmtpGettersAndSetters(): void
         $this->assertIsString($subject->getName());
         $this->assertIsArray($subject->getConnectionConfig());
         $this->assertIsString($subject->getHost());
-        $this->assertIsString($subject->getConnectionClass());
         $this->assertIsInt($subject->getPort());
         $this->assertIsInt($subject->getConnectionTimeLimit());