Details
-
Bug
-
Status: Closed (View Workflow)
-
Minor
-
Resolution: Fixed
-
3.0.7-PL1
-
None
Description
I have 4,000+ members and I wish to send a mass email. In checking for queuing (queueing?), I've seen people complain on the phpbb3 support forum that emails are not being throttled, even when they've set the batch size very low. Since my host limits emails to 1,000 per hour and fails any messages above that limit, I decided to have a looksee at the code before attempting to send the mass email.
In includes/functions_messenger.php, $num_items seems to be used to control how many messages are sent in the current execution of the function. It appears that $num_items is set properly at first, but then overridden and set to the total number of messages in the queue if the number of messages in the queue is 2.5 times the batch size.
It seems to me to be completely overriding the user's configuration defeating the purpose of the queue, and the configuration limit.
if (!isset($data_ary['package_size']))
|
{
|
$data_ary['package_size'] = 0;
|
}
|
|
|
$package_size = $data_ary['package_size'];
|
$num_items = (!$package_size || sizeof($data_ary['data']) < $package_size) ? sizeof($data_ary['data']) : $package_size;
|
|
|
// If the amount of emails to be sent is way more than package_size than we need to increase it to prevent backlogs...
|
if (sizeof($data_ary['data']) > $package_size * 2.5)
|
{
|
$num_items = sizeof($data_ary['data']);
|
}
|
|
|
.
|
.
|
.
|
[later in the code]
|
.
|
.
|
.
|
for ($i = 0; $i < $num_items; $i++)
|
{
|
// Make variables available...
|
extract(array_shift($this->queue_data[$object]['data']));
|
|
|
switch ($object)
|
{
|
case 'email':
|
$err_msg = '';
|
$to = (!$to) ? 'undisclosed-recipients:;' : $to;
|
|
|
if ($config['smtp_delivery'])
|
{
|
$result = smtpmail($addresses, mail_encode($subject), wordwrap(utf8_wordwrap($msg), 997, "\n", true), $err_msg, $headers);
|
[code continues...]
|
Some hosts have a very strict limit on number of addresses per batch. For instance, one host in particular has a limit of 4. This approach would not ever work for anyone on that host attempting to send more than 10 messages at a time.
I believe the following code should be removed as its current operation appears to be incorrect.
// If the amount of emails to be sent is way more than package_size than we need to increase it to prevent backlogs...
|
if (sizeof($data_ary['data']) > $package_size * 2.5)
|
{
|
$num_items = sizeof($data_ary['data']);
|
}
|
Attachments
Issue Links
- is duplicated by
-
-
- Closed
-