CreationTaskBase::execute() - Code Metrics - enwikipedia-acc/waca - Measure and Improve Code Quality continuously with Scrutinizer

CreationTaskBase::execute() C
last analyzed 2025-12-01 20:20 UTC

↳ Parent: CreationTaskBase

Complexity

Conditions	13
Paths	123

Size

Total Lines	73
Code Lines	43

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	43
dl	0
loc	73
rs	6.425
c	1
b	0
f	0
cc	13
nc	123
nop	0

How to fix Long Method Complexity

<?php
/******************************************************************************
 * Wikipedia Account Creation Assistance tool                                 *
 * ACC Development Team. Please see team.json for a list of contributors.     *
 *                                                                            *
 * This is free and unencumbered software released into the public domain.    *
 * Please see LICENSE.md for the full licencing statement.                    *
 ******************************************************************************/

namespace Waca\Background;

use Exception;
use Waca\DataObjects\Request;
use Waca\DataObjects\User;
use Waca\ExceptionHandler;
use Waca\Exceptions\ApplicationLogicException;
use Waca\Helpers\Interfaces\IMediaWikiClient;
use Waca\Helpers\Logger;
use Waca\Helpers\MediaWikiHelper;
use Waca\Helpers\RequestEmailHelper;
use Waca\RequestStatus;

abstract class CreationTaskBase extends BackgroundTaskBase
{
    /** @var Request */
    private $request;
    /**
     * @var MediaWikiHelper
     * Don't use this directly.
     */
    private $mwHelper = null;

    public function execute()
    {
        $this->request = $this->getRequest();
        $user = $this->getTriggerUser();
        $parameters = $this->getParameters();

        if ($this->request->getStatus() !== RequestStatus::JOBQUEUE) {
            $this->markCancelled('Request is not deferred to the job queue');

            return;
        }

        if ($this->request->getEmailSent() != 0 && !isset($parameters->emailText)) {
            $this->markFailed('Request has already been sent a templated email');

            return;
        }

        if ($this->request->getEmail() === $this->getSiteConfiguration()->getDataClearEmail()) {
            $this->markFailed('Private data of request has been purged.');

            return;
        }

        $emailText = null;
        $ccMailingList = null;
        $logTarget = null;

        if (isset($parameters->emailText) && isset($parameters->ccMailingList)) {
            $emailText = $parameters->emailText;
            $ccMailingList = $parameters->ccMailingList;
            $logTarget = "custom-y";
        }

        if ($this->getEmailTemplate() !== null) {
            $emailText = $this->getEmailTemplate()->getText();
            $ccMailingList = false;
            $logTarget = $this->getEmailTemplate()->getId();
        }

        if ($emailText === null || $ccMailingList === null) {
            $this->markFailed('Unable to get closure email text');

            return;
        }

        try {
            $this->performCreation($user);

            $this->request->setStatus(RequestStatus::CLOSED);
            $this->request->setQueue(null);
            $this->request->setReserved(null);
            $this->request->setEmailSent(true);
            $this->request->save();

            // Log the closure as the user
            $logComment = $this->getEmailTemplate() === null ? $emailText : null;
            Logger::closeRequest($this->getDatabase(), $this->request, $logTarget, $logComment, $this->getTriggerUser());
            Logger::closeRequest($this->getDatabase(), $this->request, /** @scrutinizer ignore-type */ $logTarget, $logComment, $this->getTriggerUser());

            $requestEmailHelper = new RequestEmailHelper($this->getEmailHelper());
            $requestEmailHelper->sendMail($this->request, $emailText, $this->getTriggerUser(), $ccMailingList);
        }
        catch (Exception $ex) {
            if (mb_strlen($ex->getMessage()) > 255) {
                ExceptionHandler::logExceptionToDisk($ex, $this->getSiteConfiguration());
            }

            $this->markFailed(substr($ex->getMessage(), 0, 255));

            return;
        }

        $this->markComplete();
    }

    /**
     * @return IMediaWikiClient
     */
    protected abstract function getMediaWikiClient();

    protected function getMediaWikiHelper()
    {
        if ($this->mwHelper === null) {
            $this->mwHelper = new MediaWikiHelper($this->getMediaWikiClient(), $this->getSiteConfiguration());
        }

        return $this->mwHelper;
    }

    /** @noinspection PhpUnusedParameterInspection */
    protected function getCreationReason(Request $request, User $user)
    {
        $requestId = $request->getId();
        return "Requested account at [[WP:ACC]], request #[[ACC:$requestId|$requestId]]";
    }

    /**
     * @param string $name
     *
     * @return bool
     */
    protected function checkAccountExists($name)
    {
        return $this->getMediaWikiHelper()->checkAccountExists($name);
    }

    protected function markFailed($reason = null, bool $acknowledged = false)
    {
        $this->request->setStatus(RequestStatus::HOSPITAL);
        $this->request->setQueue(null);
        $this->request->save();

        $this->getNotificationHelper()->requestCreationFailed($this->request, $this->getTriggerUser());

        Logger::hospitalised($this->getDatabase(), $this->request);

        // auto-acknowledge failed creation tasks, as these land in the hospital queue anyway.
        parent::markFailed($reason, true);
        Logger::backgroundJobAcknowledged($this->getDatabase(), $this->getJob(), "Auto-acknowledged due to request deferral to hospital queue");
    }

    /**
     * @param $user
     *
     * @throws ApplicationLogicException
     */
    protected function performCreation($user)
    {
        $mw = $this->getMediaWikiHelper();

        $reason = $this->getCreationReason($this->request, $user);

        if ($this->checkAccountExists($this->request->getName())) {
            throw new ApplicationLogicException('Account already exists');
        }

        $mw->createAccount($this->request->getName(), $this->request->getEmail(), $reason);

        if (!$this->checkAccountExists($this->request->getName())) {
            throw new ApplicationLogicException('Account creation appeared to succeed but account does not exist.');
        }

        $this->request->setStatus(RequestStatus::CLOSED);
        $this->request->save();
    }
}


enwikipedia-acc / waca

CreationTaskBase::execute() C last analyzed 2025-12-01 20:20 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

CreationTaskBase::execute() C
last analyzed 2025-12-01 20:20 UTC