BounceMailHandler - Code Metrics - Inspection of "[~]: "BounceMailHandler" -> optimize "testMode" us..." - voku/PHPMailer-BMH - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( 9127cb...6c003f )

by Lars
created 2020-10-19 15:31 UTC
BounceMailHandler F

↳ Parent: Project
Complexity

Total Complexity
102
Size/Duplication

Total Lines	916
Duplicated Lines	6.77 %
Coupling/Cohesion

Components	1
Dependencies	0
Test Coverage

Coverage
56.08%
Importance

Changes
Metric	Value
dl	62
loc	916
ccs	203
cts	362
cp	0.5608
rs	1.684
c	0
b	0
f	0
wmc	102
lcom	1
cbo	0
9 Methods

Rating	Name	Duplication	Size	Complexity
A	getVersion()	0	4	1
B	globalDelete()	0	59	7
A	isParameter()	0	14	4
B	mailboxExist()	0	59	8
A	openLocal()	0	21	3
A	openMailbox()	0	35	5
A	output()	0	10	3
F	processBounce()	62	173	30
F	processMailbox()	0	205	41
How to fix Duplicated Code Complexity

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
If you have the same expression in different places: Extract expression to a method
If you have the same method in different sub-classes: Extract method, and pull up field to the parent class
If you have the same code in unrelated classes: Consider extracting the code to a new class, and injecting that class
Complex Class

Tip: Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.
Complex classes like BounceMailHandler often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use BounceMailHandler, and based on these observations, apply Extract Interface, too.
<?php

declare(strict_types=1);

/**
 * Bounce Mail Handler (formerly known as BMH and PHPMailer-BMH)
 *
 * @copyright 2008-2009 Andry Prevost. All Rights Reserved.
 * @copyright 2011-2012 Anthon Pang.
 * @copyright 2015-2019 Lars Moelleken.
 * @license   GPL
 */
namespace BounceMailHandler;

use function bmhBodyRules;
use function bmhDSNRules;
use const CL_EXPUNGE;
use const OP_HALFOPEN;
use const OP_READONLY;
use const SORTDATE;

/**
 * BounceMailHandler class
 *
 * BounceMailHandler is a PHP program to check your IMAP/POP3 inbox and
 * delete all 'hard' bounced emails. It features a callback function where
 * you can create a custom action. This provides you the ability to write
 * a script to match your database records and either set inactive or
 * delete records with email addresses that match the 'hard' bounce results.
 */
class BounceMailHandler
{
    const SECONDS_TIMEOUT = 6000;

    const VERBOSE_DEBUG = 3; // detailed report plus debug info

    const VERBOSE_QUIET = 0; // suppress output

    const VERBOSE_REPORT = 2; // detailed report

    const VERBOSE_SIMPLE = 1; // simple report

    /**
     * mail-server
     *
     * @var string
     */
    public $mailhost = 'localhost';

    /**
     * the username of mailbox
     *
     * @var string
     */
    public $mailboxUserName = '';

    /**
     * the password needed to access mailbox
     *
     * @var string
     */
    public $mailboxPassword = '';

    /**
     * the last error msg
     *
     * @var string
     */
    public $errorMessage = '';

    /**
     * maximum limit messages processed in one batch
     *
     * @var int
     */
    public $maxMessages = 3000;

    /**
     * callback Action function name the function that handles the bounce mail. Parameters:
     *
     * int     $msgnum        the message number returned by Bounce Mail Handler
     * string  $bounce_type   the bounce type:
     *       'antispam',
     *       'autoreply',
     *       'concurrent',
     *       'content_reject',
     *       'command_reject',
     *       'internal_error',
     *       'defer',
     *       'delayed'
     *       =>
     *       array(
     *           'remove' => 0,
     *           'bounce_type' => 'temporary'
     *       ),
     *       'dns_loop',
     *       'dns_unknown',
     *       'full',
     *       'inactive',
     *       'latin_only',
     *       'other',
     *       'oversize',
     *       'outofoffice',
     *       'unknown',
     *       'unrecognized',
     *       'user_reject',
     *       'warning'
     * string  $email         the target email address
     * string  $subject       the subject, ignore now
     * string  $xheader       the XBounceHeader from the mail
     * 1 or 0  $remove        delete status, 0 is not deleted, 1 is deleted
     * string  $rule_no       bounce mail detect rule no.
     * string  $rule_cat      bounce mail detect rule category
     * int     $totalFetched  total number of messages in the mailbox
     *
     * @var mixed
     */
    public $actionFunction = 'callbackAction';

    /**
     * Callback custom body rules
     * ```
     * function customBodyRulesCallback( $result, $body, $structure, $debug )
     * {
     *    return $result;
     * }
     * ```
     *
     * @var callable|null
     */
    public $customBodyRulesCallback;

    /**
     * Callback custom DSN (Delivery Status Notification) rules
     * ```
     * function customDSNRulesCallback( $result, $dsnMsg, $dsnReport, $debug )
     * {
     *    return $result;
     * }
     * ```
     *
     * @var callable|null
     */
    public $customDSNRulesCallback;

    /**
     * test-mode, if true will not delete messages
     *
     * @var bool
     */
    public $testMode = false;

    /**
     * purge the unknown messages (or not)
     *
     * @var bool
     */
    public $purgeUnprocessed = false;

    /**
     * control the debug output, default is VERBOSE_SIMPLE
     *
     * @var int
     */
    public $verbose = self::VERBOSE_SIMPLE;

    /**
     * control the failed DSN rules output
     *
     * @var bool
     */
    public $debugDsnRule = false;

    /**
     * control the failed BODY rules output
     *
     * @var bool
     */
    public $debugBodyRule = false;

    /**
     * Control the method to process the mail header
     * if set true, uses the imap_fetchstructure function
     * otherwise, detect message type directly from headers,
     * a bit faster than imap_fetchstructure function and take less resources.
     *
     * however - the difference is negligible
     *
     * @var bool
     */
    public $useFetchstructure = true;

    /**
     * If disableDelete is equal to true, it will disable the delete function.
     *
     * @var bool
     */
    public $disableDelete = false;

    /**
     * defines new line ending
     *
     * @var string
     */
    public $bmhNewLine = "<br />\n";

    /**
     * defines port number, default is '143', other common choices are '110' (pop3), '993' (gmail)
     *
     * @var int
     */
    public $port = 143;

    /**
     * defines service, default is 'imap', choice includes 'pop3'
     *
     * @var string
     */
    public $service = 'imap';

    /**
     * defines service option, default is 'notls', other choices are 'tls', 'ssl'
     *
     * @var string
     */
    public $serviceOption = 'notls';

    /**
     * mailbox type, default is 'INBOX', other choices are (Tasks, Spam, Replies, etc.)
     *
     * @var string
     */
    public $boxname = 'INBOX';

    /**
     * determines if soft bounces will be moved to another mailbox folder
     *
     * @var bool
     */
    public $moveSoft = false;

    /**
     * mailbox folder to move soft bounces to, default is 'soft'
     *
     * @var string
     */
    public $softMailbox = 'INBOX.soft';

    /**
     * determines if hard bounces will be moved to another mailbox folder
     *
     * NOTE: If true, this will disable delete and perform a move operation instead
     *
     * @var bool
     */
    public $moveHard = false;

    /**
     * mailbox folder to move hard bounces to, default is 'hard'
     *
     * @var string
     */
    public $hardMailbox = 'INBOX.hard';

    /*
     * Mailbox folder to move unprocessed mails
     * @var string
     */
    public $unprocessedBox = 'INBOX.unprocessed';

    /**
     * deletes messages globally prior to date in variable
     *
     * NOTE: excludes any message folder that includes 'sent' in mailbox name
     * format is same as MySQL: 'yyyy-mm-dd'
     * if variable is blank, will not process global delete
     *
     * @var string
     */
    public $deleteMsgDate = '';

    /**
     * (internal variable)
     *
     * The resource handler for the opened mailbox (POP3/IMAP/NNTP/etc.)
     *
     * @var resource
     */
    protected $mailboxLink = false;

    /**
     * Holds Bounce Mail Handler version.
     *
     * @var string
     */
    private $version = '6.0-dev';

    /**
     * @return string
     */
    public function getVersion(): string
    {
        return $this->version;
    }

    /**
     * Function to delete messages in a mailbox, based on date
     *
     * NOTE: this is global ... will affect all mailboxes except any that have 'sent' in the mailbox name
     */
    public function globalDelete(): bool
    {
        $dateArr = \explode('-', $this->deleteMsgDate); // date format is yyyy-mm-dd
        $delDate = \mktime(0, 0, 0, (int) ($dateArr[1]), (int) ($dateArr[2]), (int) ($dateArr[0]));

        $port = $this->port . '/' . $this->service . '/' . $this->serviceOption;
        $mboxt = \imap_open('{' . $this->mailhost . ':' . $port . '}', $this->mailboxUserName, $this->mailboxPassword, OP_HALFOPEN);

        if ($mboxt === false) {
            return false;
        }

        $list = \imap_getmailboxes($mboxt, '{' . $this->mailhost . ':' . $port . '}', '*');

        if (\is_array($list)) {
            foreach ($list as $key => $val) {
                // get the mailbox name only
                $nameArr = \explode('}', \imap_utf7_decode($val->name));
                $nameRaw = $nameArr[\count($nameArr) - 1];

                if (\stripos($nameRaw, 'sent') === false) {
                    $mboxd = \imap_open('{' . $this->mailhost . ':' . $port . '}' . $nameRaw, $this->mailboxUserName, $this->mailboxPassword, CL_EXPUNGE);
                    $messages = \imap_sort($mboxd, SORTDATE, 0);

                    foreach ($messages as $message) {
                        $header = \imap_headerinfo($mboxd, $message);

                        // purge if prior to global delete date
                        if ($header->udate < $delDate) {
                            \imap_delete($mboxd, $message);
                        }
                    }

                    \imap_expunge($mboxd);
                    /** @noinspection UnusedFunctionResultInspection */
                    \imap_errors();
                    /** @noinspection UnusedFunctionResultInspection */
                    \imap_alerts();
                    \imap_close($mboxd);
                }
            }

            /** @noinspection UnusedFunctionResultInspection */
            \imap_errors();
            /** @noinspection UnusedFunctionResultInspection */
            \imap_alerts();
            \imap_close($mboxt);

            return true;
        }

        /** @noinspection UnusedFunctionResultInspection */
        \imap_errors();
        /** @noinspection UnusedFunctionResultInspection */
        \imap_alerts();
        \imap_close($mboxt);

        return false;
    }

    /**
     * Function to determine if a particular value is found in a imap_fetchstructure key.
     *
     * @param array  $currParameters imap_fetstructure parameters
     * @param string $varKey         imap_fetstructure key
     * @param string $varValue       value to check for
     *
     * @return bool
     */
    public function isParameter(array $currParameters, string $varKey, string $varValue): bool
    {
        foreach ($currParameters as $object) {
            if (
                \strtoupper($object->attribute) == \strtoupper($varKey)
                &&
                \strtoupper($object->value) == \strtoupper($varValue)
            ) {
                return true;
            }
        }

        return false;
    }

    /**
     * Function to check if a mailbox exists - if not found, it will create it.
     *
     * @param string $mailbox the mailbox name, must be in 'INBOX.checkmailbox' format
     * @param bool   $create  whether or not to create the checkmailbox if not found, defaults to true
     *
     * @return bool
     */
    public function mailboxExist(string $mailbox, bool $create = true): bool
    {
        if (\trim($mailbox) === '') {
            // this is a critical error with either the mailbox name blank or an invalid mailbox name
            // need to stop processing and exit at this point
            echo 'Invalid mailbox name for move operation. Cannot continue: ' . $mailbox . "<br />\n";
            exit();
        }

        $port = $this->port . '/' . $this->service . '/' . $this->serviceOption;
        /** @noinspection PhpUsageOfSilenceOperatorInspection */
        $mbox = @\imap_open('{' . $this->mailhost . ':' . $port . '}', $this->mailboxUserName, $this->mailboxPassword, OP_HALFOPEN);

        if ($mbox === false) {
            return false;
        }

        $list = \imap_getmailboxes($mbox, '{' . $this->mailhost . ':' . $port . '}', '*');
        $mailboxFound = false;

        if (\is_array($list)) {
            foreach ($list as $key => $val) {
                // get the mailbox name only
                $nameArr = \explode('}', \imap_utf7_decode($val->name));
                $nameRaw = $nameArr[\count($nameArr) - 1];
                if ($mailbox == $nameRaw) {
                    $mailboxFound = true;
                }
            }

            if ($mailboxFound === false && $create) {
                /** @noinspection PhpUsageOfSilenceOperatorInspection */
                @\imap_createmailbox($mbox, \imap_utf7_encode('{' . $this->mailhost . ':' . $port . '}' . $mailbox));
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
                /** @noinspection UnusedFunctionResultInspection */
                \imap_errors();
                /** @noinspection UnusedFunctionResultInspection */
                \imap_alerts();
                \imap_close($mbox);

                return true;
            }

            /** @noinspection UnusedFunctionResultInspection */
            \imap_errors();
            /** @noinspection UnusedFunctionResultInspection */
            \imap_alerts();
            \imap_close($mbox);

            return false;
        }

        /** @noinspection UnusedFunctionResultInspection */
        \imap_errors();
        /** @noinspection UnusedFunctionResultInspection */
        \imap_alerts();
        \imap_close($mbox);

        return false;
    }

    /**
     * open a mail box in local file system
     *
     * @param string $filePath The local mailbox file path
     *
     * @return bool
     */
    public function openLocal(string $filePath): bool
    {
        \set_time_limit(self::SECONDS_TIMEOUT);

        if (!$this->testMode) {
            $this->mailboxLink = \imap_open($filePath, '', '', CL_EXPUNGE);
        } else {
            $this->mailboxLink = \imap_open($filePath, '', '', OP_READONLY);
        }

        if (!$this->mailboxLink) {
            $this->errorMessage = 'Cannot open the mailbox file to ' . $filePath . $this->bmhNewLine . 'Error MSG: ' . \imap_last_error();
            $this->output();

            return false;
        }

        $this->output('Opened ' . $filePath);

        return true;
    }

    /**
     * open a mail box
     *
     * @return bool
     */
    public function openMailbox(): bool
    {
        // before starting the processing, let's check the delete flag and do global deletes if true
        if (\trim($this->deleteMsgDate) !== '') {
            echo 'processing global delete based on date of ' . $this->deleteMsgDate . '<br />';
            $this->globalDelete();
        }

        // disable move operations if server is Gmail ... Gmail does not support mailbox creation
        if (\stripos($this->mailhost, 'gmail') !== false) {
            $this->moveSoft = false;
            $this->moveHard = false;
        }

        $port = $this->port . '/' . $this->service . '/' . $this->serviceOption;

        \set_time_limit(self::SECONDS_TIMEOUT);

        if (!$this->testMode) {
            $this->mailboxLink = \imap_open('{' . $this->mailhost . ':' . $port . '}' . $this->boxname, $this->mailboxUserName, $this->mailboxPassword, CL_EXPUNGE);
        } else {
            $this->mailboxLink = \imap_open('{' . $this->mailhost . ':' . $port . '}' . $this->boxname, $this->mailboxUserName, $this->mailboxPassword, OP_READONLY);
        }

        if (!$this->mailboxLink) {
            $this->errorMessage = 'Cannot create ' . $this->service . ' connection to ' . $this->mailhost . $this->bmhNewLine . 'Error MSG: ' . \imap_last_error();
            $this->output();

            return false;
        }

        $this->output('Connected to: ' . $this->mailhost . ' (' . $this->mailboxUserName . ')');

        return true;
    }

    /**
     * output additional msg for debug
     *
     * @param mixed $msg          if not given, output the last error msg
     * @param int   $verboseLevel the output level of this message
     */
    public function output($msg = '', int $verboseLevel = self::VERBOSE_SIMPLE)
    {
        if ($this->verbose >= $verboseLevel) {
            if ($msg) {
                echo $msg . $this->bmhNewLine;
            } else {
                echo $this->errorMessage . $this->bmhNewLine;
            }
        }
    }

    /**
     * Function to process each individual message.
     *
     * @param int    $pos          message number
     * @param string $type         DNS or BODY type
     * @param int    $totalFetched total number of messages in mailbox
     *
     * @return array|false <p>"$result"-array or false</p>
     */
    public function processBounce(int $pos, string $type, int $totalFetched)
    {
        $header = \imap_headerinfo($this->mailboxLink, $pos);
        $subject = isset($header->subject) ? \strip_tags($header->subject) : '[NO SUBJECT]';
        $body = '';
        $headerFull = \imap_fetchheader($this->mailboxLink, $pos);
        $bodyFull = \imap_body($this->mailboxLink, $pos);

        if ($type == 'DSN') {
            // first part of DSN (Delivery Status Notification), human-readable explanation
            $dsnMsg = \imap_fetchbody($this->mailboxLink, $pos, '1');
            $dsnMsgStructure = \imap_bodystruct($this->mailboxLink, $pos, '1');

            if ($dsnMsgStructure->encoding == 4) {

                $dsnMsg = \quoted_printable_decode($dsnMsg);
            } elseif ($dsnMsgStructure->encoding == 3) {
                $dsnMsg = \base64_decode($dsnMsg, true);
            }

            // second part of DSN (Delivery Status Notification), delivery-status
            $dsnReport = \imap_fetchbody($this->mailboxLink, $pos, '2');

            // process bounces by rules
            $result = bmhDSNRules($dsnMsg, $dsnReport, $this->debugDsnRule);
            $result = \is_callable($this->customDSNRulesCallback) ? \call_user_func($this->customDSNRulesCallback, $result, $dsnMsg, $dsnReport, $this->debugDsnRule) : $result;
        } elseif ($type == 'BODY') {
            /** @noinspection PhpUsageOfSilenceOperatorInspection */
            $structure = @\imap_fetchstructure($this->mailboxLink, $pos);

            if (!\is_object($structure)) {
                return false;
            }

            switch ($structure->type) {
                case 0: // Content-type = text
                    $body = \imap_fetchbody($this->mailboxLink, $pos, '1');
                    $result = bmhBodyRules($body, $structure, $this->debugBodyRule);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                    $result = \is_callable($this->customBodyRulesCallback) ? \call_user_func($this->customBodyRulesCallback, $result, $body, $structure, $this->debugBodyRule) : $result;

                    break;

                case 1: // Content-type = multipart
                    $body = \imap_fetchbody($this->mailboxLink, $pos, '1');

                    // Detect encoding and decode - only base64
                    if ($structure->parts[0]->encoding == 4) {
                        $body = \quoted_printable_decode($body);
                    } elseif ($structure->parts[0]->encoding == 3) {
                        $body = \base64_decode($body, true);
                    }

                    $result = bmhBodyRules($body, $structure, $this->debugBodyRule);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                    $result = \is_callable($this->customBodyRulesCallback) ? \call_user_func($this->customBodyRulesCallback, $result, $body, $structure, $this->debugBodyRule) : $result;

                    break;

                case 2: // Content-type = message
                    $body = \imap_body($this->mailboxLink, $pos);

                    if ($structure->encoding == 4) {

                        $body = \quoted_printable_decode($body);
                    } elseif ($structure->encoding == 3) {
                        $body = \base64_decode($body, true);
                    }

                    $body = \substr($body, 0, 1000);
                    $result = bmhBodyRules($body, $structure, $this->debugBodyRule);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                    $result = \is_callable($this->customBodyRulesCallback) ? \call_user_func($this->customBodyRulesCallback, $result, $body, $structure, $this->debugBodyRule) : $result;

                    break;

                default: // un-support Content-type
                    $this->output('Msg #' . $pos . ' is unsupported Content-Type:' . $structure->type, self::VERBOSE_REPORT);

                    return false;
            }
        } else {
            // internal error
            $this->errorMessage = 'Internal Error: unknown type';

            return false;
        }

        $email = $result['email'];
        $bounceType = $result['bounce_type'];

        // workaround: I think there is a error in one of the reg-ex in "phpmailer-bmh_rules.php".
        if ($email && \strpos($email, 'TO:<') !== false) {
            $email = \str_replace('TO:<', '', $email);
        }

        if ($this->moveHard && $result['bounce_type'] == 'hard') {
            $remove = 'moved (hard)';
        } elseif ($this->moveSoft && $result['bounce_type'] == 'soft') {
            $remove = 'moved (soft)';
        } elseif ($this->disableDelete) {
            $remove = 0;
        } else {
            $remove = $result['remove'];
        }

        $ruleNumber = $result['rule_no'];
        $ruleCategory = $result['rule_cat'];
        $status_code = $result['status_code'];
        $action = $result['action'];
        $diagnostic_code = $result['diagnostic_code'];
        $xheader = false;

        if ($ruleNumber === '0000') {
            // unrecognized
            if (
                \trim($email) === ''
                &&
                \property_exists($header, 'fromaddress') === true
            ) {
                $email = $header->fromaddress;
            }

            if ($this->testMode) {

                $this->output('Match: ' . $ruleNumber . ':' . $ruleCategory . '; ' . $bounceType . '; ' . $email);
            } else {
                // code below will use the Callback function, but return no value
                $params = [
                    $pos,
                    $bounceType,
                    $email,
                    $subject,
                    $header,
                    $remove,
                    $ruleNumber,
                    $ruleCategory,
                    $totalFetched,
                    $body,
                    $headerFull,
                    $bodyFull,
                    $status_code,
                    $action,
                    $diagnostic_code,
                ];
                \call_user_func_array($this->actionFunction, $params);
            }
        } else {

            // match rule, do bounce action
            if ($this->testMode) {
                $this->output('Match: ' . $ruleNumber . ':' . $ruleCategory . '; ' . $bounceType . '; ' . $email);

                return true;
class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}
            }

            $params = [
                $pos,
                $bounceType,
                $email,
                $subject,
                $xheader,
                $remove,
                $ruleNumber,
                $ruleCategory,
                $totalFetched,
                $body,
                $headerFull,
                $bodyFull,
                $status_code,
                $action,
                $diagnostic_code,
            ];
            \call_user_func_array($this->actionFunction, $params);

            return $result;
        }

        return false;
    }

    /**
     * process the messages in a mailbox
     *
     * @param bool|int $max $max maximum limit messages processed in one batch,
     *                      if not given uses the property $maxMessages
     *
     * @return bool
     */
    public function processMailbox($max = false): bool
    {
        if (
            empty($this->actionFunction)
            ||
            !\is_callable($this->actionFunction)
        ) {
            $this->errorMessage = 'Action function not found!';
            $this->output();

            return false;
        }

        if ($this->moveHard && ($this->disableDelete === false)) {
            $this->disableDelete = true;
        }

        if (!empty($max)) {
            $this->maxMessages = $max;
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        }

        // initialize counters
        $totalCount = \imap_num_msg($this->mailboxLink);
        $fetchedCount = $totalCount;
        $processedCount = 0;
        $unprocessedCount = 0;
        $deletedCount = 0;
        $movedCount = 0;
        $this->output('Total: ' . $totalCount . ' messages ');

        // process maximum number of messages
        if ($fetchedCount > $this->maxMessages) {
            $fetchedCount = $this->maxMessages;
            $this->output('Processing first ' . $fetchedCount . ' messages ');
        }

        if ($this->testMode) {
            $this->output('Running in test mode, not deleting messages from mailbox<br />');
        } else {
            if ($this->disableDelete) {
                if ($this->moveHard) {
                    $this->output('Running in move mode<br />');
                } else {
                    $this->output('Running in disableDelete mode, not deleting messages from mailbox<br />');
                }
            } else {
                $this->output('Processed messages will be deleted from mailbox<br />');
            }
        }

        for ($x = 1; $x <= $fetchedCount; ++$x) {

            // fetch the messages one at a time
            if ($this->useFetchstructure) {
                /** @noinspection PhpUsageOfSilenceOperatorInspection */
                $structure = @\imap_fetchstructure($this->mailboxLink, $x);

                if (
                    $structure
                    &&
                    \is_object($structure)
                    &&
                    $structure->type == 1
                    &&
                    $structure->ifsubtype
                    &&
                    $structure->ifparameters
                    &&
                    \strtoupper($structure->subtype) == 'REPORT'
                    &&
                    $this->isParameter($structure->parameters, 'REPORT-TYPE', 'delivery-status')
                ) {
                    $processedResult = $this->processBounce($x, 'DSN', $totalCount);
                } else {
                    // not standard DSN msg
                    $this->output('Msg #' . $x . ' is not a standard DSN message', self::VERBOSE_REPORT);

                    if ($this->debugBodyRule) {
                        if ($structure->ifdescription) {
                            $this->output("  Content-Type : {$structure->description}", self::VERBOSE_DEBUG);
                        } else {
                            $this->output('  Content-Type : unsupported', self::VERBOSE_DEBUG);
                        }
                    }

                    $processedResult = $this->processBounce($x, 'BODY', $totalCount);
                }
            } else {
                $header = \imap_fetchheader($this->mailboxLink, $x);

                // Could be multi-line, if the new line begins with SPACE or HTAB
                if ($header && \preg_match("/Content-Type:((?:[^\n]|\n[\t ])+)(?:\n[^\t ]|$)/i", $header, $match)) {
                    if (
                        \preg_match("/multipart\/report/i", $match[1])
                        &&
                        \preg_match("/report-type=[\"']?delivery-status[\"']?/i", $match[1])
                    ) {
                        // standard DSN msg
                        $processedResult = $this->processBounce($x, 'DSN', $totalCount);
                    } else {
                        // not standard DSN msg
                        $this->output('Msg #' . $x . ' is not a standard DSN message', self::VERBOSE_REPORT);

                        if ($this->debugBodyRule) {
                            $this->output("  Content-Type : {$match[1]}", self::VERBOSE_DEBUG);
                        }

                        $processedResult = $this->processBounce($x, 'BODY', $totalCount);
                    }
                } else {
                    // didn't get content-type header
                    $this->output('Msg #' . $x . ' is not a well-formatted MIME mail, missing Content-Type', self::VERBOSE_REPORT);

                    if ($this->debugBodyRule) {
                        $this->output('  Headers: ' . $this->bmhNewLine . $header . $this->bmhNewLine, self::VERBOSE_DEBUG);
                    }

                    $processedResult = $this->processBounce($x, 'BODY', $totalCount);
                }
            }

            $deleteFlag[$x] = false;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
            $moveFlag[$x] = false;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}

            if ($processedResult !== false) {
                ++$processedCount;

                if (!$this->disableDelete) {
                    // delete the bounce if not in disableDelete mode
                    if (!$this->testMode) {
                        /** @noinspection PhpUsageOfSilenceOperatorInspection */
                        @\imap_delete($this->mailboxLink, $x);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
                    }

                    $deleteFlag[$x] = true;
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
                    ++$deletedCount;
                } elseif ($this->moveHard && $processedResult['bounce_type'] === 'hard') {
                    // check if the move directory exists, if not create it
                    if (!$this->testMode) {
                        $this->mailboxExist($this->hardMailbox);
                    }

                    // move the message
                    if (!$this->testMode) {
                        /** @noinspection PhpUsageOfSilenceOperatorInspection */
                        @\imap_mail_move($this->mailboxLink, (string) $x, $this->hardMailbox);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
                    }

                    $moveFlag[$x] = true;
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
                    ++$movedCount;
                } elseif ($this->moveSoft && $processedResult['bounce_type'] === 'soft') {
                    // check if the move directory exists, if not create it
                    if (!$this->testMode) {
                        $this->mailboxExist($this->softMailbox);
                    }

                    // move the message
                    if (!$this->testMode) {
                        /** @noinspection PhpUsageOfSilenceOperatorInspection */
                        @\imap_mail_move($this->mailboxLink, (string) $x, $this->softMailbox);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
                    }

                    $moveFlag[$x] = true;
                    ++$movedCount;
                }
            } else {
                // not processed
                ++$unprocessedCount;
                if (!$this->disableDelete && $this->purgeUnprocessed) {
                    // delete this bounce if not in disableDelete mode, and the flag BOUNCE_PURGE_UNPROCESSED is set
                    if (!$this->testMode) {
                        /** @noinspection PhpUsageOfSilenceOperatorInspection */
                        @\imap_delete($this->mailboxLink, $x);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
                    }

                    $deleteFlag[$x] = true;
                    ++$deletedCount;
                }

                // check if the move directory exists, if not create it
                $this->mailboxExist($this->unprocessedBox);
                // move the message
                /** @noinspection PhpUsageOfSilenceOperatorInspection */
                @\imap_mail_move($this->mailboxLink, (string) $x, $this->unprocessedBox);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
                $moveFlag[$x] = true;
            }

            \flush();
        }

        $this->output($this->bmhNewLine . 'Closing mailbox, and purging messages');

        /** @noinspection PhpUsageOfSilenceOperatorInspection */
        @\imap_expunge($this->mailboxLink);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
        /** @noinspection PhpUsageOfSilenceOperatorInspection */
        @\imap_close($this->mailboxLink);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}

        $this->output('Read: ' . $fetchedCount . ' messages');
        $this->output($processedCount . ' action taken');
        $this->output($unprocessedCount . ' no action taken');
        $this->output($deletedCount . ' messages deleted');
        $this->output($movedCount . ' messages moved');

        return true;
    }
}

Push — master ( 9127cb...6c003f )

BounceMailHandler F

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

9 Methods

How to fix Duplicated Code Complexity

Duplicated Code

Complex Class

Available Fixes

Available Fixes

voku / PHPMailer-BMH

Push — master ( 9127cb...6c003f )

BounceMailHandler F

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

9 Methods

How to fix Duplicated Code Complexity

Duplicated Code

Complex Class

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like
