fix in current name param transformation (#466)

src/js/Router.js

@@ -83,7 +83,7 @@ export default class Router extends String {
 
         // Test the passed name against the current route, matching some
         // basic wildcards, e.g. passing `events.*` matches `events.show`
-        const match = new RegExp(`^${name.replace('.', '\\.').replace('*', '.*')}$`).test(current);
+        const match = new RegExp(`^${name.replace(/\./g, '\\.').replace(/\*/g, '.*')}$`).test(current);
 
         if ([null, undefined].includes(params) || !match) return match;
 

tests/js/route.test.js

@@ -58,6 +58,13 @@ const defaultZiggy = {
             uri: '{locale}/posts/{post}',
             methods: ['PUT', 'PATCH'],
         },
+        'events.venues-index': {
+            uri: 'events/{event}/venues-index',
+            methods: ['GET', 'HEAD'],
+            bindings: {
+                event: 'id',
+            },
+        },
         'events.venues.index': {
             uri: 'events/{event}/venues',
             methods: ['GET', 'HEAD'],
@@ -881,6 +888,25 @@ describe('current()', () => {
         same(route().current(), 'events.venues.index');
     });
 
+    test('does not match similiar named routes', () => {
+        global.window.location.pathname = '/events/1/venues';
+
+        same(route().current('events.venues-index'), false);
+        same(route().current('events.venues.index'), true);
+
+        global.window.location.pathname = '/events/1/venues-index';
+
+        same(route().current('events.venues-index'), true);
+        same(route().current('events.venues.index'), false);
+    });
+
+    test('does not match similiar named routes with wildcards', () => {
+        global.window.location.pathname = '/events/1/venues-index';
+
+        same(route().current('events.venues-index'), true);
+        same(route().current('events.venues.*'), false);
+    });
+
     test('can get the current route name without window', () => {
         global.Ziggy = undefined;
         global.window = undefined;