fix, add use first flags property, and support dotAll flag (ramda#3158)

* fix, add use first flags property, and support dotAll flag * add dotAll flag test case * move eslint-env comment to top * remove eslint-env comment
kedashoe · Jan 22, 2022 · 1bd60c5 · 1bd60c5
1 parent 7277709
commit 1bd60c5
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/source/internal/_cloneRegExp.js b/source/internal/_cloneRegExp.js
@@ -1,7 +1,8 @@
 export default function _cloneRegExp(pattern) {
-  return new RegExp(pattern.source, (pattern.global     ? 'g' : '') +
-                                    (pattern.ignoreCase ? 'i' : '') +
-                                    (pattern.multiline  ? 'm' : '') +
-                                    (pattern.sticky     ? 'y' : '') +
-                                    (pattern.unicode    ? 'u' : ''));
+  return new RegExp(pattern.source, (pattern.flags ?  pattern.flags : (pattern.global     ? 'g' : '') +
+                                                                      (pattern.ignoreCase ? 'i' : '') +
+                                                                      (pattern.multiline  ? 'm' : '') +
+                                                                      (pattern.sticky     ? 'y' : '') +
+                                                                      (pattern.unicode    ? 'u' : '') +
+                                                                      (pattern.dotAll     ? 's' : '')));
 }
diff --git a/test/test.js b/test/test.js
@@ -5,6 +5,9 @@ var eq = require('./shared/eq.js');
 
 
 describe('test', function() {
+  it('returns true if string matches dotAll pattern', function() {
+    eq(R.test(/x.*z/s, 'x.\nyz'), true);
+  });
 
   it('returns true if string matches pattern', function() {
     eq(R.test(/^x/, 'xyz'), true);