diff options
| author | Michał Gołębiowski-Owczarek <m.goleb@gmail.com> | 2019-05-13 21:55:45 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-05-13 21:55:45 +0200 |
| commit | 9ec09c3b4aa5182c2a8b8f51afb861b685a4003c (patch) | |
| tree | 0123d4ccf75975fae8cf92aa9c61d2a81e8c04b2 /build/tasks/build.js | |
| parent | 3527a3840585e6a359cd712591c9c57398357b9b (diff) | |
| download | jquery-9ec09c3b4aa5182c2a8b8f51afb861b685a4003c.tar.gz jquery-9ec09c3b4aa5182c2a8b8f51afb861b685a4003c.zip | |
Build: Fix the regex parsing AMD var-modules (#4389)
The previous regex caused the final jQuery binary to have syntax errors for
var-modules with names starting with "return". For example, the following module
wouldn't work when the file is named `returnTrue.js`:
```js
define( function() {
"use strict";
return function returnTrue() {
return true;
};
} );
```
Closes gh-4389
Diffstat (limited to 'build/tasks/build.js')
| -rw-r--r-- | build/tasks/build.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/build/tasks/build.js b/build/tasks/build.js index 69916bf02..b59ef816e 100644 --- a/build/tasks/build.js +++ b/build/tasks/build.js @@ -65,7 +65,7 @@ module.exports = function( grunt ) { if ( /.\/var\//.test( path.replace( process.cwd(), "" ) ) ) { contents = contents .replace( - /define\([\w\W]*?return/, + /define\(\s*(["'])[\w\W]*?\1[\w\W]*?return/, "var " + ( /var\/([\w-]+)/.exec( name )[ 1 ] ) + " =" |