nystudio107 / craft-retour

src/services/ServicesTrait.php

Spacing +4 added lines, -4 removed lines

@@ -39,8 +39,8 @@ 
         // so we can't extract it from the passed in $config
         $majorVersion = '4';
         // Dev server container name & port are based on the major version of this plugin
-        $devPort = 3000 + (int)$majorVersion;
-        $versionName = 'v' . $majorVersion;
+        $devPort = 3000 + (int) $majorVersion;
+        $versionName = 'v'.$majorVersion;
         return [
             'components' => [
                 'events' => Events::class,
@@ -51,8 +51,8 @@ 
                     'assetClass' => RetourAsset::class,
                     'checkDevServer' => true,
                     'class' => VitePluginService::class,
-                    'devServerInternal' => 'http://craft-retour-' . $versionName . '-buildchain-dev:' . $devPort,
-                    'devServerPublic' => 'http://localhost:' . $devPort,
+                    'devServerInternal' => 'http://craft-retour-'.$versionName.'-buildchain-dev:'.$devPort,
+                    'devServerPublic' => 'http://localhost:'.$devPort,
                     'errorEntry' => 'src/js/Retour.js',
                     'useDevServer' => true,
                 ],

src/services/Redirects.php

Spacing +20 added lines, -20 removed lines

@@ -264,17 +264,17 @@ 
      */
     public function excludeUri($uri): bool
     {
-        $uri = '/' . ltrim($uri, '/');
+        $uri = '/'.ltrim($uri, '/');
         if (!empty(Retour::$settings->excludePatterns)) {
             foreach (Retour::$settings->excludePatterns as $excludePattern) {
-                $pattern = '`' . $excludePattern['pattern'] . '`i';
+                $pattern = '`'.$excludePattern['pattern'].'`i';
                 try {
                     if (preg_match($pattern, $uri) === 1) {
                         return true;
                     }
                 } catch (\Exception $e) {
                     // That's fine
-                    Craft::error('Invalid exclude URI Regex: ' . $pattern, __METHOD__);
+                    Craft::error('Invalid exclude URI Regex: '.$pattern, __METHOD__);
                 }
             }
         }
@@ -289,7 +289,7 @@ 
      *
      * @return bool|array|null
      */
-    public function findRedirectMatch(string $fullUrl, string $pathOnly, $siteId = null): bool|array|null
+    public function findRedirectMatch(string $fullUrl, string $pathOnly, $siteId = null): bool | array | null
     {
         // Get the current site
         if ($siteId === null) {
@@ -344,10 +344,10 @@ 
      *
      * @return bool|array
      */
-    public function getRedirectFromCache(string $url, int $siteId = 0): bool|array
+    public function getRedirectFromCache(string $url, int $siteId = 0): bool | array
     {
         $cache = Craft::$app->getCache();
-        $cacheKey = $this::CACHE_KEY . md5($url) . $siteId;
+        $cacheKey = $this::CACHE_KEY.md5($url).$siteId;
         $redirect = $cache->get($cacheKey);
         Craft::info(
             Craft::t(
@@ -392,7 +392,7 @@ 
                         'id' => $redirectConfig['id'],
                     ]
                 )->execute();
-                Craft::debug('Rows affected: ' . $rowsAffected, __METHOD__);
+                Craft::debug('Rows affected: '.$rowsAffected, __METHOD__);
             } catch (\Exception $e) {
                 Craft::error($e->getMessage(), __METHOD__);
             }
@@ -413,12 +413,12 @@ 
         } catch (SiteNotFoundException $e) {
             $siteId = 1;
         }
-        $cacheKey = $this::CACHE_KEY . md5($url) . $siteId;
+        $cacheKey = $this::CACHE_KEY.md5($url).$siteId;
         // Create the dependency tags
         $dependency = new TagDependency([
             'tags' => [
                 $this::GLOBAL_REDIRECTS_CACHE_TAG,
-                $this::GLOBAL_REDIRECTS_CACHE_TAG . $siteId,
+                $this::GLOBAL_REDIRECTS_CACHE_TAG.$siteId,
             ],
         ]);
         $cache->set($cacheKey, $redirect, Retour::$cacheDuration, $dependency);
@@ -644,7 +644,7 @@ 
         foreach ($redirects as $redirect) {
             // Figure out what type of source matching to do
             $redirectSrcMatch = $redirect['redirectSrcMatch'] ?? 'pathonly';
-            $redirectEnabled = (bool)$redirect['enabled'];
+            $redirectEnabled = (bool) $redirect['enabled'];
             if ($redirectEnabled === true) {
                 switch ($redirectSrcMatch) {
                     case 'pathonly':
@@ -685,12 +685,12 @@ 
 
                     // Do a regex match
                     case 'regexmatch':
-                        $matchRegEx = '`' . $redirect['redirectSrcUrlParsed'] . '`i';
+                        $matchRegEx = '`'.$redirect['redirectSrcUrlParsed'].'`i';
                         try {
                             if (preg_match($matchRegEx, $url) === 1) {
                                 $this->incrementRedirectHitCount($redirect);
                                 // If we're not associated with an EntryID, handle capture group replacement
-                                if ((int)$redirect['associatedElementId'] === 0) {
+                                if ((int) $redirect['associatedElementId'] === 0) {
                                     $redirect['redirectDestUrl'] = preg_replace(
                                         $matchRegEx,
                                         $redirect['redirectDestUrl'],
@@ -718,7 +718,7 @@ 
                             }
                         } catch (\Exception $e) {
                             // That's fine
-                            Craft::error('Invalid Redirect Regex: ' . $matchRegEx, __METHOD__);
+                            Craft::error('Invalid Redirect Regex: '.$matchRegEx, __METHOD__);
                         }
 
                         break;
@@ -843,7 +843,7 @@ 
                 try {
                     $siteId = $redirect['siteId'] ?? null;
                     if ($siteId !== null) {
-                        $siteId = (int)$siteId;
+                        $siteId = (int) $siteId;
                     }
                     $dest = UrlHelper::siteUrl($dest, null, null, $siteId);
                 } catch (\yii\base\Exception $e) {
@@ -858,7 +858,7 @@ 
                     // That's ok
                 }
                 if (!empty($queryString)) {
-                    $dest = strtok($dest, '?') . '?' . $queryString;
+                    $dest = strtok($dest, '?').'?'.$queryString;
                 }
             }
             $redirectMatchType = $redirect['redirectMatchType'] ?? 'notfound';
@@ -926,7 +926,7 @@ 
         foreach (Craft::$app->getPlugins()->getAllPlugins() as $plugin) {
             /** @var Plugin $plugin */
             if (method_exists($plugin, 'retourMatch')) {
-                $result[$plugin->getHandle()] = $plugin->name . Craft::t('retour', ' Match');
+                $result[$plugin->getHandle()] = $plugin->name.Craft::t('retour', ' Match');
             }
         }
 
@@ -963,7 +963,7 @@ 
         $db = Craft::$app->getDb();
         // Trigger a 'beforeDeleteRedirect' event
         $redirectConfig = $this->getRedirectById($id);
-        $isNew = (int)$redirectConfig['id'] === 0;
+        $isNew = (int) $redirectConfig['id'] === 0;
         $event = new RedirectEvent([
             'isNew' => $isNew,
             'legacyUrl' => $redirectConfig['redirectSrcUrlParsed'],
@@ -1138,7 +1138,7 @@ 
         // Get the validated model attributes and save them to the db
         $redirectConfig = $redirect->getAttributes();
         // 0 for a siteId needs to be converted to null
-        if (empty($redirectConfig['siteId']) || (int)$redirectConfig['siteId'] === 0) {
+        if (empty($redirectConfig['siteId']) || (int) $redirectConfig['siteId'] === 0) {
             $redirectConfig['siteId'] = null;
         }
         // Throw an event to before saving the redirect
@@ -1148,7 +1148,7 @@ 
         }
 
         // See if a redirect exists with this source URL already
-        if ((int)$redirectConfig['id'] === 0) {
+        if ((int) $redirectConfig['id'] === 0) {
             // Query the db table
             $redirect = (new Query())
                 ->from(['{{%retour_static_redirects}}'])
@@ -1161,7 +1161,7 @@ 
             }
         }
         // Trigger a 'beforeSaveRedirect' event
-        $isNew = (int)$redirectConfig['id'] === 0;
+        $isNew = (int) $redirectConfig['id'] === 0;
         $event = new RedirectEvent([
             'isNew' => $isNew,
             'legacyUrl' => $redirectConfig['redirectSrcUrlParsed'],

src/services/Statistics.php

Spacing +4 added lines, -4 removed lines

@@ -76,7 +76,7 @@ 
     public function getRecentStatistics(int $days = 1, bool $handled = false): array
     {
         // Ensure is an int
-        $handledInt = (int)$handled;
+        $handledInt = (int) $handled;
         $stats = [];
         $db = Craft::$app->getDb();
         if ($db->getIsMysql()) {
@@ -211,9 +211,9 @@ 
         $stats->userAgent = $userAgent;
         $stats->exceptionMessage = $exceptionMessage;
         $stats->exceptionFilePath = $exceptionFilePath;
-        $stats->exceptionFileLine = (int)$exceptionFileLine;
+        $stats->exceptionFileLine = (int) $exceptionFileLine;
         $stats->hitLastTime = Db::prepareDateForDb(new DateTime());
-        $stats->handledByRetour = (int)$handled;
+        $stats->handledByRetour = (int) $handled;
         $stats->hitCount++;
         $statsConfig = $stats->getAttributes();
         // Record the updated statistics
@@ -286,7 +286,7 @@ 
         $now = round(microtime(true) * 1000);
         $cache = Craft::$app->getCache();
         $then = $cache->get(self::LAST_STATISTICS_TRIM_CACHE_KEY);
-        if (($then !== false) && ($now - (int)$then < Retour::$settings->statisticsRateLimitMs)) {
+        if (($then !== false) && ($now - (int) $then < Retour::$settings->statisticsRateLimitMs)) {
             $limited = true;
         }
         $cache->set(self::LAST_STATISTICS_TRIM_CACHE_KEY, $now, 0);

src/console/controllers/StatsController.php

Spacing +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
     /**
      * @var    array|bool
      */
-    protected array|bool $allowAnonymous = [
+    protected array | bool $allowAnonymous = [
     ];
 
     // Public Methods
@@ -63,13 +63,13 @@ 
      */
     public function actionTrim(): int
     {
-        echo Craft::t('retour', 'Trimming statistics') . PHP_EOL;
+        echo Craft::t('retour', 'Trimming statistics').PHP_EOL;
         $affectedRows = Retour::$plugin->statistics->trimStatistics($this->limit);
         echo Craft::t(
                 'retour',
                 'Trimmed {rows} from retour_stats table',
                 ['rows' => $affectedRows]
-            ) . PHP_EOL;
+            ).PHP_EOL;
 
         return 0;
     }

src/helpers/FileLog.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
      */
     public static function getLogFilePath(string $fileName): string
     {
-        $logfile = $fileName . "-" . date(self::FILE_PER_DAY);
+        $logfile = $fileName."-".date(self::FILE_PER_DAY);
 
         return Craft::getAlias("@storage/logs/$logfile.log");
     }