Merge pull request #3655 from chevin99/master

Multiple slashes in visualization name re-directs to default app
elastic · Apr 28, 2015 · 2ff889c · 2ff889c
2 parents 5ab0d3d + e97f693
commit 2ff889c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/kibana/utils/slugify_id.js b/src/kibana/utils/slugify_id.js
@@ -10,7 +10,7 @@ define(function (require) {
       '=' : '-equal-'
     };
     _.each(trans, function (val, key) {
-      var regex = new RegExp(key);
+      var regex = new RegExp(key, 'g');
       id = id.replace(regex, val);
     });
     id = id.replace(/[\s]+/g, '-');

diff --git a/test/unit/specs/utils/slugify_id.js b/test/unit/specs/utils/slugify_id.js
@@ -16,7 +16,11 @@ define(function (require) {
       ['test / ^test', 'test-slash-^test'],
       ['test ?  test', 'test-questionmark-test'],
       ['test =  test', 'test-equal-test'],
-      ['test &  test', 'test-ampersand-test']
+      ['test &  test', 'test-ampersand-test'],
+      ['test/test/test', 'test-slash-test-slash-test'],
+      ['test?test?test', 'test-questionmark-test-questionmark-test'],
+      ['test&test&test', 'test-ampersand-test-ampersand-test'],
+      ['test=test=test', 'test-equal-test-equal-test']
     ];
 
     _.each(fixtures, function (fixture) {