Atenção!
O que esse código faz é: toda vez que o magento-cronjob remove todas as mensagens enviadas da tabela do banco de dados core_email_queue, ele também remove todos os destinatários dessas mensagens. Portanto, basicamente, ele não funciona para você até que esta tarefa do cronjob seja executada pelo menos uma vez.
Solução
Encontrei a resposta graças a outra pergunta aqui: a tabela core_email_queue_recipients não foi esvaziada pelo cronjob. O método Mage_Core_Model_Email_Queue::cleanQueue()
chama Mage_Core_Model_Resource_Email_Queue::removeSentMessages()
, o que é bastante simples:
public function removeSentMessages() {
$this->_getWriteAdapter()->delete($this->getMainTable(), 'processed_at IS NOT NULL');
return $this;
}
De qualquer forma, esse método não remove os destinatários antigos. Assim, assim que uma nova mensagem com message_id n for enfileirada, todos os destinatários antigos com message_id n também receberão o novo email. O que não entendo é: por que a equipe principal não viu isso e por que isso não leva a mais problemas?
Eu escrevi um pequeno módulo para corrigir isso. Ele usa uma substituição de classe para Mage_Core_Model_Resource_Email_Queue
, portanto, se alguém puder sugerir uma solução melhor (baseada em eventos?), Eu ficaria feliz.
app / code / local / Namespace / EmailQueueFix / etc / config.xml
<?xml version="1.0"?>
<config>
<modules>
<Namespace_EmailQueueFix>
<version>1.0</version>
</Namespace_EmailQueueFix>
</modules>
<global>
<models>
<core_resource>
<rewrite>
<email_queue>Namespace_EmailQueueFix_Model_Resource_Email_Queue</email_queue>
</rewrite>
</core_resource>
</models>
</global>
</config>
app / code / local / Namespace / EmailQueueFix / Model / Resource / Email / Queue.php
<?php
class Namespace_EmailQueueFix_Model_Resource_Email_Queue extends Mage_Core_Model_Resource_Email_Queue {
/**
* Remove already sent messages
* ADDED: also remove all recipients of sent messages!
*
* @return Mage_Core_Model_Resource_Email_Queue
*/
public function removeSentMessages() {
$writeAdapter = $this->_getWriteAdapter();
$readAdapter = $this->_getReadAdapter();
$select = $readAdapter->select()->from(array("ceqr" => $this->getTable('core/email_recipients')), array('*'))->joinLeft(array('ceq' => $this->getMainTable()), 'ceqr.message_id = ceq.message_id', array('*'))->where('ceq.processed_at IS NOT NULL OR ceq.message_id IS NULL');
$recipients = $readAdapter->fetchAll($select);
if ( $recipients ) {
foreach ( $recipients as $recipient ) {
$writeAdapter->delete($this->getTable('core/email_recipients'), "recipient_id = " . $recipient['recipient_id']);
}
}
$writeAdapter->delete($this->getMainTable(), 'processed_at IS NOT NULL');
return $this;
}
}
app / etc / modules / Namespace_EmailQueueFix.xml
<?xml version="1.0"?>
<config>
<modules>
<Namespace_EmailQueueFix>
<codePool>local</codePool>
<active>true</active>
</Namespace_EmailQueueFix>
<depends>
<Mage_Core/>
</depends>
</modules>
</config>