wikimedia/mediawiki-core

Method __construct has 20 arguments (exceeds 4 allowed). Consider refactoring.

Open

        ServiceOptions $options,
        PageIdentity $page,
        RevisionStore $revisionStore,
        RevisionRenderer $revisionRenderer,
        SlotRoleRegistry $slotRoleRegistry,

StatusValue has 23 functions (exceeds 20 allowed). Consider refactoring.

Open

class StatusValue implements Stringable {

    /**
     * @var bool
     * @internal Only for use by Status. Use {@link self::isOK()} or {@link self::setOK()}.

DjVuHandler has 23 functions (exceeds 20 allowed). Consider refactoring.

Open

class DjVuHandler extends ImageHandler {
    private const EXPENSIVE_SIZE_LIMIT = 10_485_760; // 10MiB

    // Constants for getHandlerState
    private const STATE_DJVU_IMAGE = 'djvuImage';

PostgresInstaller has 23 functions (exceeds 20 allowed). Consider refactoring.

Open

class PostgresInstaller extends DatabaseInstaller {

    /** @inheritDoc */
    protected $globalNames = [
        'wgDBserver',

SpecialBlock has 23 functions (exceeds 20 allowed). Consider refactoring.

Open

class SpecialBlock extends FormSpecialPage {

    private BlockUtils $blockUtils;
    private BlockPermissionCheckerFactory $blockPermissionCheckerFactory;
    private BlockUserFactory $blockUserFactory;

Method __construct has 20 arguments (exceeds 4 allowed). Consider refactoring.

Open

        ServiceOptions $config,
        array $parsoidSettings,
        ObjectFactory $objectFactory,
        Language $contentLanguage,
        StatsdDataFactoryInterface $stats,

BotPassword has 23 functions (exceeds 20 allowed). Consider refactoring.

Open

class BotPassword {

    public const APPID_MAXLENGTH = 32;

    /**

AbstractContent has 23 functions (exceeds 20 allowed). Consider refactoring.

Open

abstract class AbstractContent implements Content {
    /**
     * Name of the content model this Content object represents.
     * Use with CONTENT_MODEL_XXX constants
     *

Similar blocks of code found in 4 locations. Consider refactoring.

Open

FilterItem.prototype.toggleConflicted = function ( conflicted ) {
    conflicted = conflicted === undefined ? !this.conflicted : conflicted;

    if ( this.conflicted !== conflicted ) {
        this.conflicted = conflicted;

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

FilterGroup.prototype.toggleVisible = function ( isVisible ) {
    isVisible = isVisible === undefined ? !this.visible : isVisible;

    if ( this.visible !== isVisible ) {
        this.visible = isVisible;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    hide: function( speed ) {
        if ( standardSpeed( speed ) ) {
            return this._hide.apply( this, arguments );
        } else {
            var args = _normalizeArguments.apply( this, arguments );

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

            this.rows[ row ] = {
                isFull: false,
                width: 0,
                items: [],
                $element: $( '<div>' )

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    [ 'state', 'always', 'catch', 'pipe', 'then', 'promise', 'progress', 'done', 'fail' ]
        .forEach( ( method ) => {
            lifecycle[ method ] = function () {
                OO.ui.warnDeprecation(
                    'Using the return value of openWindow as a promise is deprecated. ' +

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

FilterItem.prototype.toggleIncluded = function ( included ) {
    included = included === undefined ? !this.included : included;

    if ( this.included !== included ) {
        this.included = included;

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

FilterItem.prototype.toggleFullyCovered = function ( isFullyCovered ) {
    isFullyCovered = isFullyCovered === undefined ? !this.fullycovered : isFullyCovered;

    if ( this.fullyCovered !== isFullyCovered ) {
        this.fullyCovered = isFullyCovered;

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

            this.rows[ row ] = {
                isFull: false,
                width: 0,
                items: [],
                $element: $( '<div>' )

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    show: function( speed ) {
        if ( standardSpeed( speed ) ) {
            return this._show.apply( this, arguments );
        } else {
            var args = _normalizeArguments.apply( this, arguments );

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            } else if (format === 'ddd') {
                ii = indexOf.call(this._shortWeekdaysParse, llc);
                return ii !== -1 ? ii : null;
            } else {
                ii = indexOf.call(this._minWeekdaysParse, llc);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            if (format === 'MMM') {
                ii = indexOf.call(this._shortMonthsParse, llc);
                return ii !== -1 ? ii : null;
            } else {
                ii = indexOf.call(this._longMonthsParse, llc);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    [ 'state', 'always', 'catch', 'pipe', 'then', 'promise', 'progress', 'done', 'fail' ]
        .forEach( ( method ) => {
            lifecycle[ method ] = function () {
                OO.ui.warnDeprecation(
                    'Using the return value of closeWindow as a promise is deprecated. ' +

• Read up
  Read up