e107inc/e107

Avoid deeply nested control flow statements.

Open

                                if (e107::callMethod($cls, 'bounce', $bounceInfo) === false)
                                {
                                    e107::getLog()->add($plug . ' bounce process failed', $bounceInfo, E_LOG_FATAL, 'BOUNCE', LOG_TO_ROLLING);
                                }

Avoid too many return statements within this method.

Open

        return true;

Avoid too many return statements within this method.

Open

            return false;

Avoid too many return statements within this method.

Open

                return LAN_MAILOUT_214;

Avoid too many return statements within this method.

Open

                return LAN_MAILOUT_215;

Avoid too many return statements within this method.

Open

                    return LAN_MAILOUT_214;

Avoid too many return statements within this method.

Open

                return LAN_MAILOUT_217;

Avoid too many return statements within this method.

Open

        return LAN_MAILOUT_216 . ' (' . $status . ')';        // General coding error

Return type of selectEmails() is undeclared type \handle

Open

    public function selectEmails($count = 1)

• Exclude checks
  • Disable engine
  • Disable check

Suspicious array access to bool

Open

        if ($counters['add'] == 0)

• Exclude checks
  • Disable engine
  • Disable check

Invalid operator: left operand of - is null (expected number)

Open

                $targetData['mail_status'] = max($targetData['mail_status'] - 1, MAIL_STATUS_PENDING);        // One off retry count

• Exclude checks
  • Disable engine
  • Disable check

Return type of saveEmail() is undeclared type \mail

Open

    public function saveEmail($emailData, $isNew = false)

• Exclude checks
  • Disable engine
  • Disable check

Saw a token Phan may have failed to parse after '* @return bool|array| - TRUE on success, FALSE on failure': after bool|array, saw '|'

Open

     * @return bool|array| - TRUE on success, FALSE on failure

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "mail_source_id" of array type array{}

Open

            if ($this->currentBatchInfo['mail_source_id'] != $email['mail_source_id'])

• Exclude checks
  • Disable engine
  • Disable check

Returning type bool|int but selectTargetStatus() is declared to return \Returns

Open

        return $result;

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \e107Email->AltBody

Open

            print_a($this->mailer->AltBody);

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "mail_status" of array type array{}

Open

                $targetData['mail_status'] = max($targetData['mail_status'] - 1, MAIL_STATUS_PENDING);        // One off retry count

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset "mail_status" of array type array{}

Open

            if ($targetData['mail_status'] > MAIL_STATUS_PENDING)

• Exclude checks
  • Disable engine
  • Disable check

Invalid operator: unary operand of (expr)-- is \handle|true (expected int or string or float)

Open

                $count--;

• Exclude checks
  • Disable engine
  • Disable check

Return type of selectTargetStatus() is undeclared type \Returns

Open

    public function selectTargetStatus($handle, $start = 0, $count = 0, $fields = '*', $filters = false, $orderField = 'mail_target_id', $sortOrder = 'asc')

• Exclude checks
  • Disable engine
  • Disable check

Assigning array to property but \e107MailManager->mailOverrides is false

Open

        $this->mailOverrides = $overrides;

• Exclude checks
  • Disable engine
  • Disable check

Invalid operator: unary operand of (expr)-- is false (expected int or string or float)

Open

            $this->queryActive--;

• Exclude checks
  • Disable engine
  • Disable check

Return type of selectEmailStatus() is undeclared type \Returns

Open

    public function selectEmailStatus($start = 0, $count = 0, $fields = '*', $filters = false, $orderField = 'mail_source_id', $sortOrder = 'asc')

• Exclude checks
  • Disable engine
  • Disable check

Assigning array{mailer:mixed} to property but \e107MailManager->mailOverrides is false

Open

        $this->mailOverrides = $overrides;

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \e107Email->Body

Open

            print_a($this->mailer->Body);

• Exclude checks
  • Disable engine
  • Disable check

Returning type bool|int but selectEmailStatus() is declared to return \Returns

Open

        return $result;

• Exclude checks
  • Disable engine
  • Disable check

Returning type array<string></string> but mailRetrieveCounters() is declared to return bool

Open

            return $this->mailCounters[$handle];

• Exclude checks
  • Disable engine
  • Disable check