Skip to content

Commit

Permalink
[ticket/9559] Do not override queue package size in the case of an ov…
Browse files Browse the repository at this point in the history
…erflow

This reverts r8025 to includes/functions_messenger.php.  That code caused
problems on hosters with email limits less than package_size*2.5.  And it isn't
desirable in general to over-ride the admin's configuration options.

PHPBB3-9559
  • Loading branch information
jellydoughnut committed Jul 22, 2010
1 parent b7ae0fe commit 2f312b2
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions phpBB/includes/functions_messenger.php
Original file line number Diff line number Diff line change
Expand Up @@ -671,11 +671,18 @@ function process()
$package_size = $data_ary['package_size'];
$num_items = (!$package_size || sizeof($data_ary['data']) < $package_size) ? sizeof($data_ary['data']) : $package_size;

/*
* This code is commented out because it causes problems on some web hosts.
* The core problem is rather restrictive email sending limits.
* This code is nly useful if you have no such restrictions from the
* web host and the package size setting is wrong.
// 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']);
}
*/

switch ($object)
{
Expand Down

0 comments on commit 2f312b2

Please sign in to comment.