Merge pull request #103 from studioespresso/101-craft-5-regex-redirec…

…t-to-other-url-does-not-work

Check if we have a full url to redirect to

.github/workflows/ci.yml

@@ -1,6 +1,7 @@
 name: ci
 on:
   workflow_dispatch:
+  pull_request:
   push:
     branches:
       - develop

src/services/NotFoundService.php

@@ -122,15 +122,19 @@ private function getMatchingRedirect(NotFoundModel $model): RedirectRecord|array
         $regexRedirects = $this->getAllRegexRedirects($model);
         foreach ($regexRedirects as $regexRedirect) {
             $pattern = '`' . $regexRedirect->pattern . '`i';
-            if (preg_match(
-                    $pattern,
-                    $model->urlPath
-                ) === 1) {
-                $parsedUrl = preg_replace($pattern, $regexRedirect->redirect, $model->fullUrl);
-                return ["record" => $regexRedirect, "url" => $parsedUrl];
+
+            if (str_contains($regexRedirect->redirect, 'http')) {
+                $url = $model->urlPath;
+            } else {
+                $url = $model->fullUrl;
             }
-        }
 
+            if (preg_match($pattern, $model->urlPath, $matches)) {
+                // Replace placeholders ($1, $2, etc.) with actual captured groups
+                $finalRedirectUrl = preg_replace($pattern, $regexRedirect->redirect, $url);
+                return ["record" => $regexRedirect, "url" => $finalRedirectUrl];
+            }
+        }
         return false;
     }