More questions in studying the code in an attempt to answer my own questions... from lines 379-393, looks to be some duplication duplication:
<?php
// Send Out Email
// -- Remove duplicate email ($toemail)
$toemail = array_flip($toemail);
$toemail = array_flip($toemail);
// -- Email $toemail ($toemail)
for($i = 0; $i < count($toemail); $i++) {
mail($toemail[$i], $subject, $content, $headers);
}
// -- Remove duplicate email ($tousers)
$touemail = array_flip($touemail);
$touemail = array_flip($touemail);
// -- Email $touemail ($tousers)
for($i = 0; $i < count($touemail); $i++) {
mail($touemail[$i], $subject, $content, $headers);
}
?>
In addition, from a small optimization/best practices standpoint, my understanding is that it could be written slightly more efficiently by eliminating a ton of count iterations on the for loop:
<?php
// -- Remove duplicate email ($tousers)
$touemail = array_flip($touemail);
$touemail = array_flip($touemail);
// -- Email $touemail ($tousers)
$j = count($touemail);
for($i = 0; $i < $j; $i++) {
mail($touemail[$i], $subject, $content, $headers);
}
?>
So, in answering my own question, it seems that you’re just calling the PHP in-built mail function. This would seem to lend itself to timeouts for large lists, no?
MODX Staff