BounceMailHandler\BounceMailHandler::processBounce()

Complexity

Conditions	25
Paths	964

Size

Total Lines	160
Code Lines	112

Duplication

Lines	51
Ratio	31.88 %

Code Coverage

Tests	50
CRAP Score	142.3367

Importance

Changes	11
Bugs	3	Features	5

<?php
  public function processBounce($pos, $type, $totalFetched)
  {
    $header = imap_header($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);
      }

      // 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);
    } elseif ($type == 'BODY') {
      $structure = imap_fetchstructure($this->mailboxLink, $pos);

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

$structure is of type object, but the function expects a string.

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

          $result = bmhBodyRules($body, $structure, $this->debugBodyRule);

$structure is of type object, but the function expects a string.

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

          $body = substr($body, 0, 1000);
          $result = bmhBodyRules($body, $structure, $this->debugBodyRule);

$structure is of type object, but the function expects a string.

          break;

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

'Msg #' . $pos . ' is unsupported Content-Type:' . $structure->type is of type string, but the function expects a boolean.

          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:<')) {
      $email = str_replace('TO:<', '', $email);
    }

    if ($this->moveHard && $result['remove'] == 1) {
      $remove = 'moved (hard)';
    } elseif ($this->moveSoft && $result['remove'] == 1) {
      $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 code seems to be duplicated across your project.

        $this->output('Match: ' . $ruleNumber . ':' . $ruleCategory . '; ' . $bounceType . '; ' . $email);

'Match: ' . $ruleNumber . ':' . $ruleCategory . '; ' . $bounceType . '; ' . $email is of type string, but the function expects a boolean.

      } else {
        // code below will use the Callback function, but return no value
        $params = array(
            $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 {

This code seems to be duplicated across your project.

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

'Match: ' . $ruleNumber . ':' . $ruleCategory . '; ' . $bounceType . '; ' . $email is of type string, but the function expects a boolean.

        return true;
      } else {
        $params = array(
            $pos,
            $bounceType,
            $email,
            $subject,
            $xheader,
            $remove,
            $ruleNumber,
            $ruleCategory,
            $totalFetched,
            $body,
            $headerFull,
            $bodyFull,
            $status_code,
            $action,
            $diagnostic_code,
        );

        return call_user_func_array($this->actionFunction, $params);
      }
    }

    return false;
  }