nystudio107\webperf\controllers\ChartsController::actionDashboardStatsAverage()

Complexity

Conditions	9
Paths	72

Size

Total Lines	53
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function actionDashboardStatsAverage(
        string $range = 'day',
        string $column = 'pageLoad',
        int $siteId = 0
    ): Response {
        PermissionHelper::controllerPermissionCheck('webperf:dashboard');
        $data = [];
        $days = 1;
        switch ($range) {
            case 'day':

Line indented incorrectly; expected 8 spaces, found 12

                $days = 1;
                break;
            case 'week':

Line indented incorrectly; expected 8 spaces, found 12

                $days = 7;
                break;
            case 'month':

Line indented incorrectly; expected 8 spaces, found 12

                $days = 30;
                break;
        }
        // Different dbs do it different ways
        $stats = null;
        $db = Craft::$app->getDb();
        if ($db->getIsMysql()) {
            // Query the db
            $query = (new Query())
                ->from('{{%webperf_data_samples}}')
                ->select([

The opening parenthesis of a multi-line function call should be the last content on the line.

                    'AVG('.$column.') AS avg',
                ])

For multi-line function calls, the closing parenthesis should be on a new line.

                ->where("dateUpdated >= ( CURDATE() - INTERVAL '{$days}' DAY )");
            if ((int)$siteId !== 0) {
                $query->andWhere(['siteId' => $siteId]);
            }
            $stats = $query->all();
        }
        if ($db->getIsPgsql()) {
            // Query the db
            $query = (new Query())
                ->from('{{%retour_stats}}')
                ->select([

The opening parenthesis of a multi-line function call should be the last content on the line.

                    'AVG("'.$column.'") AS avg',
                ])

For multi-line function calls, the closing parenthesis should be on a new line.

                ->where("\"dateUpdated\" >= ( CURRENT_TIMESTAMP - INTERVAL '{$days} days' )");
            if ((int)$siteId !== 0) {
                $query->andWhere(['siteId' => $siteId]);
            }
            $stats = $query->all();
        }
        if ($stats) {
            $data = ArrayHelper::getColumn($stats, 'avg');
        }

        return $this->asJson($data);
    }