Passed
Push — develop ( 9ebfda...cfec7a )
by Andrew
04:32
created
src/helpers/MultiSite.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -42,53 +42,53 @@  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
                     'All Sites'
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['currentSiteId'] = 0;
91
-            $variables['sitesMenuLabel'] = '';
90
+            $variables[ 'currentSiteId' ] = 0;
91
+            $variables[ 'sitesMenuLabel' ] = '';
92 92
         }
93 93
     }
94 94
 
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
         if ($siteHandle !== null) {
107 107
             $site = Craft::$app->getSites()->getSiteByHandle($siteHandle);
108 108
             if (!$site) {
109
-                throw new NotFoundHttpException('Invalid site handle: '.$siteHandle);
109
+                throw new NotFoundHttpException('Invalid site handle: ' . $siteHandle);
110 110
             }
111 111
             $siteId = $site->id;
112 112
         } else {
Please login to merge, or discard this patch.
src/controllers/TablesController.php 1 patch
Spacing   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
         $siteId = 0
63 63
     ): Response {
64 64
         PermissionHelper::controllerPermissionCheck('retour:dashboard');
65
-        $data = [];
65
+        $data = [ ];
66 66
         $sortField = 'hitCount';
67 67
         $sortType = 'DESC';
68 68
         // Figure out the sorting type
@@ -76,39 +76,39 @@  discard block
 block discarded – undo
76 76
         // Query the db table
77 77
         $offset = ($page - 1) * $per_page;
78 78
         $query = (new Query())
79
-            ->from(['{{%retour_stats}}'])
79
+            ->from([ '{{%retour_stats}}' ])
80 80
             ->offset($offset)
81 81
             ->limit($per_page)
82 82
             ->orderBy("{$sortField} {$sortType}");
83
-        if ((int)$siteId !== 0) {
84
-            $query->where(['siteId' => $siteId]);
83
+        if ((int) $siteId !== 0) {
84
+            $query->where([ 'siteId' => $siteId ]);
85 85
         }
86 86
         if ($filter !== '') {
87
-            $query->where(['like', 'redirectSrcUrl', $filter]);
88
-            $query->orWhere(['like', 'referrerUrl', $filter]);
87
+            $query->where([ 'like', 'redirectSrcUrl', $filter ]);
88
+            $query->orWhere([ 'like', 'referrerUrl', $filter ]);
89 89
         }
90 90
         $stats = $query->all();
91 91
         if ($stats) {
92 92
             // Add in the `addLink` field
93 93
             foreach ($stats as &$stat) {
94
-                $stat['addLink'] = '';
95
-                if (!$stat['handledByRetour']) {
96
-                    $encodedUrl = urlencode('/'.ltrim($stat['redirectSrcUrl'], '/'));
97
-                    $stat['addLink'] = UrlHelper::cpUrl('retour/add-redirect', [
94
+                $stat[ 'addLink' ] = '';
95
+                if (!$stat[ 'handledByRetour' ]) {
96
+                    $encodedUrl = urlencode('/' . ltrim($stat[ 'redirectSrcUrl' ], '/'));
97
+                    $stat[ 'addLink' ] = UrlHelper::cpUrl('retour/add-redirect', [
98 98
                         'defaultUrl' => $encodedUrl
99 99
                     ]);
100 100
                 }
101 101
             }
102 102
             // Format the data for the API
103
-            $data['data'] = $stats;
103
+            $data[ 'data' ] = $stats;
104 104
             $query = (new Query())
105
-                ->from(['{{%retour_stats}}']);
105
+                ->from([ '{{%retour_stats}}' ]);
106 106
             if ($filter !== '') {
107
-                $query->where(['like', 'redirectSrcUrl', $filter]);
108
-                $query->orWhere(['like', 'referrerUrl', $filter]);
107
+                $query->where([ 'like', 'redirectSrcUrl', $filter ]);
108
+                $query->orWhere([ 'like', 'referrerUrl', $filter ]);
109 109
             }
110 110
             $count = $query->count();
111
-            $data['links']['pagination'] = [
111
+            $data[ 'links' ][ 'pagination' ] = [
112 112
                 'total' => $count,
113 113
                 'per_page' => $per_page,
114 114
                 'current_page' => $page,
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
         $siteId = 0
144 144
     ): Response {
145 145
         PermissionHelper::controllerPermissionCheck('retour:redirects');
146
-        $data = [];
146
+        $data = [ ];
147 147
         $sortField = 'hitCount';
148 148
         $sortType = 'DESC';
149 149
         // Figure out the sorting type
@@ -157,34 +157,34 @@  discard block
 block discarded – undo
157 157
         // Query the db table
158 158
         $offset = ($page - 1) * $per_page;
159 159
         $query = (new Query())
160
-            ->from(['{{%retour_static_redirects}}'])
160
+            ->from([ '{{%retour_static_redirects}}' ])
161 161
             ->offset($offset)
162 162
             ->limit($per_page)
163 163
             ->orderBy("{$sortField} {$sortType}");
164
-        if ((int)$siteId !== 0) {
165
-            $query->where(['siteId' => $siteId]);
164
+        if ((int) $siteId !== 0) {
165
+            $query->where([ 'siteId' => $siteId ]);
166 166
         }
167 167
         if ($filter !== '') {
168
-            $query->where(['like', 'redirectSrcUrl', $filter]);
169
-            $query->orWhere(['like', 'redirectDestUrl', $filter]);
168
+            $query->where([ 'like', 'redirectSrcUrl', $filter ]);
169
+            $query->orWhere([ 'like', 'redirectDestUrl', $filter ]);
170 170
         }
171 171
         $redirects = $query->all();
172 172
         // Add in the `deleteLink` field
173 173
         foreach ($redirects as &$redirect) {
174
-            $redirect['deleteLink'] = UrlHelper::cpUrl('retour/delete-redirect/'.$redirect['id']);
175
-            $redirect['editLink'] = UrlHelper::cpUrl('retour/edit-redirect/'.$redirect['id']);
174
+            $redirect[ 'deleteLink' ] = UrlHelper::cpUrl('retour/delete-redirect/' . $redirect[ 'id' ]);
175
+            $redirect[ 'editLink' ] = UrlHelper::cpUrl('retour/edit-redirect/' . $redirect[ 'id' ]);
176 176
         }
177 177
         // Format the data for the API
178 178
         if ($redirects) {
179
-            $data['data'] = $redirects;
179
+            $data[ 'data' ] = $redirects;
180 180
             $query = (new Query())
181
-                ->from(['{{%retour_static_redirects}}']);
181
+                ->from([ '{{%retour_static_redirects}}' ]);
182 182
             if ($filter !== '') {
183
-                $query->where(['like', 'redirectSrcUrl', $filter]);
184
-                $query->orWhere(['like', 'redirectDestUrl', $filter]);
183
+                $query->where([ 'like', 'redirectSrcUrl', $filter ]);
184
+                $query->orWhere([ 'like', 'redirectDestUrl', $filter ]);
185 185
             }
186 186
             $count = $query->count();
187
-            $data['links']['pagination'] = [
187
+            $data[ 'links' ][ 'pagination' ] = [
188 188
                 'total' => $count,
189 189
                 'per_page' => $per_page,
190 190
                 'current_page' => $page,
Please login to merge, or discard this patch.