Slash issue

Refer: https://processwire.com/talk/topic/8697-jumplinks/?page=12#comment-126561
mikerockett · Aug 26, 2016 · 59b35e2 · 59b35e2
1 parent 7ac20ad
commit 59b35e2
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/ProcessJumplinks.module.php b/ProcessJumplinks.module.php
@@ -602,6 +602,10 @@ protected function scanAndRedirect()
                 array('\?', '/?', '\&', '\:'),
                 $jumplink->source
             ));
+            // As a workaround for query strings attached to trailing slashes
+            // ensure that slashes are not made optional in the middle of the source.
+            // Refer: https://processwire.com/talk/topic/8697-jumplinks/?do=findComment&comment=126551
+            $source = preg_replace('~(.+)/\?(.+)~', '\\1/\?\\2', $source);
 
             // Reverse ':' escaping for wildcards
             $source = preg_replace("~\{([a-z]+)\\\:([a-z]+)\}~i", "{\\1:\\2}", $source);