refactor: When saving a redirect, Retour no longer deletes redirects …

…that have the `redirectDestUrl` of the redirect being saved as their `redirectSrcUrl` ([#303](#303))
nystudio107 · Jun 24, 2024 · 794cbf6 · 794cbf6
1 parent 566b52d
commit 794cbf6
Showing 1 changed file with 0 additions and 24 deletions.
diff --git a/src/services/Redirects.php b/src/services/Redirects.php
@@ -1172,30 +1172,6 @@ public function saveRedirect(array $redirectConfig): bool
                 return false;
             }
         }
-        // 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);