Passed
Push — v3 ( d51017...07cc4f )
by Andrew
15:22 queued 15s
created
src/controllers/TablesController.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@  discard block
 block discarded – undo
123 123
             ->orderBy([$sortField => $sortType])
124 124
             ->filterWhere(['like', 'redirectSrcUrl', $filter])
125 125
             ->orFilterWhere(['like', 'referrerUrl', $filter]);
126
-        if ((int)$siteId !== 0) {
126
+        if ((int) $siteId !== 0) {
127 127
             $query->andWhere(['siteId' => $siteId]);
128 128
         }
129 129
         if ($handled !== 'all') {
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
                 }
145 145
                 $stat['addLink'] = '';
146 146
                 if (!$stat['handledByRetour']) {
147
-                    $encodedUrl = urlencode('/' . ltrim($stat['redirectSrcUrl'], '/'));
147
+                    $encodedUrl = urlencode('/'.ltrim($stat['redirectSrcUrl'], '/'));
148 148
                     // Add the siteId to the URL, but keep the current behavior of passing in siteId=0 for "all"
149 149
                     $statSiteId = $stat['siteId'] ?? 0;
150 150
                     try {
@@ -152,7 +152,7 @@  discard block
 block discarded – undo
152 152
                     } catch (SiteNotFoundException $e) {
153 153
                         $primarySite = null;
154 154
                     }
155
-                    if ($primarySite !== null && $statSiteId == (int)$primarySite->id) {
155
+                    if ($primarySite !== null && $statSiteId == (int) $primarySite->id) {
156 156
                         $statSiteId = 0;
157 157
                     }
158 158
                     $stat['addLink'] = UrlHelper::cpUrl('retour/add-redirect', [
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
             ->orderBy([$sortField => $sortType])
228 228
             ->filterWhere(['like', 'redirectSrcUrl', $filter])
229 229
             ->orFilterWhere(['like', 'redirectDestUrl', $filter]);
230
-        if ((int)$siteId !== 0) {
230
+        if ((int) $siteId !== 0) {
231 231
             $query->andWhere(['siteId' => $siteId]);
232 232
         }
233 233
         if ($shortLinks) {
@@ -274,7 +274,7 @@  discard block
 block discarded – undo
274 274
             }
275 275
             $redirect['siteBaseUrl'] = $site->getBaseUrl();
276 276
 
277
-            $redirect['editLink'] = UrlHelper::cpUrl('retour/edit-redirect/' . $redirect['id']);
277
+            $redirect['editLink'] = UrlHelper::cpUrl('retour/edit-redirect/'.$redirect['id']);
278 278
         }
279 279
         // Format the data for the API
280 280
         if ($redirects) {
Please login to merge, or discard this patch.
src/controllers/RedirectsController.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
         $variables['docsUrl'] = self::DOCUMENTATION_URL;
86 86
         $variables['pluginName'] = $pluginName;
87 87
         $variables['title'] = $templateTitle;
88
-        $siteHandleUri = Craft::$app->isMultiSite ? '/' . $siteHandle : '';
88
+        $siteHandleUri = Craft::$app->isMultiSite ? '/'.$siteHandle : '';
89 89
         $variables['crumbs'] = [
90 90
             [
91 91
                 'label' => $pluginName,
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
             ],
94 94
             [
95 95
                 'label' => $templateTitle,
96
-                'url' => UrlHelper::cpUrl('retour/redirects' . $siteHandleUri),
96
+                'url' => UrlHelper::cpUrl('retour/redirects'.$siteHandleUri),
97 97
             ],
98 98
         ];
99 99
         $variables['docTitle'] = "{$pluginName} - {$templateTitle}";
@@ -153,13 +153,13 @@  discard block
 block discarded – undo
153 153
         if ($redirect->siteId) {
154 154
             $site = $sites->getSiteById($redirect->siteId);
155 155
             if ($site) {
156
-                MultiSiteHelper::requirePermission('editSite:' . $site->uid);
156
+                MultiSiteHelper::requirePermission('editSite:'.$site->uid);
157 157
             }
158 158
         }
159 159
         if ($siteId) {
160 160
             $site = $sites->getSiteById($siteId);
161 161
             if ($site) {
162
-                MultiSiteHelper::requirePermission('editSite:' . $site->uid);
162
+                MultiSiteHelper::requirePermission('editSite:'.$site->uid);
163 163
             }
164 164
         }
165 165
         $pluginName = Retour::$settings->pluginName;
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
             ],
196 196
             [
197 197
                 'label' => $templateTitle,
198
-                'url' => UrlHelper::cpUrl('retour/edit-redirect/' . $redirectId),
198
+                'url' => UrlHelper::cpUrl('retour/edit-redirect/'.$redirectId),
199 199
             ],
200 200
         ];
201 201
         $variables['docTitle'] = "{$pluginName} - Redirects - {$templateTitle}";
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
         if ($redirectConfig === null) {
253 253
             throw new NotFoundHttpException('Redirect not found');
254 254
         }
255
-        $redirectConfig['id'] = (int)$redirectConfig['id'];
255
+        $redirectConfig['id'] = (int) $redirectConfig['id'];
256 256
         // Handle enforcing trailing slashes
257 257
         $generalConfig = Craft::$app->getConfig()->getGeneral();
258 258
         if ($generalConfig->addTrailingSlashesToUrls && $redirectConfig['redirectMatchType'] === 'exactmatch') {
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
         $variables['docsUrl'] = self::DOCUMENTATION_URL;
341 341
         $variables['pluginName'] = $pluginName;
342 342
         $variables['title'] = $templateTitle;
343
-        $siteHandleUri = Craft::$app->isMultiSite ? '/' . $siteHandle : '';
343
+        $siteHandleUri = Craft::$app->isMultiSite ? '/'.$siteHandle : '';
344 344
         $variables['crumbs'] = [
345 345
             [
346 346
                 'label' => $pluginName,
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
348 348
             ],
349 349
             [
350 350
                 'label' => $templateTitle,
351
-                'url' => UrlHelper::cpUrl('retour/shortlinks' . $siteHandleUri),
351
+                'url' => UrlHelper::cpUrl('retour/shortlinks'.$siteHandleUri),
352 352
             ],
353 353
         ];
354 354
         $variables['docTitle'] = "{$pluginName} - {$templateTitle}";
@@ -403,13 +403,13 @@  discard block
 block discarded – undo
403 403
         if (!preg_match('/\.[^\/]+$/', $url) && strpos($url, '?') === false && strpos($url, '#') === false) {
404 404
             // If it's a root relative URL, assume it's a site URL
405 405
             if (UrlHelper::isRootRelativeUrl($url)) {
406
-                return rtrim($url, '/') . '/';
406
+                return rtrim($url, '/').'/';
407 407
             }
408 408
             // If the URL matches any of the site's base URLs, assume it's a site URL
409 409
             $sites = Craft::$app->getSites()->getAllSites();
410 410
             foreach ($sites as $site) {
411 411
                 if (strpos($url, $site->getBaseUrl()) === 0) {
412
-                    return rtrim($url, '/') . '/';
412
+                    return rtrim($url, '/').'/';
413 413
                 }
414 414
             }
415 415
         }
Please login to merge, or discard this patch.