Recommendations::data() - Code Metrics - Inspection of "1.0.0-beta.30" - nystudio107/craft-webperf - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — v1 ( 3980d8...7dfb05 )

by Andrew

created 2019-01-29 16:16 UTC

Recommendations::data() B

↳ Parent: Recommendations

Complexity

Conditions	8
Paths	50

Size

Total Lines	82
Code Lines	59

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	59
dl	0
loc	82
rs	7.6501
c	0
b	0
f	0
cc	8
nc	50
nop	4

How to fix Long Method

<?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\services;

use nystudio107\webperf\base\Recommendation;
use nystudio107\webperf\models\RecommendationDataSample;
use nystudio107\webperf\recommendations\CraftQueryCount;
use nystudio107\webperf\recommendations\CraftQueryTime;
use nystudio107\webperf\recommendations\CraftTotalTime;
use nystudio107\webperf\recommendations\CraftTwigTime;
use nystudio107\webperf\recommendations\DomInteractive;
use nystudio107\webperf\recommendations\FirstByte;
use nystudio107\webperf\recommendations\FirstContentfulPaint;
use nystudio107\webperf\recommendations\MemoryLimit;

use Craft;
filter:
    dependency_paths: ["lib/*"]
use craft\base\Component;
filter:
    dependency_paths: ["lib/*"]
use craft\db\Query;
filter:
    dependency_paths: ["lib/*"]

use yii\helpers\Markdown;
filter:
    dependency_paths: ["lib/*"]

/**

 * @author    nystudio107

 * @package   Webperf

 * @since     1.0.0

 */

class Recommendations extends Component
{
    // Constants
    // =========================================================================

    const RECOMMENDATIONS_LIST = [
        CraftQueryCount::class,
        CraftQueryTime::class,
        CraftTwigTime::class,
        CraftTotalTime::class,
        FirstByte::class,
        FirstContentfulPaint::class,
        DomInteractive::class,
        MemoryLimit::class,
    ];

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

    /**
     * Return a list of recommendations
     *
     * @param RecommendationDataSample $sample

     *
     * @return array
     */
    public function list(RecommendationDataSample $sample): array
    {
        $data = [];
        foreach (self::RECOMMENDATIONS_LIST as $recClass) {
            /** @var Recommendation $rec */

            $rec = new $recClass(['sample' => $sample]);
            if ($rec->hasRecommendation) {
                $data[] = [
                    'summary' => Markdown::processParagraph($rec->summary),
                    'detail' => Markdown::processParagraph($rec->detail),
                    'learnMoreUrl' => $rec->learnMoreUrl,
                ];
            }
        }
        return $data;
    }

    /**
     * Return a list of recommendations
     *
     * @param string $pageUrl

     * @param string $start

     * @param string $end

     * @param int    $siteId

     *
     * @return array
     */
    public function data(
        $pageUrl = '',
        string $start = '',
        string $end = '',
        $siteId = 0
    ): array {
        $data = [];
        $db = Craft::$app->getDb();
        // 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);
        if ($db->getIsMysql()) {
            // Query the db table
            $query = (new Query())
                ->select([

                    '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(craftDbMs) AS craftDbMs',
                    'AVG(craftDbCnt) AS craftDbCnt',
                    'AVG(craftTwigMs) AS craftTwigMs',
                    'AVG(craftTwigCnt) AS craftTwigCnt',
                    'AVG(craftOtherMs) AS craftOtherMs',
                    'AVG(craftOtherCnt) AS craftOtherCnt',
                    'AVG(craftTotalMemory) AS craftTotalMemory',
                ])
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
                ->from(['{{%webperf_data_samples}}'])
                ->where(['between', 'dateCreated', $start, $end]);
            if (!empty($pageUrl)) {
                $query->andWhere(['url' => $pageUrl]);
            }
            if ((int)$siteId !== 0) {
                $query->andWhere(['siteId' => $siteId]);
            }
            $stats = $query->all();
        }
        if ($db->getIsPgsql()) {
            // Query the db table
            $query = (new Query())
                ->select([

                    '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("craftDbMs") AS craftDbMs',
                    'AVG("craftDbCnt") AS craftDbCnt',
                    'AVG("craftTwigMs") AS craftTwigMs',
                    'AVG("craftTwigCnt") AS craftTwigCnt',
                    'AVG("craftOtherMs") AS craftOtherMs',
                    'AVG("craftOtherCnt") AS craftOtherCnt',
                    'AVG("craftTotalMemory") AS craftTotalMemory',
                ])
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.
                ->from(['{{%webperf_data_samples}}'])
                ->where(['between', 'dateCreated', $start, $end]);
            if (!empty($pageUrl)) {
                $query->andWhere(['url' => $pageUrl]);
            }
            if ((int)$siteId !== 0) {
                $query->andWhere(['siteId' => $siteId]);
            }
            $stats = $query->all();
        }
        if ($stats) {

            $data = $stats[0];
        }

        return $data;
    }
}


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-27 17:08 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-27 17:08 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-27 17:08 UTC by Report Bug Copy Issue Report PHP version not specified Loading history... Coding Style introduced 2019-01-27 17:08 UTC by Report Bug Copy Issue Report Missing @category tag in file comment Loading history... Coding Style introduced 2019-01-27 17:08 UTC by Report Bug Copy Issue Report Missing @package tag in file comment Loading history... Coding Style introduced 2019-01-27 17:08 UTC by Report Bug Copy Issue Report Missing @author tag in file comment Loading history... Coding Style introduced 2019-01-27 17:08 UTC by Report Bug Copy Issue Report Missing @license tag in file comment Loading history...
10
11			namespace nystudio107\webperf\services;
12
13			use nystudio107\webperf\base\Recommendation;
14			use nystudio107\webperf\models\RecommendationDataSample;
15			use nystudio107\webperf\recommendations\CraftQueryCount;
16			use nystudio107\webperf\recommendations\CraftQueryTime;
17			use nystudio107\webperf\recommendations\CraftTotalTime;
18			use nystudio107\webperf\recommendations\CraftTwigTime;
19			use nystudio107\webperf\recommendations\DomInteractive;
20			use nystudio107\webperf\recommendations\FirstByte;
21			use nystudio107\webperf\recommendations\FirstContentfulPaint;
22			use nystudio107\webperf\recommendations\MemoryLimit;
23
24			use Craft;
			0 ignored issues – show Bug introduced 2019-01-08 04:06 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...
25			use craft\base\Component;
			0 ignored issues – show Bug introduced 2019-01-28 04:39 UTC by Report Bug Copy Issue Report The type `craft\base\Component` 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...
26			use craft\db\Query;
			0 ignored issues – show Bug introduced 2019-01-28 06:28 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...
27
28			use yii\helpers\Markdown;
			0 ignored issues – show Bug introduced 2019-01-28 06:28 UTC by Report Bug Copy Issue Report The type `yii\helpers\Markdown` 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...
29
30			/**
			0 ignored issues – show Coding Style introduced 2019-01-28 06:28 UTC by Report Bug Copy Issue Report Missing short description in doc comment Loading history...
31			* @author nystudio107
			0 ignored issues – show Coding Style introduced 2019-01-28 23:43 UTC by Report Bug Copy Issue Report The tag in position 1 should be the @package tag Loading history... Coding Style introduced 2019-01-28 23:43 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-28 23:43 UTC by Report Bug Copy Issue Report Tag value indented incorrectly; expected 2 spaces but found 4 Loading history...
32			* @package Webperf
			0 ignored issues – show Coding Style introduced 2019-01-28 23:43 UTC by Report Bug Copy Issue Report Tag value indented incorrectly; expected 1 spaces but found 3 Loading history...
33			* @since 1.0.0
			0 ignored issues – show Coding Style introduced 2019-01-28 23:43 UTC by Report Bug Copy Issue Report The tag in position 3 should be the @author tag Loading history... Coding Style introduced 2019-01-28 23:43 UTC by Report Bug Copy Issue Report Tag value indented incorrectly; expected 3 spaces but found 5 Loading history...
34			*/
			0 ignored issues – show Coding Style introduced 2019-01-27 17:08 UTC by Report Bug Copy Issue Report Missing @category tag in class comment Loading history... Coding Style introduced 2019-01-27 17:08 UTC by Report Bug Copy Issue Report Missing @license tag in class comment Loading history... Coding Style introduced 2019-01-27 17:08 UTC by Report Bug Copy Issue Report Missing @link tag in class comment Loading history...
35			class Recommendations extends Component
36			{
37			// Constants
38			// =========================================================================
39
40			const RECOMMENDATIONS_LIST = [
41			CraftQueryCount::class,
42			CraftQueryTime::class,
43			CraftTwigTime::class,
44			CraftTotalTime::class,
45			FirstByte::class,
46			FirstContentfulPaint::class,
47			DomInteractive::class,
48			MemoryLimit::class,
49			];
50
51			// Public Methods
52			// =========================================================================
53
54			/**
55			* Return a list of recommendations
56			*
57			* @param RecommendationDataSample $sample
			0 ignored issues – show Coding Style introduced 2019-01-28 04:39 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
58			*
59			* @return array
60			*/
61			public function list(RecommendationDataSample $sample): array
62			{
63			$data = [];
64			foreach (self::RECOMMENDATIONS_LIST as $recClass) {
65			/** @var Recommendation $rec */
			0 ignored issues – show Coding Style introduced 2019-01-28 04:39 UTC by Report Bug Copy Issue Report The open comment tag must be the only content on the line Loading history... Coding Style introduced 2019-01-28 04:39 UTC by Report Bug Copy Issue Report Missing short description in doc comment Loading history... Coding Style introduced 2019-01-28 04:39 UTC by Report Bug Copy Issue Report The close comment tag must be the only content on the line Loading history...
66			$rec = new $recClass(['sample' => $sample]);
67			if ($rec->hasRecommendation) {
68			$data[] = [
69			'summary' => Markdown::processParagraph($rec->summary),
70			'detail' => Markdown::processParagraph($rec->detail),
71			'learnMoreUrl' => $rec->learnMoreUrl,
72			];
73			}
74			}
75			return $data;
76			}
77
78			/**
79			* Return a list of recommendations
80			*
81			* @param string $pageUrl
			0 ignored issues – show Coding Style introduced 2019-01-28 04:39 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
82			* @param string $start
			0 ignored issues – show Coding Style introduced 2019-01-23 06:22 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
83			* @param string $end
			0 ignored issues – show Coding Style introduced 2019-01-23 06:22 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
84			* @param int $siteId
			0 ignored issues – show Coding Style introduced 2019-01-22 06:21 UTC by Report Bug Copy Issue Report Missing parameter comment Loading history...
85			*
86			* @return array
87			*/
88			public function data(
89			$pageUrl = '',
90			string $start = '',
91			string $end = '',
92			$siteId = 0
93			): array {
94			$data = [];
95			$db = Craft::$app->getDb();
96			// Add a day since YYYY-MM-DD is really YYYY-MM-DD 00:00:00
97			$end = date('Y-m-d', strtotime($end.'+1 day'));
98			$pageUrl = urldecode($pageUrl);
99			if ($db->getIsMysql()) {
100			// Query the db table
101			$query = (new Query())
102			->select([
			0 ignored issues – show Coding Style introduced 2019-01-19 05:19 UTC by Report Bug Copy Issue Report The opening parenthesis of a multi-line function call should be the last content on the line. Loading history...
103			'COUNT(url) AS cnt',
104
105			'AVG(pageLoad) AS pageLoad',
106			'AVG(domInteractive) AS domInteractive',
107			'AVG(firstContentfulPaint) AS firstContentfulPaint',
108			'AVG(firstPaint) AS firstPaint',
109			'AVG(firstByte) AS firstByte',
110			'AVG(connect) AS connect',
111			'AVG(dns) AS dns',
112
113			'AVG(craftTotalMs) AS craftTotalMs',
114			'AVG(craftDbMs) AS craftDbMs',
115			'AVG(craftDbCnt) AS craftDbCnt',
116			'AVG(craftTwigMs) AS craftTwigMs',
117			'AVG(craftTwigCnt) AS craftTwigCnt',
118			'AVG(craftOtherMs) AS craftOtherMs',
119			'AVG(craftOtherCnt) AS craftOtherCnt',
120			'AVG(craftTotalMemory) AS craftTotalMemory',
121			])
			0 ignored issues – show Coding Style introduced 2019-01-19 05:19 UTC by Report Bug Copy Issue Report For multi-line function calls, the closing parenthesis should be on a new line. If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); // Closing parenthesis on a new line. Loading history...
122			->from(['{{%webperf_data_samples}}'])
123			->where(['between', 'dateCreated', $start, $end]);
124			if (!empty($pageUrl)) {
125			$query->andWhere(['url' => $pageUrl]);
126			}
127			if ((int)$siteId !== 0) {
128			$query->andWhere(['siteId' => $siteId]);
129			}
130			$stats = $query->all();
131			}
132			if ($db->getIsPgsql()) {
133			// Query the db table
134			$query = (new Query())
135			->select([
			0 ignored issues – show Coding Style introduced 2019-01-29 16:20 UTC by Report Bug Copy Issue Report The opening parenthesis of a multi-line function call should be the last content on the line. Loading history...
136			'COUNT("url") AS cnt',
137
138			'AVG("pageLoad") AS pageLoad',
139			'AVG("domInteractive") AS domInteractive',
140			'AVG("firstContentfulPaint") AS firstContentfulPaint',
141			'AVG("firstPaint") AS firstPaint',
142			'AVG("firstByte") AS firstByte',
143			'AVG("connect") AS connect',
144			'AVG("dns") AS dns',
145
146			'AVG("craftTotalMs") AS craftTotalMs',
147			'AVG("craftDbMs") AS craftDbMs',
148			'AVG("craftDbCnt") AS craftDbCnt',
149			'AVG("craftTwigMs") AS craftTwigMs',
150			'AVG("craftTwigCnt") AS craftTwigCnt',
151			'AVG("craftOtherMs") AS craftOtherMs',
152			'AVG("craftOtherCnt") AS craftOtherCnt',
153			'AVG("craftTotalMemory") AS craftTotalMemory',
154			])
			0 ignored issues – show Coding Style introduced 2019-01-29 16:20 UTC by Report Bug Copy Issue Report For multi-line function calls, the closing parenthesis should be on a new line. If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); // Closing parenthesis on a new line. Loading history...
155			->from(['{{%webperf_data_samples}}'])
156			->where(['between', 'dateCreated', $start, $end]);
157			if (!empty($pageUrl)) {
158			$query->andWhere(['url' => $pageUrl]);
159			}
160			if ((int)$siteId !== 0) {
161			$query->andWhere(['siteId' => $siteId]);
162			}
163			$stats = $query->all();
164			}
165			if ($stats) {
			0 ignored issues – show Comprehensibility Best Practice introduced 2019-01-29 16:20 UTC by Report Bug Copy Issue Report The variable `$stats` does not seem to be defined for all execution paths leading up to this point. Loading history...
166			$data = $stats[0];
167			}
168
169			return $data;
170			}
171			}
172

nystudio107 / craft-webperf

Push — v1 ( 3980d8...7dfb05 )

Recommendations::data() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like