Merge branch 'release/3.2.28' into v3

Author: khalwat

CHANGELOG.md

@@ -1,5 +1,13 @@
 # Retour Changelog
 
+## 3.2.28 - 2025.08.17
+### Changed
+* Use `StringHelper::convertToUtf8()` instead of our homebrew solution
+* Throw an exception if an attempt is made to edit a redirect that doesn't exist ([#345](https://github.com/nystudio107/craft-retour/issues/345))
+
+### Fixed
+* Fixed a regression that caused the redirect loop detection to no longer function ([#348](https://github.com/nystudio107/craft-retour/issues/348))
+
 ## 3.2.27 - 2025.06.13
 ### Changed
 * Allow `RedirectEvent`s to change the redirect that is saved ([#342](https://github.com/nystudio107/craft-retour/issues/342))

buildchain/package-lock.json

@@ -2,7 +2,7 @@
   "name": "nystudio107/craft-retour",
   "description": "Retour allows you to intelligently redirect legacy URLs, so that you don't lose SEO value when rebuilding & restructuring a website",
   "type": "craft-plugin",
-  "version": "3.2.27",
+  "version": "3.2.28",
   "keywords": [
     "craftcms",
     "craft-plugin",

composer.json

@@ -135,15 +135,7 @@ public function actionEditRedirect(
         if ($redirect === null) {
             $redirectConfig = Retour::$plugin->redirects->getRedirectById($redirectId);
             if ($redirectConfig === null) {
-                $redirectConfig = [];
-                Craft::error(
-                    Craft::t(
-                        'retour',
-                        "Couldn't load redirect id {id}",
-                        ['id' => $redirectId]
-                    ),
-                    __METHOD__
-                );
+                throw new NotFoundHttpException('Redirect not found');
             }
             $redirect = new StaticRedirectsModel($redirectConfig);
         }
@@ -279,7 +271,7 @@ public function actionSaveRedirect()
         }
         // Save the redirect
         $redirectConfig = $redirect->getAttributes();
-        Retour::$plugin->redirects->saveRedirect($redirectConfig);
+        Retour::$plugin->redirects->saveRedirect($redirectConfig, false);
         // Handle the case where the redirect wasn't saved because it'd create a redirect loop
         $testRedirectConfig = Retour::$plugin->redirects->getRedirectByRedirectSrcUrl(
             $redirectConfig['redirectSrcUrl'],

docs/package-lock.json

@@ -11,6 +11,7 @@
 
 namespace nystudio107\retour\helpers;
 
+use craft\helpers\StringHelper;
 use Stringy\Stringy;
 
 /**
@@ -87,12 +88,7 @@ public static function cleanupText($text): string
             return '';
         }
         // Convert to UTF-8
-        if (\function_exists('iconv')) {
-            $text = iconv(mb_detect_encoding($text, mb_detect_order(), true), 'UTF-8//IGNORE', $text);
-        } else {
-            ini_set('mbstring.substitute_character', 'none');
-            $text = mb_convert_encoding($text, 'UTF-8', 'UTF-8');
-        }
+        $text = StringHelper::convertToUtf8($text);
         // Strip HTML tags
         $text = strip_tags($text);
         // Remove excess whitespace

src/controllers/RedirectsController.php

@@ -1086,9 +1086,10 @@ public function deleteShortlinkById(int $redirectId)
 
     /**
      * @param array $redirectConfig
-     * @return bool whether the redirect was saved or not
+     * @param bool $checkForRedirectLoop
+     * @return bool
      */
-    public function saveRedirect(array $redirectConfig): bool
+    public function saveRedirect(array $redirectConfig, bool $checkForRedirectLoop = true): bool
     {
         // Handle URL encoded URLs by decoding them before saving them
         if (isset($redirectConfig['redirectMatchType']) && $redirectConfig['redirectMatchType'] === 'exactmatch') {
@@ -1201,6 +1202,32 @@ public function saveRedirect(array $redirectConfig): bool
                 return false;
             }
         }
+        if ($checkForRedirectLoop) {
+            // To prevent redirect loops, see if any static redirects have our redirectDestUrl as their redirectSrcUrl
+            $testRedirectConfig = $this->getRedirectByRedirectSrcUrl(
+                $redirectConfig['redirectDestUrl'],
+                $redirectConfig['siteId']
+            );
+            if ($testRedirectConfig !== null) {
+                Craft::debug(
+                    Craft::t(
+                        'retour',
+                        'Deleting redirect to prevent a loop: {redirect}',
+                        ['redirect' => print_r($testRedirectConfig, true)]
+                    ),
+                    __METHOD__
+                );
+                // Delete the redirect that has a redirectSrcUrl the same as this record's redirectDestUrl
+                try {
+                    $db->createCommand()->delete(
+                        '{{%retour_static_redirects}}',
+                        ['id' => $testRedirectConfig['id']]
+                    )->execute();
+                } catch (Exception $e) {
+                    Craft::error($e->getMessage(), __METHOD__);
+                }
+            }
+        }
         // Trigger a 'afterSaveRedirect' event
         $this->trigger(self::EVENT_AFTER_SAVE_REDIRECT, $event);