library-mirrors/jquery
1
0
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2024-11-23 02:54:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Build: Fix the regex parsing AMD var-modules (#4389)

Browse Source 
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
This commit is contained in:
Michał Gołębiowski-Owczarek 2019-05-13 21:55:45 +02:00 committed by GitHub
parent 3527a38405
commit 9ec09c3b4a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
build/tasks/build.js
View File

@ -65,7 +65,7 @@ module.exports = function( grunt ) {
if ( /.\/var\//.test( path.replace( process.cwd(), "" ) ) ) { if ( /.\/var\//.test( path.replace( process.cwd(), "" ) ) ) {
contents = contents contents = contents
.replace( .replace(
/define\([\w\W]*?return/, /define\(\s*(["'])[\w\W]*?\1[\w\W]*?return/,
"var " + "var " +
( /var\/([\w-]+)/.exec( name )[ 1 ] ) + ( /var\/([\w-]+)/.exec( name )[ 1 ] ) +
" =" " ="