File PageEditStash.php has 328 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 checkCache has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.

Open

    public function checkCache( PageIdentity $page, Content $content, UserIdentity $user ) {
        $legacyUser = $this->userFactory->newFromUserIdentity( $user );
        if (
            // The context is not an HTTP POST request
            !$legacyUser->getRequest()->wasPosted() ||

• Read up
  Read up

Method checkCache has 77 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function checkCache( PageIdentity $page, Content $content, UserIdentity $user ) {
        $legacyUser = $this->userFactory->newFromUserIdentity( $user );
        if (
            // The context is not an HTTP POST request
            !$legacyUser->getRequest()->wasPosted() ||

Method parseAndCache has 66 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function parseAndCache( $pageUpdater, Content $content, UserIdentity $user, string $summary ) {
        $logger = $this->logger;

        if ( $pageUpdater instanceof WikiPage ) {
            wfDeprecated( __METHOD__ . ' with WikiPage instance', '1.42' );

Function parseAndCache has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    public function parseAndCache( $pageUpdater, Content $content, UserIdentity $user, string $summary ) {
        $logger = $this->logger;

        if ( $pageUpdater instanceof WikiPage ) {
            wfDeprecated( __METHOD__ . ' with WikiPage instance', '1.42' );

• Read up
  Read up

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

Open

        BagOStuff $cache,
        IConnectionProvider $dbProvider,
        LoggerInterface $logger,
        StatsdDataFactoryInterface $stats,
        UserEditTracker $userEditTracker,

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

Open

        $key,
        Content $pstContent,
        ParserOutput $parserOutput,
        $timestamp,
        UserIdentity $user

Avoid too many return statements within this method.

Open

        return self::ERROR_PARSE;

Avoid too many return statements within this method.

Open

                return self::ERROR_CACHE;