BounceMailHandler\BounceMailHandler::processMailbox()

Complexity

Conditions	36
Paths	4065

Size

Total Lines	195
Code Lines	112

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	54
CRAP Score	144.5273

Importance

Changes

0

<?php
  public function processMailbox($max = false)
  {
    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;

The property $maxMessages was declared of type integer, but $max is of type boolean. Maybe add a type cast?

    }

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

'Total: ' . $totalCount . ' messages ' is of type string, but the function expects a boolean.

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

'Processing first ' . $fetchedCount . ' messages ' is of type string, but the function expects a boolean.

    }

    if ($this->testMode) {
      $this->output('Running in test mode, not deleting messages from mailbox<br />');

'Running in test mode, not deleting messages from mailbox<br />' is of type string, but the function expects a boolean.

    } else {
      if ($this->disableDelete) {
        if ($this->moveHard) {
          $this->output('Running in move mode<br />');

'Running in move mode<br />' is of type string, but the function expects a boolean.

        } else {
          $this->output('Running in disableDelete mode, not deleting messages from mailbox<br />');

'Running in disableDelete mode, not deleting messages from mailbox<br />' is of type string, but the function expects a boolean.

        }
      } else {
        $this->output('Processed messages will be deleted from mailbox<br />');

'Processed messages will be deleted from mailbox<br />' is of type string, but the function expects a boolean.

      }
    }

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

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

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

'Msg #' . $x . ' is not a standard DSN message' is of type string, but the function expects a boolean.

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

' Content-Type : unsupported' is of type string, but the function expects a boolean.

            }
          }

          $processed = $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 (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
            $processed = $this->processBounce($x, 'DSN', $totalCount);
          } else {
            // not standard DSN msg
            $this->output('Msg #' . $x . ' is not a standard DSN message', self::VERBOSE_REPORT);

'Msg #' . $x . ' is not a standard DSN message' is of type string, but the function expects a boolean.

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

            $processed = $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);

'Msg #' . $x . ' is not a well-formatted MIME mail, missing Content-Type' is of type string, but the function expects a boolean.

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

' Headers: ' . $this->bmhNewLine . $header . $this->bmhNewLine is of type string, but the function expects a boolean.

          }

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

      $deleteFlag[$x] = false;

$deleteFlag was never initialized. Although not strictly required by PHP, it is generally a good practice to add $deleteFlag = array(); before regardless.

      $moveFlag[$x] = false;

$moveFlag was never initialized. Although not strictly required by PHP, it is generally a good practice to add $moveFlag = array(); before regardless.

      if ($processed) {
        $processedCount++;

        if (!$this->disableDelete) {
          // delete the bounce if not in disableDelete mode
          if (!$this->testMode) {
            /** @noinspection PhpUsageOfSilenceOperatorInspection */
            @imap_delete($this->mailboxLink, $x);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

          }

          $deleteFlag[$x] = true;

The variable $deleteFlag does not seem to be defined for all execution paths leading up to this point.

          $deletedCount++;
        } elseif ($this->moveHard) {
          // 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, $x, $this->hardMailbox);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

          }

          $moveFlag[$x] = true;

The variable $moveFlag does not seem to be defined for all execution paths leading up to this point.

          $movedCount++;
        } elseif ($this->moveSoft) {
          // 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, $x, $this->softMailbox);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

          }

          $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);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

          }

          $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, $x, $this->unprocessedBox);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        $moveFlag[$x] = true;
      }

      flush();
    }

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

$this->bmhNewLine . 'Closing mailbox, and purging messages' is of type string, but the function expects a boolean.

    /** @noinspection PhpUsageOfSilenceOperatorInspection */
    @imap_expunge($this->mailboxLink);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

    imap_close($this->mailboxLink);

    $this->output('Read: ' . $fetchedCount . ' messages');

'Read: ' . $fetchedCount . ' messages' is of type string, but the function expects a boolean.

    $this->output($processedCount . ' action taken');

$processedCount . ' action taken' is of type string, but the function expects a boolean.

    $this->output($unprocessedCount . ' no action taken');

$unprocessedCount . ' no action taken' is of type string, but the function expects a boolean.

    $this->output($deletedCount . ' messages deleted');

$deletedCount . ' messages deleted' is of type string, but the function expects a boolean.

    $this->output($movedCount . ' messages moved');

$movedCount . ' messages moved' is of type string, but the function expects a boolean.

    return true;
  }