Method sendInternal has 145 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected static function sendInternal(
        array $to,
        MailAddress $from,
        $subject,
        $body,

Function sendInternal has a Cognitive Complexity of 38 (exceeds 5 allowed). Consider refactoring.

Open

    protected static function sendInternal(
        array $to,
        MailAddress $from,
        $subject,
        $body,

• Read up
  Read up

File UserMailer.php has 278 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/**
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or

Function send has a Cognitive Complexity of 19 (exceeds 5 allowed). Consider refactoring.

Open

    public static function send( $to, $from, $subject, $body, $options = [] ) {
        $services = MediaWikiServices::getInstance();
        $allowHTMLEmail = $services->getMainConfig()->get(
            MainConfigNames::AllowHTMLEmail );

• Read up
  Read up

Method send has 59 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function send( $to, $from, $subject, $body, $options = [] ) {
        $services = MediaWikiServices::getInstance();
        $allowHTMLEmail = $services->getMainConfig()->get(
            MainConfigNames::AllowHTMLEmail );

Consider simplifying this complex logical expression.

Open

        if (
            !(
                !is_array( $body ) &&
                strlen( $body ) >= $minBodyLen
            )

Method send has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public static function send( $to, $from, $subject, $body, $options = [] ) {

Method sendInternal has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

        array $to,
        MailAddress $from,
        $subject,
        $body,
        $options = []

Avoid too many return statements within this method.

Open

                return Status::newFatal( 'php-mail-error', self::$mErrorString );

Avoid too many return statements within this method.

Open

            return Status::newGood();

Avoid too many return statements within this method.

Open

                return Status::newFatal( 'pear-mail-error', $mail_object->getMessage() );

Avoid too many return statements within this method.

Open

            return Status::newGood();

Avoid too many return statements within this method.

Open

                return Status::newFatal( 'php-mail-error-unknown' );

Avoid too many return statements within this method.

Open

                    return $status;

Avoid too many return statements within this method.

Open

                return Status::newGood();

Avoid too many return statements within this method.

Open

            return Status::newFatal( 'php-mail-error', $ret );