Skip to content

Commit

Permalink
make sure $bounce_unsubscribe_threshold is less or equal to blacklist…
Browse files Browse the repository at this point in the history
…-email-on-bounce
  • Loading branch information
michield committed Jul 25, 2016
1 parent 29d5ce6 commit 29790c5
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions public_html/lists/admin/init.php
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -471,6 +471,10 @@
if (!defined('BLACKLIST_EMAIL_ON_BOUNCE')) { if (!defined('BLACKLIST_EMAIL_ON_BOUNCE')) {
define('BLACKLIST_EMAIL_ON_BOUNCE', 5); define('BLACKLIST_EMAIL_ON_BOUNCE', 5);
} }
if ($bounce_unsubscribe_threshold < BLACKLIST_EMAIL_ON_BOUNCE) {

This comment has been minimized.

Copy link
@rafael0116

rafael0116 Feb 7, 2019

Shouldn't this be > ?

This comment has been minimized.

Copy link
@michield

michield Feb 7, 2019

Author Member

yes, I think you're right. FYI @xh3n1
@rafael0116 fancy creating a PR?

$bounce_unsubscribe_threshold = BLACKLIST_EMAIL_ON_BOUNCE;
}

if (!defined('UNBLACKLIST_IN_PROFILE')) { if (!defined('UNBLACKLIST_IN_PROFILE')) {
define('UNBLACKLIST_IN_PROFILE', false); define('UNBLACKLIST_IN_PROFILE', false);
} }
Expand Down

1 comment on commit 29790c5

@michield
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, this should fix that 1a2cbef

Please sign in to comment.