RfXVoteCalculatorController::getToolShortname() - Code Metrics - Inspection of "Make internal route names and i18n messages consis..." - x-tools/xtools - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — master ( 35c320...7deb12 )

by MusikAnimal
created 2018-04-21 13:27 UTC
RfXVoteCalculatorController::getToolShortname() A

↳ Parent: RfXVoteCalculatorController
Complexity

Conditions	1
Paths	1
Size

Total Lines	3
Code Lines	1
Duplication

Lines	0
Ratio	0 %
Code Coverage

Tests	0
CRAP Score	2
Importance

Changes
Metric	Value
cc	1
eloc	1
nc	1
nop	0
dl	0
loc	3
ccs	0
cts	0
cp	0
crap	2
rs	10
c	0
b	0
f	0
<?php
/**
 * This file contains the code that powers the RfX Vote Calculator page of XTools.
 */

namespace AppBundle\Controller;

use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\CssSelector\Exception\InternalErrorException;
use Symfony\Component\Debug\Exception\ContextErrorException;
use Symfony\Component\HttpFoundation\Request;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Xtools\ProjectRepository;
use Xtools\PageRepository;
use Xtools\RFX;
use Xtools\User;

/**
 * Controller for the RfX Vote Calculator.
 */
class RfXVoteCalculatorController extends Controller
{

    /**
     * Get the name of the tool's index route.
     * @return string
     * @codeCoverageIgnore
     */
    public function getIndexRoute()
    {
        return 'RfXVoteCalculator';
    }

    /**
     * Renders the index page for RfXVoteCalculator
     *
     * @Route("/rfxvote", name="RfXVoteCalculator")
     * @Route("/rfxvote/", name="RfXVoteCalculatorSlash")
     * @Route("/rfxvote/index.php", name="RfXVoteCalculatorIndexPhp")
     *
     * @return Response
filter:
    dependency_paths: ["lib/*"]
     */
    public function indexAction()
    {
        // Grab the request object, grab the values out of it.
        $request = Request::createFromGlobals();

        $projectQuery = $request->query->get('project');
        $username = $request->query->get('username');

        if ($projectQuery != '' && $username != '') {
            $routeParams = [ 'project' => $projectQuery, 'username' => $username ];
            return $this->redirectToRoute(
                'rfxvoteResult',
                $routeParams
            );
        } elseif ($projectQuery != '') {
            return $this->redirectToRoute(
                'rfxvoteResult',
                [
                    'project' => $projectQuery
                ]
            );
        }

        // Instantiate the project if we can, or use the default.
        $project = (!empty($projectQuery))
            ? ProjectRepository::getProject($projectQuery, $this->container)
            : ProjectRepository::getDefaultProject($this->container);

        return $this->render(
            'rfxVoteCalculator/index.html.twig',
            [
                'xtPageTitle' => 'tool-rfxvote',
                'xtSubtitle' => 'tool-rfxvote-desc',
                'xtPage' => 'rfxvote',
                'project' => $project,
            ]
        );
    }

    /**
     * Result View of RfXVoteCalculator
     * @Route("/rfxvote/{project}/{username}", name="rfxvoteResult")
     * @param string $project  The project we're working on
     * @param string $username Username of the user we're analysing.
     * @return Response
     * @codeCoverageIgnore
     */
    public function resultAction($project, $username)
    {
        $conn = $this->getDoctrine()->getManager('replicas')->getConnection();
        $conn = $this->getDoctrine()->getManager('replicas')->/** @scrutinizer ignore-call */ getConnection();

        $projectData = ProjectRepository::getProject($project, $this->container);
        $projectRepo = $projectData->getRepository();
        $userData = new User($username);
        $pageRepo = new PageRepository();
        $pageRepo->setContainer($this->container);

        $dbName = $projectData->getDatabaseName();

        $rfxParam = $this->getParameter('rfx');

        if (!$projectData->exists() || $rfxParam == null) {
            $this->addFlash('notice', ['invalid-project', $project]);
            $this->addFlash('notice', /** @scrutinizer ignore-type */ ['invalid-project', $project]);
            return $this->redirectToRoute('rfxvote');
        }

        $namespaces = $projectData->getNamespaces();

        if (!isset($rfxParam[$projectData->getDomain()])) {
            $this->addFlash('notice', ['invalid-project-cant-use', $project]);
            return $this->redirectToRoute('rfxvote');
        }

        $pageTypes = $rfxParam[$projectData->getDomain()]['pages'];
        $namespace
            = $rfxParam[$projectData->getDomain()]['rfx_namespace'] !== null
            ? $rfxParam[$projectData->getDomain()]['rfx_namespace'] : 4;

        $finalData = [];

        // We should probably figure out a better way to do this...
        $ignoredPages = '';

        if (isset($rfxParam[$projectData->getDomain()]['excluded_title'])) {
            $titlesExcluded
                = $rfxParam[$projectData->getDomain()]['excluded_title'];
            foreach ($titlesExcluded as $ignoredPage) {
                $ignoredPages .= "AND p.page_title != \"$ignoredPage\"\r\n";
            }
        }

        if (isset($rfxParam[$projectData->getDomain()]['excluded_regex'])) {
            $titlesExcluded
                = $rfxParam[$projectData->getDomain()]['excluded_regex'];
            foreach ($titlesExcluded as $ignoredPage) {
                $ignoredPages .= "AND p.page_title NOT LIKE \"%$ignoredPage%\"\r\n";
            }
        }

        /**
         * Contains the total number of !votes the user made, keyed by the RfX
         * type and then the vote type.
         * @var array
         */
        $totals = [];

        foreach ($pageTypes as $type) {
            $type = explode(':', $type, 2)[1];

            $type = str_replace(' ', '_', $type);

            $pageTable = $projectRepo->getTableName($dbName, 'page');
            $revisionTable
                = $projectRepo->getTableName($dbName, 'revision');

            $sql = "SELECT DISTINCT p.page_namespace, p.page_title
                    FROM $pageTable p
                    RIGHT JOIN $revisionTable r on p.page_id=r.rev_page
                    WHERE p.page_namespace = :namespace
                    AND r.rev_user_text = :username
                    And p.page_title LIKE \"$type/%\"
                    AND p.page_title NOT LIKE \"%$type/$username%\"
                    $ignoredPages";

            $sth = $conn->prepare($sql);
            $sth->bindParam('namespace', $namespace);
            $sth->bindParam('username', $username);

            $sth->execute();

            $titles = [];

            while ($row = $sth->fetch()) {
                $titles[] = $namespaces[$row['page_namespace']] .
                    ':' .$row['page_title'];
            }

            // Chunking... it's possible to make a URI too long
            $titleArray = array_chunk($titles, 20);

            foreach ($titleArray as $titlesWorked) {
                $pageData = $pageRepo->getPagesWikitext($projectData, $titlesWorked);

                foreach ($pageData as $title => $text) {
                    $type = str_replace('_', ' ', $type);
                    $rfx = new RFX(
                        $text,
                        $rfxParam[$projectData->getDomain()]['sections'],
                        $namespaces[2],
                        $rfxParam[$projectData->getDomain()]['date_regexp'],
                        $username
                    );
                    $section = $rfx->getUserSectionFound();

                    if ($section == '') {
                        // Skip over ones where the user didn't !vote.
                        continue;
                    }

                    if (!isset($totals[$type])) {
                        $totals[$type] = [];
                    }
                    if (!isset($totals[$type][$section])) {
                        $totals[$type][$section] = 0;
                    }
                    if (!isset($totals[$type]['total'])) {
                        $totals[$type]['total'] = 0;
                    }
                    $totals[$type][$section] += 1;
                    $totals[$type]['total'] += 1;

                    // Todo: i18n-ize this
                    $finalData[$type][$section][$title]['Support']
                        = sizeof($rfx->getSection('support'));
                    $finalData[$type][$section][$title]['Oppose']
                        = sizeof($rfx->getSection('oppose'));
                    $finalData[$type][$section][$title]['Neutral']
                        = sizeof($rfx->getSection('neutral'));
                    $finalData[$type][$section][$title]['Date']
                        = $rfx->getEndDate();
                    $finalData[$type][$section][$title]['name']
                        = explode('/', $title)[1];

                    unset($rfx);
                }
            }
        }

        return $this->render(
            'rfxVoteCalculator/result.html.twig',
            [
                'xtPage' => 'rfxvote',
                'xtTitle' => $username,
                'user' => $userData,
                'project' => $projectData,
                'data'=> $finalData,
                'totals' => $totals,
            ]
        );
    }
}

x-tools / xtools

Push — master ( 35c320...7deb12 )

RfXVoteCalculatorController::getToolShortname() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like
