View GitHub Repository craft-retour

Schedule Inspection
Subscribe
Passed
Push — develop ( 43cf79...ba3995 )
by Andrew
04:55
created
Status
Category
Toggle Patches
src/controllers/StatisticsController.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch 
@@ -40,7 +40,7 @@  discard block
 
 block discarded – undo
40 40 
     // Protected Properties
41 41 
     // =========================================================================
42 42
43 
-    protected $allowAnonymous = [];
43 
+    protected $allowAnonymous = [ ];
44 44
45 45 
     // Public Methods
46 46 
     // =========================================================================
 
@@ -56,7 +56,7 @@  discard block
 
 block discarded – undo
56 56 
      */
57 57 
     public function actionDashboard(string $siteHandle = null, bool $showWelcome = false): Response
58 58 
     {
59 
-        $variables = [];
59 
+        $variables = [ ];
60 60 
         PermissionHelper::controllerPermissionCheck('retour:dashboard');
61 61 
         // Trim the statistics
62 62 
         Retour::$plugin->statistics->trimStatistics();
 
@@ -71,20 +71,20 @@  discard block
 
 block discarded – undo
71 71 
         } catch (InvalidConfigException $e) {
72 72 
             Craft::error($e->getMessage(), __METHOD__);
73 73 
         }
74 
-        $variables['baseAssetsUrl'] = Craft::$app->assetManager->getPublishedUrl(
74 
+        $variables[ 'baseAssetsUrl' ] = Craft::$app->assetManager->getPublishedUrl(
75 75 
             '@nystudio107/retour/assetbundles/retour/dist',
76 76 
             true
77 77 
         );
78 78 
         // Enabled sites
79 79 
         MultiSiteHelper::setMultiSiteVariables($siteHandle, $siteId, $variables);
80 
-        $variables['controllerHandle'] = 'dashboard';
80 
+        $variables[ 'controllerHandle' ] = 'dashboard';
81 81
82 82 
         // Basic variables
83 
-        $variables['fullPageForm'] = false;
84 
-        $variables['docsUrl'] = self::DOCUMENTATION_URL;
85 
-        $variables['pluginName'] = $pluginName;
86 
-        $variables['title'] = $templateTitle;
87 
-        $variables['crumbs'] = [
83 
+        $variables[ 'fullPageForm' ] = false;
84 
+        $variables[ 'docsUrl' ] = self::DOCUMENTATION_URL;
85 
+        $variables[ 'pluginName' ] = $pluginName;
86 
+        $variables[ 'title' ] = $templateTitle;
87 
+        $variables[ 'crumbs' ] = [
88 88 
             [
89 89 
                 'label' => $pluginName,
90 90 
                 'url' => UrlHelper::cpUrl('retour'),
 
@@ -94,9 +94,9 @@  discard block
 
 block discarded – undo
94 94 
                 'url' => UrlHelper::cpUrl('retour/dashboard'),
95 95 
             ],
96 96 
         ];
97 
-        $variables['docTitle'] = "{$pluginName} - {$templateTitle}";
98 
-        $variables['selectedSubnavItem'] = 'dashboard';
99 
-        $variables['showWelcome'] = $showWelcome;
97 
+        $variables[ 'docTitle' ] = "{$pluginName} - {$templateTitle}";
98 
+        $variables[ 'selectedSubnavItem' ] = 'dashboard';
99 
+        $variables[ 'showWelcome' ] = $showWelcome;
100 100
101 101 
         // Render the template
102 102 
         return $this->renderTemplate('retour/dashboard/index', $variables);
 
@@ -114,7 +114,7 @@  discard block
 
 block discarded – undo
114 114 
             Craft::t(
115 115 
                 'retour',
116 116 
                 'Retour statistics cleared: {error}',
117 
-                ['error' => $error]
117 
+                [ 'error' => $error ]
118 118 
             ),
119 119 
             __METHOD__
120 120 
         );
Please login to merge, or discard this patch.
src/helpers/MultiSite.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch 
@@ -42,52 +42,52 @@  discard block
 
 block discarded – undo
42 42 
         $sites = Craft::$app->getSites();
43 43 
         if (Craft::$app->getIsMultiSite()) {
44 44 
             // Set defaults based on the section settings
45 
-            $variables['enabledSiteIds'] = [];
46 
-            $variables['siteIds'] = [];
45 
+            $variables[ 'enabledSiteIds' ] = [ ];
46 
+            $variables[ 'siteIds' ] = [ ];
47 47
48 48 
             /** @var Site $site */
49 49 
             foreach ($sites->getEditableSiteIds() as $editableSiteId) {
50 
-                $variables['enabledSiteIds'][] = $editableSiteId;
51 
-                $variables['siteIds'][] = $editableSiteId;
50 
+                $variables[ 'enabledSiteIds' ][ ] = $editableSiteId;
51 
+                $variables[ 'siteIds' ][ ] = $editableSiteId;
52 52 
             }
53 53
54 54 
             // Make sure the $siteId they are trying to edit is in our array of editable sites
55 
-            if (!\in_array($siteId, $variables['enabledSiteIds'], false)) {
56 
-                if (!empty($variables['enabledSiteIds'])) {
55 
+            if (!\in_array($siteId, $variables[ 'enabledSiteIds' ], false)) {
56 
+                if (!empty($variables[ 'enabledSiteIds' ])) {
57 57 
                     if ($siteId !== 0) {
58 
-                        $siteId = reset($variables['enabledSiteIds']);
58 
+                        $siteId = reset($variables[ 'enabledSiteIds' ]);
59 59 
                     }
60 60 
                 } else {
61 
-                    self::requirePermission('editSite:'.$siteId);
61 
+                    self::requirePermission('editSite:' . $siteId);
62 62 
                 }
63 63 
             }
64 64 
         }
65 65 
         // Set the currentSiteId and currentSiteHandle
66 
-        $variables['currentSiteId'] = empty($siteId) ? 0 : $siteId;
67 
-        $variables['currentSiteHandle'] = empty($siteHandle)
66 
+        $variables[ 'currentSiteId' ] = empty($siteId) ? 0 : $siteId;
67 
+        $variables[ 'currentSiteHandle' ] = empty($siteHandle)
68 68 
             ? Craft::$app->getSites()->currentSite->handle
69 69 
             : $siteHandle;
70 70
71 71 
         // Page title
72 
-        $variables['showSites'] = (
72 
+        $variables[ 'showSites' ] = (
73 73 
             Craft::$app->getIsMultiSite() &&
74 
-            \count($variables['enabledSiteIds'])
74 
+            \count($variables[ 'enabledSiteIds' ])
75 75 
         );
76 76
77 
-        if ($variables['showSites']) {
78 
-            if ($variables['currentSiteId'] === 0) {
79 
-                $variables['sitesMenuLabel'] = Craft::t(
77 
+        if ($variables[ 'showSites' ]) {
78 
+            if ($variables[ 'currentSiteId' ] === 0) {
79 
+                $variables[ 'sitesMenuLabel' ] = Craft::t(
80 80 
                     'retour',
81 81 
                     'Global'
82 82 
                 );
83 83 
             } else {
84 
-                $variables['sitesMenuLabel'] = Craft::t(
84 
+                $variables[ 'sitesMenuLabel' ] = Craft::t(
85 85 
                     'site',
86 
-                    $sites->getSiteById((int)$variables['currentSiteId'])->name
86 
+                    $sites->getSiteById((int) $variables[ 'currentSiteId' ])->name
87 87 
                 );
88 88 
             }
89 89 
         } else {
90 
-            $variables['sitesMenuLabel'] = '';
90 
+            $variables[ 'sitesMenuLabel' ] = '';
91 91 
         }
92 92 
     }
93 93 
 
@@ -105,7 +105,7 @@  discard block
 
 block discarded – undo
105 105 
         if ($siteHandle !== null) {
106 106 
             $site = Craft::$app->getSites()->getSiteByHandle($siteHandle);
107 107 
             if (!$site) {
108 
-                throw new NotFoundHttpException('Invalid site handle: '.$siteHandle);
108 
+                throw new NotFoundHttpException('Invalid site handle: ' . $siteHandle);
109 109 
             }
110 110 
             $siteId = $site->id;
111 111 
         } else {
Please login to merge, or discard this patch.