CountAction::executeApiAction() - Code Metrics - Inspection of "fix" - mdaniels5757/waca - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — rbac ( be68b4...52c28b )

by Michael

created 2020-04-26 02:29 UTC

CountAction::executeApiAction() A

↳ Parent: CountAction

Complexity

Conditions	3
Paths	3

Size

Total Lines	30
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
eloc	16
c	0
b	0
f	0
dl	0
loc	30
ccs	0
cts	20
cp	0
rs	9.7333
cc	3
nc	3
nop	1
crap	12

<?php
/******************************************************************************
 * Wikipedia Account Creation Assistance tool                                 *
 *                                                                            *
 * All code in this file is released into the public domain by the ACC        *
 * Development Team. Please see team.json for a list of contributors.         *
 ******************************************************************************/

namespace Waca\API\Actions;

use DOMElement;
use Waca\API\ApiException as ApiException;
use Waca\API\IApiAction as IApiAction;
use Waca\DataObjects\User;
use Waca\Tasks\ApiPageBase;
use Waca\WebRequest;

/**
 * API Count action
 */
class CountAction extends ApiPageBase implements IApiAction
{
    /**
     * The target user
     * @var User $user
     */
    private $user;

    public function executeApiAction(DOMElement $apiDocument)
    {
        $username = WebRequest::getString('user');
        if ($username === null) {
            throw new ApiException("Please specify a username");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        }

        $userElement = $this->document->createElement("user");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $userElement->setAttribute("name", $username);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $apiDocument->appendChild($userElement);

        $user = User::getByUsername($username, $this->getDatabase());

        if ($user === false) {
            $userElement->setAttribute("missing", "true");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

            return $apiDocument;
        }

        $this->user = $user;

        $userElement->setAttribute("level", $this->user->getStatus());
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $userElement->setAttribute("created", $this->getAccountsCreated());
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

        $userElement->setAttribute("today", $this->getToday());
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

        // Let the IRC bot handle the result of this.
        $this->fetchAdminData($userElement);

        return $apiDocument;
    }


    private function getAccountsCreated()
    {
        $query = <<<QUERY
        SELECT COUNT(*) AS count
        FROM log
            LEFT JOIN emailtemplate ON concat('Closed ', emailtemplate.id) = log.action
            INNER JOIN user ON log.user = user.id
        WHERE
            (oncreated = '1' OR log.action = 'Closed custom-y')
            AND log.objecttype = 'Request'
            AND user.username = :username;
QUERY;

        $statement = $this->getDatabase()->prepare($query);
        $statement->execute(array(":username" => $this->user->getUsername()));
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $result = $statement->fetchColumn();
        $statement->closeCursor();

        return $result;
    }


    private function getToday()
    {
        $query = <<<QUERY
        SELECT
            COUNT(*) AS count
        FROM log
            LEFT JOIN emailtemplate ON concat('Closed ', emailtemplate.id) = log.action
            INNER JOIN user ON log.user = user.id
        WHERE
            log.timestamp LIKE :date
            AND (oncreated = '1' OR log.action = 'Closed custom-y')
            AND user.username = :username;
QUERY;

        $statement = $this->getDatabase()->prepare($query);
        $statement->bindValue(":username", $this->user->getUsername());
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->bindValue(":date", date('Y-m-d') . "%");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->execute();
        $today = $statement->fetchColumn();
        $statement->closeCursor();

        return $today;
    }


    private function fetchAdminData(DOMElement $userElement)
    {
        $query = "SELECT COUNT(*) AS count FROM log WHERE log.user = :userid AND log.action = :action;";
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;

        $statement = $this->getDatabase()->prepare($query);
        $statement->bindValue(":userid", $this->user->getId());
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->bindValue(":action", "Suspended");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->execute();
        $sus = $statement->fetchColumn();
        $userElement->setAttribute("suspended", $sus);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->closeCursor();

        $statement->bindValue(":action", "Promoted");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->execute();
        $pro = $statement->fetchColumn();
        $userElement->setAttribute("promoted", $pro);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->closeCursor();

        $statement->bindValue(":action", "Approved");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->execute();
        $app = $statement->fetchColumn();
        $userElement->setAttribute("approved", $app);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->closeCursor();

        $statement->bindValue(":action", "Demoted");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->execute();
        $dem = $statement->fetchColumn();
        $userElement->setAttribute("demoted", $dem);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->closeCursor();

        $statement->bindValue(":action", "Declined");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->execute();
        $dec = $statement->fetchColumn();
        $userElement->setAttribute("declined", $dec);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->closeCursor();

        $statement->bindValue(":action", "Renamed");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->execute();
        $rnc = $statement->fetchColumn();
        $userElement->setAttribute("renamed", $rnc);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->closeCursor();

        $statement->bindValue(":action", "Edited");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->execute();
        $mec = $statement->fetchColumn();
        $userElement->setAttribute("edited", $mec);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->closeCursor();

        $statement->bindValue(":action", "Prefchange");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->execute();
        $pcc = $statement->fetchColumn();
        $userElement->setAttribute("prefchange", $pcc);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $statement->closeCursor();

        // Combine all three actions affecting Welcome templates into one count.
        $combinedquery = $this->getDatabase()->prepare(<<<SQL
            SELECT
                COUNT(*) AS count
            FROM log
            WHERE log.user = :userid
                AND log.action IN ('CreatedTemplate', 'EditedTemplate', 'DeletedTemplate');
SQL
        );

        $combinedquery->bindValue(":userid", $this->user->getId());
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $combinedquery->execute();
        $dtc = $combinedquery->fetchColumn();
        $userElement->setAttribute("welctempchange", $dtc);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $combinedquery->closeCursor();

        // Combine both actions affecting Email templates into one count.
        $combinedquery = $this->getDatabase()->prepare(<<<SQL
            SELECT COUNT(*) AS count
            FROM log
            WHERE log.user = :userid
                AND log.action IN ('CreatedEmail', 'EditedEmail');
SQL
        );

        $combinedquery->bindValue(":userid", $this->user->getId());
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $combinedquery->execute();
        $cec = $combinedquery->fetchColumn();
        $userElement->setAttribute("emailtempchange", $cec);
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        $combinedquery->closeCursor();
    }

}



1			<?php
2			/******************************************************************************
3			* Wikipedia Account Creation Assistance tool *
4			* *
5			* All code in this file is released into the public domain by the ACC *
6			* Development Team. Please see team.json for a list of contributors. *
7			******************************************************************************/
8
9			namespace Waca\API\Actions;
10
11			use DOMElement;
12			use Waca\API\ApiException as ApiException;
13			use Waca\API\IApiAction as IApiAction;
14			use Waca\DataObjects\User;
15			use Waca\Tasks\ApiPageBase;
16			use Waca\WebRequest;
17
18			/**
19			* API Count action
20			*/
21			class CountAction extends ApiPageBase implements IApiAction
22			{
23			/**
24			* The target user
25			* @var User $user
26			*/
27			private $user;
28
29			public function executeApiAction(DOMElement $apiDocument)
30			{
31			$username = WebRequest::getString('user');
32			if ($username === null) {
33			throw new ApiException("Please specify a username");
			0 ignored issues – show Coding Style Comprehensibility introduced 2020-04-23 23:50 UTC by Report Bug Copy Issue Report The string literal `Please specify a username` does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...
34			}
35
36			$userElement = $this->document->createElement("user");
			0 ignored issues – show Coding Style Comprehensibility introduced 2020-04-23 23:50 UTC by Report Bug Copy Issue Report The string literal `user` does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...
37			$userElement->setAttribute("name", $username);
			0 ignored issues – show Coding Style Comprehensibility introduced 2020-04-23 23:50 UTC by Report Bug Copy Issue Report The string literal `name` does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...

mdaniels5757 / waca

Push — rbac ( be68b4...52c28b )

CountAction::executeApiAction() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like