nystudio107\webperf\controllers\TablesController::actionPageDetail()

Complexity

Conditions	19
Paths	63

Size

Total Lines	124
Code Lines	80

Duplication

Lines	0
Ratio	0 %

Importance

Changes	12
Bugs	0	Features	0

<?php
    public function actionPageDetail(
        string $start = '',
        string $end = '',
        string $sort = 'pageLoad|DESC',
        int $page = 1,
        int $per_page = 20,
        $filter = '',
        $pageUrl = '',
        $siteId = 0
    ): Response {
        PermissionHelper::controllerPermissionCheck('webperf:performance');
        $data = [];
        $sortField = 'pageLoad';
        $sortType = 'DESC';
        // Add a day since YYYY-MM-DD is really YYYY-MM-DD 00:00:00
        $end = date('Y-m-d', strtotime($end.'+1 day'));
        $pageUrl = urldecode($pageUrl);
        // Figure out the sorting type
        if ($sort !== '') {
            if (strpos($sort, '|') === false) {
                $sortField = $sort;
            } else {
                list($sortField, $sortType) = explode('|', $sort);
            }
        }
        $sortType = strtoupper($sortType);
        $sortType = self::SORT_MAP[$sortType] ?? self::SORT_MAP['DESC'];
        // Validate untrusted data
        if (!in_array($sortField, self::ALLOWED_PAGE_DETAIL_SORT_FIELDS, true)) {
            throw new BadRequestHttpException(Craft::t('webperf', 'Invalid sort field specified.'));
        }
        // Query the db table
        $offset = ($page - 1) * $per_page;
        $query = (new Query())
            ->from(['{{%webperf_data_samples}}'])
            ->offset($offset)
            ->where(['url' => $pageUrl])
            ->andWhere(['between', 'dateCreated', $start, $end])

Space after closing parenthesis of function call prohibited

        ;
        if ((int)$siteId !== 0) {
            $query->andWhere(['siteId' => $siteId]);
        }
        if ($filter !== '') {
            $query
                ->andWhere(['like', 'device', $filter])
                ->orWhere(['like', 'os', $filter])
                ->orWhere(['like', 'browser', $filter])
                ->orWhere(['like', 'countryCode', $filter])

Space after closing parenthesis of function call prohibited

            ;
        }
        $query
            ->orderBy([$sortField => $sortType])
            ->limit($per_page)

Space after closing parenthesis of function call prohibited

        ;
        $stats = $query->all();
        if ($stats) {
            $user = Craft::$app->getUser()->getIdentity();
            // Compute the largest page load time
            $maxTotalPageLoad = 0;
            foreach ($stats as &$stat) {
                // Determine the stat type
                if (!empty($stat['pageLoad']) && !empty($stat['craftTotalMs'])) {
                    $stat['type'] = 'both';
                }
                if (empty($stat['firstByte'])) {
                    $stat['type'] = 'craft';
                }
                if (empty($stat['craftTotalMs'])) {
                    $stat['type'] = 'frontend';
                }
                if ($stat['pageLoad'] > $maxTotalPageLoad) {
                    $maxTotalPageLoad = (int)$stat['pageLoad'];
                }
            }
            // Massage the stats
            foreach ($stats as &$stat) {
                if (!empty($stats['dateCreated'])) {
                    $date = DateTimeHelper::toDateTime($stats['dateCreated']);
                    $stats['dateCreated'] = $date->format('Y-m-d H:i:s');
                }
                $stat['mobile'] = (bool)$stat['mobile'];
                $stat['maxTotalPageLoad'] = (int)$maxTotalPageLoad;
                // Decode any emojis in the title
                if (!empty($stat['title'])) {
                    $stat['title'] = html_entity_decode($stat['title'], ENT_NOQUOTES, 'UTF-8');
                }
                $stat['deleteLink'] = UrlHelper::actionUrl('webperf/data-samples/delete-sample-by-id', [

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

                    'id' => $stat['id']
                ]);

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

                // Override based on permissions
                if (!$user->can('webperf:delete-data-samples')) {
                    $stat['deleteLink'] = '';
                }
            }
            // Format the data for the API
            $data['data'] = $stats;
            $query = (new Query())
                ->select(['[[url]]'])
                ->from(['{{%webperf_data_samples}}'])
                ->where(['url' => $pageUrl])
                ->andWhere(['between', 'dateCreated', $start, $end])

Space after closing parenthesis of function call prohibited

            ;
            if ($filter !== '') {
                $query
                    ->andWhere(['like', 'device', $filter])
                    ->orWhere(['like', 'os', $filter])
                    ->orWhere(['like', 'browser', $filter])
                    ->orWhere(['like', 'countryCode', $filter])

Space after closing parenthesis of function call prohibited

                ;
            }
            $count = $query->count();
            $data['links']['pagination'] = [
                'total' => $count,
                'per_page' => $per_page,
                'current_page' => $page,
                'last_page' => ceil($count / $per_page),
                'next_page_url' => null,
                'prev_page_url' => null,
                'from' => $offset + 1,
                'to' => $offset + ($count > $per_page ? $per_page : $count),
            ];
        }

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