From 8393814e191136ab14c81a646ab700d7d419821f Mon Sep 17 00:00:00 2001 From: SNH_NL Date: Sun, 10 Jan 2021 17:11:05 +0100 Subject: [PATCH] Add some specific debugging hint No idea what works best. Ideally I would want to log ID/SUBJECT/STORE so I know where to start looking if this happens. If you propose otherwise, then please adjust the pull request directly and push. Only small win here is a little more specific debugging. https://stackoverflow.com/questions/53581543/exception-mage-core-exception-with-message-message-recipients-data-must-be-se --- app/code/core/Mage/Core/Model/Email/Queue.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/code/core/Mage/Core/Model/Email/Queue.php b/app/code/core/Mage/Core/Model/Email/Queue.php index 71c5880efb1..07a16bf4bea 100644 --- a/app/code/core/Mage/Core/Model/Email/Queue.php +++ b/app/code/core/Mage/Core/Model/Email/Queue.php @@ -96,7 +96,8 @@ protected function _afterSave() protected function _beforeSave() { if (empty($this->_recipients) || !is_array($this->_recipients)) { - Mage::throwException(Mage::helper('core')->__('Message recipients data must be set.')); + $error = Mage::helper('core')->__('Message recipients data must be set.'); + Mage::throwException($error . " Msg Id:" . $this->getId()); } return parent::_beforeSave(); }