Added some specific debugging hint to email queue empty recipient error (#1386)

seansan · web-flow · commit 63d402e9cd13 · 2023-01-13T11:13:41.000Z
diff --git a/app/code/core/Mage/Core/Model/Email/Queue.php b/app/code/core/Mage/Core/Model/Email/Queue.php
@@ -93,7 +93,8 @@ protected function _afterSave()
     protected function _beforeSave()
     {
         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.'));
+            $error = Mage::helper('core')->__('Message recipients data must be set.');
+            Mage::throwException("{$error} - ID: " . $this->getId());
         }
         return parent::_beforeSave();
     }

Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,8 @@ protected function _afterSave()`
`93`	`93`	`protected function _beforeSave()`
`94`	`94`	`{`
`95`	`95`	`if (empty($this->_recipients) \|\| !is_array($this->_recipients) \|\| empty($this->_recipients[0])) { // additional check of recipients information (email address)`
`96`		`- Mage::throwException(Mage::helper('core')->__('Message recipients data must be set.'));`
	`96`	`+ $error = Mage::helper('core')->__('Message recipients data must be set.');`
	`97`	`+ Mage::throwException("{$error} - ID: " . $this->getId());`
`97`	`98`	`}`
`98`	`99`	`return parent::_beforeSave();`
`99`	`100`	`}`