From 03ff848f162f30aa8e0a473eb6c3f118d37214cb Mon Sep 17 00:00:00 2001
From: Denis Kochnov <d.kochnov.optimus@gmail.com>
Date: Wed, 29 Mar 2017 15:53:04 +0300
Subject: [PATCH] Refs #10652: Aschroder_SMTPPro_Model_Email_Queue - fix queue
 send method to continue sending 					
    and mark mail as processed if non protocol error occurs 	     Aschroder
 SMTPPro - fix admin module configuration file to allow multi store
 configuration

---
 .../Aschroder/SMTPPro/Model/Email/Queue.php   | 10 ++++-
 .../local/Aschroder/SMTPPro/etc/system.xml    | 42 +++++++++----------
 2 files changed, 30 insertions(+), 22 deletions(-)

diff --git a/app/code/local/Aschroder/SMTPPro/Model/Email/Queue.php b/app/code/local/Aschroder/SMTPPro/Model/Email/Queue.php
index 8a2fe7b..f3bfd27 100644
--- a/app/code/local/Aschroder/SMTPPro/Model/Email/Queue.php
+++ b/app/code/local/Aschroder/SMTPPro/Model/Email/Queue.php
@@ -126,7 +126,15 @@ public function send()
                     Mage::logException($e);
                     Mage::setIsDeveloperMode($oldDevMode);
 
-                    return false;
+                    // 553 - Requested action not taken: mailbox name not allowed
+                    if ($e instanceof Zend_Mail_Protocol_Exception && $e->getCode() != 553) {
+                        return false;
+                    }
+
+                    $message->setProcessedAt(Varien_Date::formatDate(true));
+                    $message->save();
+
+                    continue;
                 }
 
                 // after each valid message has been sent - pause if required
diff --git a/app/code/local/Aschroder/SMTPPro/etc/system.xml b/app/code/local/Aschroder/SMTPPro/etc/system.xml
index d81e468..25283b0 100644
--- a/app/code/local/Aschroder/SMTPPro/etc/system.xml
+++ b/app/code/local/Aschroder/SMTPPro/etc/system.xml
@@ -5,8 +5,8 @@
       <label>Aschroder Extensions</label>
       <sort_order>600</sort_order>
       <show_in_default>1</show_in_default>
-      <show_in_website>0</show_in_website>
-      <show_in_store>0</show_in_store>
+      <show_in_website>1</show_in_website>
+      <show_in_store>1</show_in_store>
     </aschroder>
   </tabs>
   <sections>
@@ -17,8 +17,8 @@
       <frontend_type>text</frontend_type>
       <sort_order>110</sort_order>
       <show_in_default>1</show_in_default>
-      <show_in_website>0</show_in_website>
-      <show_in_store>0</show_in_store>
+      <show_in_website>1</show_in_website>
+      <show_in_store>1</show_in_store>
       <groups>
         <general module="smtppro" translate="label comment">
           <label>General Settings</label>
@@ -26,7 +26,7 @@
           <sort_order>10</sort_order>
           <show_in_default>1</show_in_default>
           <show_in_website>1</show_in_website>
-          <show_in_store>0</show_in_store>
+          <show_in_store>1</show_in_store>
           <comment><![CDATA[<div style='background-color: #efefef;margin-bottom: 10px;height: 40px;'> <img style='float:left;width: 150px;' src='http://www.aschroder.com/smtppro-logo.png' /> <span style='float:left;font-size: 20px; margin:10px;'>SMTP Pro Email Extension</span> </div> Configure your SMTP connection below. If you have any questions or would like any help please visit <a href='http://magesmtppro.com' target='_blank'>magesmtppro.com</a>.]]></comment>
           <fields>
             <option translate="label">
@@ -35,8 +35,8 @@
               <source_model>smtppro/system_config_source_smtp_option</source_model>
               <sort_order>10</sort_order>
               <show_in_default>1</show_in_default>
-              <show_in_website>0</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_website>1</show_in_website>
+              <show_in_store>1</show_in_store>
             </option>
 
 
@@ -47,7 +47,7 @@
               <sort_order>20</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>google</option></depends>
             </googleapps_email>
             <googleapps_gpassword translate="label comment">
@@ -58,7 +58,7 @@
               <sort_order>23</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>google</option></depends>
             </googleapps_gpassword>
 
@@ -68,7 +68,7 @@
                   <sort_order>20</sort_order>
                   <show_in_default>1</show_in_default>
                   <show_in_website>1</show_in_website>
-                  <show_in_store>0</show_in_store>
+                  <show_in_store>1</show_in_store>
                   <depends><option>sendgrid</option></depends>
               </sendgrid_email>
               <sendgrid_password translate="label comment">
@@ -79,7 +79,7 @@
                   <sort_order>23</sort_order>
                   <show_in_default>1</show_in_default>
                   <show_in_website>1</show_in_website>
-                  <show_in_store>0</show_in_store>
+                  <show_in_store>1</show_in_store>
                   <depends><option>sendgrid</option></depends>
               </sendgrid_password>
 
@@ -89,7 +89,7 @@
                   <sort_order>20</sort_order>
                   <show_in_default>1</show_in_default>
                   <show_in_website>1</show_in_website>
-                  <show_in_store>0</show_in_store>
+                  <show_in_store>1</show_in_store>
                   <depends><option>mailup</option></depends>
               </mailup_email>
               <mailup_password translate="label comment">
@@ -100,7 +100,7 @@
                   <sort_order>23</sort_order>
                   <show_in_default>1</show_in_default>
                   <show_in_website>1</show_in_website>
-                  <show_in_store>0</show_in_store>
+                  <show_in_store>1</show_in_store>
                   <depends><option>mailup</option></depends>
               </mailup_password>
 
@@ -112,7 +112,7 @@
               <sort_order>20</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>ses</option></depends>
             </ses_access_key>
             <ses_private_key translate="label comment">
@@ -123,7 +123,7 @@
               <sort_order>23</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>ses</option></depends>
             </ses_private_key>
 
@@ -134,7 +134,7 @@
               <sort_order>20</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>smtp</option></depends>
             </smtp_authentication>
             <smtp_username translate="label">
@@ -143,7 +143,7 @@
               <sort_order>23</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>smtp</option><smtp_authentication separator=",">login,plain,crammd5</smtp_authentication></depends>
             </smtp_username>
             <smtp_password translate="label">
@@ -153,7 +153,7 @@
               <sort_order>26</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>smtp</option><smtp_authentication separator=",">login,plain,crammd5</smtp_authentication></depends>
             </smtp_password>
             <smtp_host translate="label">
@@ -162,7 +162,7 @@
               <sort_order>29</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>smtp</option></depends>
             </smtp_host>
             <smtp_port translate="label">
@@ -171,7 +171,7 @@
               <sort_order>32</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>smtp</option></depends>
             </smtp_port>
             <smtp_ssl translate="label comment">
@@ -182,7 +182,7 @@
               <sort_order>35</sort_order>
               <show_in_default>1</show_in_default>
               <show_in_website>1</show_in_website>
-              <show_in_store>0</show_in_store>
+              <show_in_store>1</show_in_store>
               <depends><option>smtp</option></depends>
             </smtp_ssl>