TablesController::actionErrorsDetail() - Code Metrics - Inspection of "Fix postres queries" - nystudio107/craft-webperf - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — v1 ( ce78b2...2675f6 )

by Andrew

created 2019-02-05 04:33 UTC

TablesController::actionErrorsDetail() C

↳ Parent: TablesController

Complexity

Conditions	13
Paths	60

Size

Total Lines	124
Code Lines	74

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	74
dl	0
loc	124
rs	5.8606
c	0
b	0
f	0
cc	13
nc	60
nop	8

How to fix Long Method Complexity Many Parameters

<?php
/**
 * Webperf plugin for Craft CMS 3.x
 *
 * Monitor the performance of your webpages through real-world user timing data
 *
 * @link      https://nystudio107.com

 * @copyright Copyright (c) 2019 nystudio107

 */


namespace nystudio107\webperf\controllers;

use nystudio107\webperf\helpers\Permission as PermissionHelper;

use Craft;
filter:
    dependency_paths: ["lib/*"]
use craft\db\Query;
filter:
    dependency_paths: ["lib/*"]
use craft\helpers\DateTimeHelper;
filter:
    dependency_paths: ["lib/*"]
use craft\helpers\UrlHelper;
filter:
    dependency_paths: ["lib/*"]
use craft\web\Controller;
filter:
    dependency_paths: ["lib/*"]

use yii\web\ForbiddenHttpException;
filter:
    dependency_paths: ["lib/*"]
use yii\web\Response;
filter:
    dependency_paths: ["lib/*"]

/**

 * @author    nystudio107

 * @package   Webperf

 * @since     1.0.0

 */

class TablesController extends Controller
{
    // Constants
    // =========================================================================

    // Protected Properties
    // =========================================================================

    /**

     * @var    bool|array

     */
    protected $allowAnonymous = [];

    // Public Methods
    // =========================================================================

    /**
     * Handle requests for the performance index table
     *
     * @param string $sort

     * @param int    $page

     * @param int    $per_page

     * @param string $filter

     * @param string $start

     * @param string $end

     * @param int    $siteId

     *
     * @return Response
     * @throws ForbiddenHttpException
     */
    public function actionPagesIndex(
        string $start = '',
        string $end = '',
        string $sort = 'pageLoad|DESC',
        int $page = 1,
        int $per_page = 20,
        $filter = '',
        $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'));
        // Figure out the sorting type
        if ($sort !== '') {
            if (strpos($sort, '|') === false) {
                $sortField = $sort;
            } else {
                list($sortField, $sortType) = explode('|', $sort);
            }
        }
        // Query the db table
        $offset = ($page - 1) * $per_page;
        $query = (new Query())
            ->select([

                '[[url]]',
                'MIN([[title]]) AS [[title]]',
                'COUNT([[url]]) AS [[cnt]]',
                'AVG([[pageLoad]]) AS [[pageLoad]]',
                'AVG([[domInteractive]]) AS [[domInteractive]]',
                'AVG([[firstContentfulPaint]]) AS [[firstContentfulPaint]]',
                'AVG([[firstPaint]]) AS [[firstPaint]]',
                'AVG([[firstByte]]) AS [[firstByte]]',
                'AVG([[connect]]) AS [[connect]]',
                'AVG([[dns]]) AS [[dns]]',
                'AVG([[craftTotalMs]]) AS [[craftTotalMs]]',
                'AVG([[craftDbCnt]]) AS [[craftDbCnt]]',
                'AVG([[craftDbMs]]) AS [[craftDbMs]]',
                'AVG([[craftTwigCnt]]) AS [[craftTwigCnt]]',
                'AVG([[craftTwigMs]]) AS [[craftTwigMs]]',
                'AVG([[craftTotalMemory]]) AS [[craftTotalMemory]]',
            ])
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
            ->from(['{{%webperf_data_samples}}'])
            ->offset($offset)
            ->where(['between', 'dateCreated', $start, $end])

        ;
        if ((int)$siteId !== 0) {
            $query->andWhere(['siteId' => $siteId]);
        }
        if ($filter !== '') {
            $query
                ->andWhere(['like', 'url', $filter])
                ->orWhere(['like', 'title', $filter])

            ;
        }
        $query
            ->orderBy("[[{$sortField}]] {$sortType}")
            ->groupBy('url')
            ->limit($per_page)

        ;

        $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 = $stat['pageLoad'];
                }
            }
            // Massage the stats
            $index = 1;
            foreach ($stats as &$stat) {
                $stat['id'] = $index++;
                $stat['cnt'] = (int)$stat['cnt'];
                $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');
                }
                // Set up the appropriate helper links
                $stat['deleteLink'] = UrlHelper::actionUrl('webperf/data-samples/delete-samples-by-url', [

                    'pageUrl' => $stat['url'],
                    'siteId' => $siteId
                ]);
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
                $stat['detailPageUrl'] = UrlHelper::cpUrl('webperf/performance/page-detail', [

                    'pageUrl' => $stat['url'],
                    'siteId' => $siteId,
                ]);
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
                // Override based on permissions
                if (!$user->can('webperf:delete-data-samples')) {
                    $stat['deleteLink'] = '';
                }
                if (!$user->can('webperf:performance-detail')) {
                    $stat['detailPageUrl'] = '';
                }
            }
            // Format the data for the API
            $data['data'] = $stats;
            $query = (new Query())
                ->select(['[[url]]'])
                ->from(['{{%webperf_data_samples}}'])
                ->groupBy('[[url]]')
                ->where(['between', 'dateCreated', $start, $end])

                ;
            if ($filter !== '') {
                $query->andWhere(['like', 'url', $filter]);
                $query->orWhere(['like', 'title', $filter]);
            }
            $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);
    }

    /**
     * Handle requests for the performance detail table
     *
     * @param string $sort

     * @param int    $page

     * @param int    $per_page

     * @param string $filter

     * @param string $pageUrl

     * @param string $start

     * @param string $end

     * @param int    $siteId

     *
     * @return Response
     * @throws ForbiddenHttpException
     */
    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);
            }
        }
        // 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])

        ;
        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])

            ;
        }
        $query
            ->orderBy("{$sortField} {$sortType}")
            ->limit($per_page)

        ;
        $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', [

                    'id' => $stat['id']
                ]);
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis 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])

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

                ;
            }
            $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);
    }

    /**
     * Handle requests for the pages index table
     *
     * @param string $sort

     * @param int    $page

     * @param int    $per_page

     * @param string $filter

     * @param string $start

     * @param string $end

     * @param int    $siteId

     *
     * @return Response
     * @throws ForbiddenHttpException
     */
    public function actionErrorsIndex(
        string $start = '',
        string $end = '',
        string $sort = 'url|DESC',
        int $page = 1,
        int $per_page = 20,
        $filter = '',
        $siteId = 0
    ): Response {
        PermissionHelper::controllerPermissionCheck('webperf:errors');
        $data = [];
        $sortField = 'url';
        $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'));
        // Figure out the sorting type
        if ($sort !== '') {
            if (strpos($sort, '|') === false) {
                $sortField = $sort;
            } else {
                list($sortField, $sortType) = explode('|', $sort);
            }
        }
        $db = Craft::$app->getDb();
        // Query the db table
        $offset = ($page - 1) * $per_page;
        $query = (new Query())
            ->select([

                '[[url]]',
                'MIN([[title]]) as [[title]]',
                'MAX([[dateCreated]]) as [[latestErrorDate]]',
                'SUM([[type]] = \'craft\') as [[craftCount]]',
                'SUM([[type]] = \'boomerang\') as [[boomerangCount]]',
                'COUNT([[url]]) AS cnt',
            ])
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
            ->from(['{{%webperf_error_samples}}'])
            ->offset($offset)
            ->where(['between', 'dateCreated', $start, $end])

        ;
        if ($db->getIsMysql()) {
            $query
                ->addSelect([

                    'SUM([[type]] = \'craft\') as [[craftCount]]',
                    'SUM([[type]] = \'boomerang\') as [[boomerangCount]]',
                ]);
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
        }
        if ($db->getIsPgsql()) {
            $query
                ->addSelect([

                    'SUM(case when [[type]] = \'craft\' then 1 else 0 end) as [[craftCount]]',
                    'SUM(case when [[type]] = \'boomerang\' then 1 else 0 end) as [[boomerangCount]]',
                ]);
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
        }
        if ((int)$siteId !== 0) {
            $query->andWhere(['siteId' => $siteId]);
        }
        if ($filter !== '') {
            $query
                ->andWhere(['like', 'url', $filter])
                ->orWhere(['like', 'title', $filter])
                ->orWhere(['like', 'pageErrors', $filter])

            ;
        }
        $query
            ->orderBy("[[{$sortField}]] {$sortType}")
            ->groupBy('url')
            ->limit($per_page)

        ;

        $stats = $query->all();
        if ($stats) {
            $user = Craft::$app->getUser()->getIdentity();
            // Massage the stats
            foreach ($stats as &$stat) {
                $stat['cnt'] = (int)$stat['cnt'];
                $stat['craftCount'] = (int)$stat['craftCount'];
                $stat['boomerangCount'] = (int)$stat['boomerangCount'];
                // Decode any emojis in the title
                if (!empty($stat['title'])) {
                    $stat['title'] = html_entity_decode($stat['title'], ENT_NOQUOTES, 'UTF-8');
                }
                // Set up the appropriate helper links
                $stat['deleteLink'] = UrlHelper::actionUrl('webperf/error-samples/delete-samples-by-url', [

                    'pageUrl' => $stat['url'],
                    'siteId' => $siteId
                ]);
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
                $stat['detailPageUrl'] = UrlHelper::cpUrl('webperf/errors/page-detail', [

                    'pageUrl' => $stat['url'],
                    'siteId' => $siteId,
                ]);
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
                // Override based on permissions
                if (!$user->can('webperf:delete-error-samples')) {
                    $stat['deleteLink'] = '';
                }
                if (!$user->can('webperf:errors-detail')) {
                    $stat['detailPageUrl'] = '';
                }
            }
            // Format the data for the API
            $data['data'] = $stats;
            $query = (new Query())
                ->select(['[[url]]'])
                ->from(['{{%webperf_error_samples}}'])
                ->groupBy('[[url]]')
                ->where(['between', 'dateCreated', $start, $end])

            ;
            if ($filter !== '') {
                $query
                    ->andWhere(['like', 'url', $filter])
                    ->orWhere(['like', 'title', $filter])
                    ->orWhere(['like', 'pageErrors', $filter])

                ;
            }
            $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);
    }


    /**
     * Handle requests for the performance detail table
     *
     * @param string $sort

     * @param int    $page

     * @param int    $per_page

     * @param string $filter

     * @param string $pageUrl

     * @param string $start

     * @param string $end

     * @param int    $siteId

     *
     * @return Response
     * @throws ForbiddenHttpException
     */
    public function actionErrorsDetail(
        string $start = '',
        string $end = '',
        string $sort = 'dateCreated|DESC',
        int $page = 1,
        int $per_page = 20,
        $filter = '',
        $pageUrl = '',
        $siteId = 0
    ): Response {
        PermissionHelper::controllerPermissionCheck('webperf:errors');
        $data = [];
        $sortField = 'dateCreated';
        $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);
            }
        }
        // Query the db table
        $offset = ($page - 1) * $per_page;
        $query = (new Query())
            ->from(['{{%webperf_error_samples}} webperf_error_samples'])
            ->select([

                '[[webperf_error_samples.url]]',
                '[[webperf_error_samples.id]]',
                '[[webperf_error_samples.type]]',
                '[[webperf_error_samples.dateCreated]]',
                '[[webperf_error_samples.pageErrors]]',
                '[[webperf_error_samples.id]]',

                '[[webperf_data_samples.device]]',
                '[[webperf_data_samples.os]]',
                '[[webperf_data_samples.browser]]',
                '[[webperf_data_samples.countryCode]]',
                '[[webperf_data_samples.mobile]]',
            ])
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
            ->offset($offset)
            ->where(['[[webperf_error_samples.url]]' => $pageUrl])
            ->andWhere(['between', '[[webperf_error_samples.dateCreated]]', $start, $end])
            ->leftJoin('{{%webperf_data_samples}} webperf_data_samples', '[[webperf_data_samples.requestId]] = [[webperf_error_samples.requestId]]')

        ;
        if ((int)$siteId !== 0) {
            $query->andWhere(['siteId' => $siteId]);
        }
        if ($filter !== '') {
            $query
                ->andWhere(['like', 'pageErrors', $filter])
                /*
                ->orWhere(['like', 'device', $filter])
                ->orWhere(['like', 'os', $filter])
                ->orWhere(['like', 'browser', $filter])
                ->orWhere(['like', 'countryCode', $filter])
                */
            ;
        }
        $query
            ->orderBy("{$sortField} {$sortType}")
            ->limit($per_page)

        ;
        $stats = $query->all();
        if ($stats) {
            $user = Craft::$app->getUser()->getIdentity();
            // 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');
                }
                if (isset($stat['mobile'])) {
                    $stat['mobile'] = (bool)$stat['mobile'];
                }
                // 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/error-samples/delete-sample-by-id', [

                    'id' => $stat['id']
                ]);
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
                // Override based on permissions
                if (!$user->can('webperf:delete-error-samples')) {
                    $stat['deleteLink'] = '';
                }
            }
            // Format the data for the API
            $data['data'] = $stats;
            $query = (new Query())
                ->select(['[[url]]'])
                ->from(['{{%webperf_error_samples}}'])
                ->where(['url' => $pageUrl])
                ->andWhere(['between', 'dateCreated', $start, $end])

            ;
            if ($filter !== '') {
                $query
                    ->andWhere(['like', 'pageErrors', $filter])
                    /*
                    ->orWhere(['like', 'device', $filter])
                    ->orWhere(['like', 'os', $filter])
                    ->orWhere(['like', 'browser', $filter])
                    ->orWhere(['like', 'countryCode', $filter])
                    */
                ;
            }
            $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),
            ];
        }

        Craft::error('crap: '.print_r($data, true), __METHOD__);
        return $this->asJson($data);
    }

    // Protected Methods
    // =========================================================================
}


1			<?php
2			/**
3			* Webperf plugin for Craft CMS 3.x
4			*
5			* Monitor the performance of your webpages through real-world user timing data
6			*
7			* @link https://nystudio107.com
			0 ignored issues – show Coding Style introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report The tag in position 1 should be the @copyright tag Loading history...
8			* @copyright Copyright (c) 2019 nystudio107
			0 ignored issues – show Coding Style introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report @copyright tag must contain a year and the name of the copyright holder Loading history...
9			*/
			0 ignored issues – show Coding Style introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report PHP version not specified Loading history... Coding Style introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report Missing @category tag in file comment Loading history... Coding Style introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report Missing @package tag in file comment Loading history... Coding Style introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report Missing @author tag in file comment Loading history... Coding Style introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report Missing @license tag in file comment Loading history...
10
11			namespace nystudio107\webperf\controllers;
12
13			use nystudio107\webperf\helpers\Permission as PermissionHelper;
14
15			use Craft;
			0 ignored issues – show Bug introduced 2019-01-20 07:54 UTC by Report Bug Copy Issue Report The type `Craft` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
16			use craft\db\Query;
			0 ignored issues – show Bug introduced 2019-01-20 07:54 UTC by Report Bug Copy Issue Report The type `craft\db\Query` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
17			use craft\helpers\DateTimeHelper;
			0 ignored issues – show Bug introduced 2019-01-28 04:39 UTC by Report Bug Copy Issue Report The type `craft\helpers\DateTimeHelper` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
18			use craft\helpers\UrlHelper;
			0 ignored issues – show Bug introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report The type `craft\helpers\UrlHelper` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
19			use craft\web\Controller;
			0 ignored issues – show Bug introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report The type `craft\web\Controller` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
20
21			use yii\web\ForbiddenHttpException;
			0 ignored issues – show Bug introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report The type `yii\web\ForbiddenHttpException` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
22			use yii\web\Response;
			0 ignored issues – show Bug introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report The type `yii\web\Response` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
23
24			/**
			0 ignored issues – show Coding Style introduced 2019-01-02 04:57 UTC by Report Bug Copy Issue Report Missing short description in doc comment Loading history...
25			* @author nystudio107
			0 ignored issues – show Coding Style introduced 2019-01-20 07:54 UTC by Report Bug Copy Issue Report The tag in position 1 should be the @package tag Loading history... Coding Style introduced 2019-01-20 07:54 UTC by Report Bug Copy Issue Report Content of the @author tag must be in the form "Display Name <[email protected]>" Loading history... Coding Style introduced 2019-01-20 07:54 UTC by Report Bug Copy Issue Report Tag value indented incorrectly; expected 2 spaces but found 4 Loading history...
26			* @package Webperf
			0 ignored issues – show Coding Style introduced 2019-01-20 07:54 UTC by Report Bug Copy Issue Report Tag value indented incorrectly; expected 1 spaces but found 3 Loading history...
27			* @since 1.0.0

nystudio107 / craft-webperf

Push — v1 ( ce78b2...2675f6 )

TablesController::actionErrorsDetail() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity Many Parameters

Long Method

Many Parameters

Duplication Side-by-Side

Filter issues like