Attenzione!
Ciò che fa questo codice è: ogni volta che magento-cronjob rimuove tutti i messaggi inviati dalla tabella del database core_email_queue, rimuove anche tutti i destinatari di questi messaggi. Quindi, fondamentalmente, non funziona per te fino a quando questo task cronjob non verrà eseguito almeno una volta.
Soluzione
Ho trovato la risposta grazie a un'altra domanda qui: la tabella core_email_queue_recipients non è stata svuotata dal cronjob. Il metodo Mage_Core_Model_Email_Queue::cleanQueue()
chiama Mage_Core_Model_Resource_Email_Queue::removeSentMessages()
, che è piuttosto semplice:
public function removeSentMessages() {
$this->_getWriteAdapter()->delete($this->getMainTable(), 'processed_at IS NOT NULL');
return $this;
}
Comunque, questo metodo non rimuove i vecchi destinatari. Pertanto, non appena un nuovo messaggio con message_id n viene messo in coda, anche tutti i vecchi destinatari con message_id n riceveranno la nuova e-mail. La cosa che non capisco è: perché il core team non l'ha visto e perché questo non porta a ulteriori problemi?
Ho scritto un piccolo modulo per risolvere questo problema. Utilizza un override di classe per Mage_Core_Model_Resource_Email_Queue
, quindi se qualcuno può suggerire una soluzione migliore (basata sugli eventi?), Sarei felice.
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 / Modello / 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>