nystudio107 / craft-webperf

src/services/DataSamples.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
             ->from(['{{%webperf_data_samples}}'])
             ->where(['not', [$column => null]])
             ;
-        if ((int)$siteId !== 0) {
+        if ((int) $siteId !== 0) {
             $query->andWhere(['siteId' => $siteId]);
         }
 
@@ -74,7 +74,7 @@ 
                 ['not', ['title' => '']],
             ])
         ;
-        if ((int)$siteId !== 0) {
+        if ((int) $siteId !== 0) {
             $query->andWhere(['siteId' => $siteId]);
         }
         $result = $query->one();

src/base/Recommendation.php

Spacing +1 added lines, -1 removed lines

@@ -45,6 +45,6 @@
      */
     public function displayMs(int $number): string
     {
-        return number_format((float)$number / 1000, 2).'s';
+        return number_format((float) $number / 1000, 2).'s';
     }
 }

src/controllers/ChartsController.php

Spacing +8 added lines, -8 removed lines

@@ -81,7 +81,7 @@ 
             ])
             ->where(['between', 'dateCreated', $start, $end])
             ->andWhere(['not', [$column => null]]);
-        if ((int)$siteId !== 0) {
+        if ((int) $siteId !== 0) {
             $query->andWhere(['siteId' => $siteId]);
         }
         if ($pageUrl !== '') {
@@ -91,7 +91,7 @@ 
         // Massage the data
         if ($stats) {
             foreach ($stats as &$stat) {
-                $stat['cnt'] = (int)$stat['cnt'];
+                $stat['cnt'] = (int) $stat['cnt'];
             }
             $data = $stats[0];
         }
@@ -136,7 +136,7 @@ 
             ])
             ->where(['between', 'dateCreated', $start, $end])
             ->andWhere(['not', [$column => null]]);
-        if ((int)$siteId !== 0) {
+        if ((int) $siteId !== 0) {
             $query->andWhere(['siteId' => $siteId]);
         }
         $query
@@ -147,7 +147,7 @@ 
         // Massage the data
         if ($stats) {
             foreach ($stats as &$stat) {
-                $stat['cnt'] = (int)$stat['cnt'];
+                $stat['cnt'] = (int) $stat['cnt'];
                 $stat['detailPageUrl'] = UrlHelper::cpUrl('webperf/performance/page-detail', [
                     'pageUrl' => $stat['url'],
                     'siteId' => $siteId,
@@ -218,7 +218,7 @@ 
             ])
             ->where(['between', 'dateCreated', $start, $end])
             ;
-        if ((int)$siteId !== 0) {
+        if ((int) $siteId !== 0) {
             $query->andWhere(['siteId' => $siteId]);
         }
         if ($pageUrl !== '') {
@@ -347,7 +347,7 @@ 
             ])
             ->where(['between', 'dateCreated', $start, $end])
         ;
-        if ((int)$siteId !== 0) {
+        if ((int) $siteId !== 0) {
             $query->andWhere(['siteId' => $siteId]);
         }
         if ($pageUrl !== '') {
@@ -391,11 +391,11 @@ 
                 $craftErrors['labels'][$index] = $stat['sampleDate'];
                 switch ($stat['type']) {
                     case 'boomerang':
-                        $boomerangErrors['data'][$index] = (int)$stat['cnt'];
+                        $boomerangErrors['data'][$index] = (int) $stat['cnt'];
                         $boomerangErrors['labels'][$index] = $stat['sampleDate'];
                         break;
                     case 'craft':
-                        $craftErrors['data'][$index] = (int)$stat['cnt'];
+                        $craftErrors['data'][$index] = (int) $stat['cnt'];
                         $craftErrors['labels'][$index] = $stat['sampleDate'];
                         break;
                 }

src/services/ErrorSamples.php

Spacing +3 added lines, -3 removed lines

@@ -82,7 +82,7 @@ 
             ->from(['{{%webperf_error_samples}}'])
             ->where(['not', [$column => null]])
             ;
-        if ((int)$siteId !== 0) {
+        if ((int) $siteId !== 0) {
             $query->andWhere(['siteId' => $siteId]);
         }
 
@@ -120,7 +120,7 @@ 
                 ->andWhere(['url' => $pageUrl])
             ;
         }
-        if ((int)$siteId !== 0) {
+        if ((int) $siteId !== 0) {
             $query->andWhere(['siteId' => $siteId]);
         }
 
@@ -146,7 +146,7 @@ 
                 ['not', ['title' => '']],
             ])
         ;
-        if ((int)$siteId !== 0) {
+        if ((int) $siteId !== 0) {
             $query->andWhere(['siteId' => $siteId]);
         }
         $result = $query->one();

src/Webperf.php

Spacing +16 added lines, -16 removed lines

@@ -315,7 +315,7 @@ 
         Event::on(
             Plugins::class,
             Plugins::EVENT_AFTER_INSTALL_PLUGIN,
-            function (PluginEvent $event) {
+            function(PluginEvent $event) {
                 if ($event->plugin === $this) {
                     // Invalidate our caches after we've been installed
                     $this->clearAllCaches();
@@ -343,7 +343,7 @@ 
         Event::on(
             CraftVariable::class,
             CraftVariable::EVENT_INIT,
-            function (Event $event) {
+            function(Event $event) {
                 /** @var CraftVariable $variable */
                 $variable = $event->sender;
                 $variable->set('webperf', WebperfVariable::class);
@@ -353,7 +353,7 @@ 
         Event::on(
             Plugins::class,
             Plugins::EVENT_AFTER_LOAD_PLUGINS,
-            function () {
+            function() {
                 // Install these only after all other plugins have loaded
                 $request = Craft::$app->getRequest();
                 // Only respond to non-console site requests
@@ -377,7 +377,7 @@ 
         Event::on(
             UrlManager::class,
             UrlManager::EVENT_REGISTER_SITE_URL_RULES,
-            function (RegisterUrlRulesEvent $event) {
+            function(RegisterUrlRulesEvent $event) {
                 Craft::debug(
                     'UrlManager::EVENT_REGISTER_SITE_URL_RULES',
                     __METHOD__
@@ -400,7 +400,7 @@ 
         Event::on(
             UrlManager::class,
             UrlManager::EVENT_REGISTER_CP_URL_RULES,
-            function (RegisterUrlRulesEvent $event) {
+            function(RegisterUrlRulesEvent $event) {
                 Craft::debug(
                     'UrlManager::EVENT_REGISTER_CP_URL_RULES',
                     __METHOD__
@@ -416,7 +416,7 @@ 
         Event::on(
             UserPermissions::class,
             UserPermissions::EVENT_REGISTER_PERMISSIONS,
-            function (RegisterUserPermissionsEvent $event) {
+            function(RegisterUserPermissionsEvent $event) {
                 Craft::debug(
                     'UserPermissions::EVENT_REGISTER_PERMISSIONS',
                     __METHOD__
@@ -446,7 +446,7 @@ 
             Event::on(
                 View::class,
                 View::EVENT_END_PAGE,
-                function () {
+                function() {
                     Craft::debug(
                         'View::EVENT_END_PAGE',
                         __METHOD__
@@ -474,7 +474,7 @@ 
             Event::on(
                 View::class,
                 View::EVENT_END_BODY,
-                function () {
+                function() {
                     Craft::debug(
                         'View::EVENT_END_BODY',
                         __METHOD__
@@ -497,7 +497,7 @@ 
             Event::on(
                 Application::class,
                 Application::EVENT_AFTER_REQUEST,
-                function () {
+                function() {
                     Craft::debug(
                         'Application::EVENT_AFTER_REQUEST',
                         __METHOD__
@@ -523,21 +523,21 @@ 
         if (self::$settings->displaySidebar && $currentUser && $currentUser->can('webperf:sidebar')) {
             $view = Craft::$app->getView();
             // Entries sidebar
-            $view->hook('cp.entries.edit.details', function (&$context) {
+            $view->hook('cp.entries.edit.details', function(&$context) {
                 /** @var  Element $element */
                 $element = $context['entry'] ?? null;
 
                 return $this->renderSidebar($element);
             });
             // Category Groups sidebar
-            $view->hook('cp.categories.edit.details', function (&$context) {
+            $view->hook('cp.categories.edit.details', function(&$context) {
                 /** @var  Element $element */
                 $element = $context['category'] ?? null;
 
                 return $this->renderSidebar($element);
             });
             // Commerce Product Types sidebar
-            $view->hook('cp.commerce.product.edit.details', function (&$context) {
+            $view->hook('cp.commerce.product.edit.details', function(&$context) {
                 /** @var  Element $element */
                 $element = $context['product'] ?? null;
 
@@ -755,7 +755,7 @@ 
     {
         $cache = Craft::$app->getCache();
         // See if there are any recommendations to add as a badge
-        $recommendations = $cache->getOrSet(self::RECOMMENDATIONS_CACHE_KEY, function () {
+        $recommendations = $cache->getOrSet(self::RECOMMENDATIONS_CACHE_KEY, function() {
             $data = [];
             $now = new \DateTime();
             $end = $now->format('Y-m-d');
@@ -773,7 +773,7 @@ 
             $recommendations = '';
         }
 
-        return (string)$recommendations;
+        return (string) $recommendations;
     }
 
     /**
@@ -785,7 +785,7 @@ 
     {
         $cache = Craft::$app->getCache();
         // See if there are any recommendations to add as a badge
-        $errors = $cache->getOrSet(self::ERRORS_CACHE_KEY, function () {
+        $errors = $cache->getOrSet(self::ERRORS_CACHE_KEY, function() {
             $now = new \DateTime();
             $end = $now->format('Y-m-d');
             $start = $now->modify('-30 days')->format('Y-m-d');
@@ -799,6 +799,6 @@ 
             $errors = '⚠';
         }
 
-        return (string)$errors;
+        return (string) $errors;
     }
 }

src/recommendations/MemoryLimit.php

Spacing +7 added lines, -7 removed lines

@@ -35,14 +35,14 @@ 
      */
     public function evaluate()
     {
-        $phpMemoryLimit = (int)$this->memoryLimit();
-        $this->sample->craftTotalMemory = (int)$this->sample->craftTotalMemory;
+        $phpMemoryLimit = (int) $this->memoryLimit();
+        $this->sample->craftTotalMemory = (int) $this->sample->craftTotalMemory;
         if ($phpMemoryLimit && $this->sample->craftTotalMemory) {
             $ratio = $phpMemoryLimit / $this->sample->craftTotalMemory;
-            $displayCraftTotalMemory = (($this->sample->craftTotalMemory / 1024) / 1024) . 'M';
-            $displayPhpMemoryLimit = (($phpMemoryLimit / 1024) / 1024) . 'M';
-            $displayCraftMinMemory = ((self::MIN_CRAFT_MEMORY / 1024) / 1024) . 'M';
-            $displayCraftMaxMemory = ((self::MAX_CRAFT_MEMORY / 1024) / 1024) . 'M';
+            $displayCraftTotalMemory = (($this->sample->craftTotalMemory / 1024) / 1024).'M';
+            $displayPhpMemoryLimit = (($phpMemoryLimit / 1024) / 1024).'M';
+            $displayCraftMinMemory = ((self::MIN_CRAFT_MEMORY / 1024) / 1024).'M';
+            $displayCraftMaxMemory = ((self::MAX_CRAFT_MEMORY / 1024) / 1024).'M';
             $this->summary = Craft::t(
                 'webperf',
                 'Check the `memory_limit` setting in your `php.ini` file',
@@ -114,6 +114,6 @@ 
             }
         }
 
-        return (int)$memoryLimit;
+        return (int) $memoryLimit;
     }
 }

src/controllers/MetricsController.php

Spacing +3 added lines, -3 removed lines

@@ -163,8 +163,8 @@ 
             if ($userAgent) {
                 $parser = new Parser($userAgent);
                 $sample->device = ($parser->device->model ?? '');
-                $sample->browser = ($parser->browser->name ?? '') . ' ' . $parser->browser->getVersion();
-                $sample->os = ($parser->os->name ?? '') . ' ' . $parser->os->getVersion();
+                $sample->browser = ($parser->browser->name ?? '').' '.$parser->browser->getVersion();
+                $sample->os = ($parser->os->name ?? '').' '.$parser->os->getVersion();
                 $sample->mobile = $parser->isMobile();
             }
             // Save the data sample
@@ -201,7 +201,7 @@ 
             $now = round(microtime(true) * 1000);
             $cache = Craft::$app->getCache();
             $then = $cache->get(self::LAST_BEACON_CACHE_KEY);
-            if (($then !== false) && ($now - (int)$then < Webperf::$settings->rateLimitMs)) {
+            if (($then !== false) && ($now - (int) $then < Webperf::$settings->rateLimitMs)) {
                 $limited = true;
                 Craft::warning('Beacon ignored due to rate limiting', __METHOD__);
             }