ENGCOM-3092: Backport 2.2 - Fix issue 17152 - prevent email being marked as not se… #18376

Author: Stanislav Idolov

app/code/Magento/Sales/Model/Order/Email/Sender.php

@@ -65,6 +65,8 @@ public function __construct(
     }
 
     /**
+     * Send order email if it is enabled in configuration.
+     *
      * @param Order $order
      * @return bool
      */
@@ -81,17 +83,21 @@ protected function checkAndSend(Order $order)
 
         try {
             $sender->send();
-            $sender->sendCopyTo();
         } catch (\Exception $e) {
             $this->logger->error($e->getMessage());
-
             return false;
         }
-
+        try {
+            $sender->sendCopyTo();
+        } catch (\Exception $e) {
+            $this->logger->error($e->getMessage());
+        }
         return true;
     }
 
     /**
+     * Populate order email template with customer information.
+     *
      * @param Order $order
      * @return void
      */
@@ -113,6 +119,8 @@ protected function prepareTemplate(Order $order)
     }
 
     /**
+     * Create Sender object using appropriate template and identity.
+     *
      * @return Sender
      */
     protected function getSender()
@@ -126,6 +134,8 @@ protected function getSender()
     }
 
     /**
+     * Get template options.
+     *
      * @return array
      */
     protected function getTemplateOptions()
@@ -137,6 +147,8 @@ protected function getTemplateOptions()
     }
 
     /**
+     * Render shipping address into html.
+     *
      * @param Order $order
      * @return string|null
      */
@@ -148,6 +160,8 @@ protected function getFormattedShippingAddress($order)
     }
 
     /**
+     * Render billing address into html.
+     *
      * @param Order $order
      * @return string|null
      */