AppBundle\Controller\AutomatedEditsController::resultAction()

Complexity

Conditions	13
Paths	324

Size

Total Lines	150
Code Lines	79

Duplication

Lines	13
Ratio	8.67 %

Importance

Changes

0

<?php
    public function resultAction($project, $username, $start = null, $end = null)
    {
        // Pull the labs helper and check if enabled
        $lh = $this->get("app.labs_helper");
        $lh->checkEnabled("autoedits");

        // Pull information about the project from the Labs Helper
        $dbValues = $lh->databasePrepare($project, "AutomatedEdits");

        $dbName = $dbValues["dbName"];
        $wikiName = $dbValues["wikiName"];
        $url = $dbValues["url"];

        // Grab our database connection
        $dbh = $this->get('doctrine')->getManager("replicas")->getConnection();

        // Variable parsing.
        // Username needs to be uppercase first (yay Mediawiki),
        // and we also need to handle undefined dates.
        $username = ucfirst($username);

        if ($start == null) {
            $start = date("Y-m-d", strtotime("-1 month"));
        }

        if ($end == null) {
            $end = date("Y-m-d");
        }

        // Validating the dates.  If the dates are invalid, we'll redirect
        // to the project and username view.
        if (strtotime($start) === false || strtotime($end) === false) {

This code seems to be duplicated across your project.

            // Make sure to add the flash notice first.
            $this->addFlash("notice", ["invalid_date"]);

array('invalid_date') is of type array<integer,string,{"0":"string"}>, but the function expects a string.

            // Then redirect us!
            return $this->redirectToRoute(
                "autoeditsResult",
                [
                    "project" => $project,
                    "username" => $username,
                ]
            );
        }

        // Now, load the semi-automated edit types.
        $AEBTypes = [];

$AEBTypes is not used, you could remove the assignment.

        $AEBTypes = $this->getParameter("automated_tools");

        // Create a collection of queries that we're going to run.
        $queries = [];

        $rev = $lh->getTable("revision", $dbName);
        $arc = $lh->getTable("archive", $dbName);
  
        $cond_begin = ( $start ) ? " AND rev_timestamp > :start " : null;
        $cond_end = ( $end ) ? " AND rev_timestamp < :end ": null;

        foreach ($AEBTypes as $toolname => $check) {
            $toolname = $dbh->quote($toolname, \PDO::PARAM_STR);
            $check = $dbh->quote($check, \PDO::PARAM_STR);
        
            $queries[] .= "
                SELECT $toolname as toolname, count(*) as count
                FROM $rev
                WHERE rev_user_text = :username 
                AND rev_comment REGEXP $check
                $cond_begin
                $cond_end
            ";
        }

        // Next, add two simple queries for the live and deleted edits.
        $queries[] = "
            SELECT 'live' as toolname ,count(*) as count
            from $rev
            WHERE rev_user_text = :username
            $cond_begin
            $cond_end
        ";

        $cond_begin = str_replace("rev_timestamp", "ar_timestamp", $cond_begin);
        $cond_end = str_replace("rev_timestamp", "ar_timestamp", $cond_end);

        $queries[] = "
            SELECT 'deleted' as toolname, count(*) as count
            from $arc
            WHERE ar_user_text = :username
            $cond_begin
            $cond_end
        ";

        // Create a big query and execute.
        $stmt = implode(" UNION ", $queries);

        $sth = $dbh->prepare($stmt);

        $sth->bindParam("username", $username);
        $sth->bindParam("start", $start);
        $sth->bindParam("end", $end);

        $sth->execute();

        // handling results
        $results = [];
        $total_semi = 0;
        $total = 0;

        while ($row = $sth->fetch()) {
            // Different variables need to get set if the tool is
            // the live edits or deleted edits.
            // If it is neither and greater than 0,
            // add them to the array we're rendering and to our running total
            if ($row["toolname"] == "live") {
                $total += $row["count"];
            } elseif ($row["toolname"] == "deleted") {
                $total += $row["count"];
            } elseif ($row["count"] > 0) {
                $results[$row["toolname"]] = $row["count"];
                $total_semi = $total_semi+$row["count"];
            }
        }

        // Sort the array and do some simple math.
        arsort($results);

        if ($total != 0) {
            $total_pct = ($total_semi / $total) * 100;
        } else {
            $total_pct = 0;
        }


        // Render the view with all variables set.
        return $this->render('autoEdits/result.html.twig', [
            'base_dir' => realpath($this->getParameter('kernel.root_dir').'/..'),
            //"xtPageTitle" => "autoedits",

67% of this comment could be valid code. Did you maybe forget this after debugging?

            'xtPage' => "autoedits",
            "username" => $username,
            "url" => $url,
            "wikiName" => $wikiName,
            "semi_automated" => $results,
            "start" => date("Y-m-d", strtotime($start)),
            "end" => date("Y-m-d", strtotime($end)),
            "total_semi" => $total_semi,
            "total" => $total,
            "total_pct" => $total_pct,

        ]);
    }
}