refactor: Only look at adding site prefixes to pathonly redirects via GraphQL

khalwat · khalwat · commit 5ae06e95dd06 · 2025-03-13T11:41:37.000-04:00
diff --git a/src/gql/resolvers/RetourResolver.php b/src/gql/resolvers/RetourResolver.php
@@ -76,7 +76,7 @@ public static function resolve(mixed $source, array $arguments, mixed $context,
                 Retour::$plugin->statistics->incrementStatistics($uri, false, $siteId);
             }
         }
-        if ($redirect !== null && isset($redirect['redirectDestUrl'])) {
+        if ($redirect !== null && isset($redirect['redirectDestUrl']) && $redirect['redirectSrcMatch'] === 'pathonly') {
             $dest = $redirect['redirectDestUrl'];
             $path = $redirect['redirectDestUrl'];
             // Combine the URL and path together, merging them as appropriate

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ public static function resolve(mixed $source, array $arguments, mixed $context,`
`76`	`76`	`Retour::$plugin->statistics->incrementStatistics($uri, false, $siteId);`
`77`	`77`	`}`
`78`	`78`	`}`
`79`		`- if ($redirect !== null && isset($redirect['redirectDestUrl'])) {`
	`79`	`+ if ($redirect !== null && isset($redirect['redirectDestUrl']) && $redirect['redirectSrcMatch'] === 'pathonly') {`
`80`	`80`	`$dest = $redirect['redirectDestUrl'];`
`81`	`81`	`$path = $redirect['redirectDestUrl'];`
`82`	`82`	`// Combine the URL and path together, merging them as appropriate`