Check to ensure there are no invalid emails without recipient in queue (#1109)

Author: lemundo-team

app/code/core/Mage/Core/Model/Email/Queue.php

@@ -95,7 +95,7 @@ protected function _afterSave()
      */
     protected function _beforeSave()
     {
-        if (empty($this->_recipients) || !is_array($this->_recipients)) {
+        if (empty($this->_recipients) || !is_array($this->_recipients) || empty($this->_recipients[0])) { // additional check of recipients information (email address)
             Mage::throwException(Mage::helper('core')->__('Message recipients data must be set.'));
         }
         return parent::_beforeSave();
@@ -238,13 +238,13 @@ public function send()
 
                 try {
                     $mailer->send();
+                    unset($mailer);
+                    $message->setProcessedAt(Varien_Date::formatDate(true));
+                    $message->save(); // save() is throwing exception when recipient is not set
                 } catch (Exception $e) {
                     Mage::logException($e);
                 }
 
-                unset($mailer);
-                $message->setProcessedAt(Varien_Date::formatDate(true));
-                $message->save();
             }
         }