Merge pull request #11226 from marmelab/fix-escape-path-escaping

Escape backslashes in escapePath

Author: slax57

packages/ra-core/src/util/escapePath.spec.ts

@@ -0,0 +1,13 @@
+import expect from 'expect';
+
+import escapePath from './escapePath';
+
+describe('escapePath', () => {
+    it('escapes parentheses', () => {
+        expect(escapePath('/foo(bar)')).toEqual('/foo\\(bar\\)');
+    });
+
+    it('escapes backslashes before escaping parentheses', () => {
+        expect(escapePath('/foo\\(bar)')).toEqual('/foo\\\\\\(bar\\)');
+    });
+});

packages/ra-core/src/util/escapePath.ts

@@ -7,4 +7,4 @@
  *
  * escapePath('/foo(bar)') => 'foo\(bar\)'
  */
-export default url => url.replace(/(\(|\))/g, '\\$1');
+export default url => url.replace(/\\/g, '\\\\').replace(/[()]/g, '\\$&');