Fix: $.map() expects an array, but a string could be passed in during smart search, which could cause an error with the latest jQuery libraries

AllanJard · AllanJard · commit 485b259e5c02 · 2015-04-29T11:10:17.000+01:00
This fixes DataTables/DataTables #546
diff --git a/js/core/core.filter.js b/js/core/core.filter.js
@@ -279,7 +279,7 @@ function _fnFilterCreateSearch( search, regex, smart, caseInsensitive )
 		 * 
 		 * ^(?=.*?\bone\b)(?=.*?\btwo three\b)(?=.*?\bfour\b).*$
 		 */
-		var a = $.map( search.match( /"[^"]+"|[^ ]+/g ) || '', function ( word ) {
+		var a = $.map( search.match( /"[^"]+"|[^ ]+/g ) || [''], function ( word ) {
 			if ( word.charAt(0) === '"' ) {
 				var m = word.match( /^"(.*)"$/ );
 				word = m ? m[1] : word;

Original file line number	Diff line number	Diff line change
`@@ -279,7 +279,7 @@ function _fnFilterCreateSearch( search, regex, smart, caseInsensitive )`
`279`	`279`	`*`
`280`	`280`	`* ^(?=.?\bone\b)(?=.?\btwo three\b)(?=.?\bfour\b).$`
`281`	`281`	`*/`
`282`		`- var a = $.map( search.match( /"[^"]+"\|[^ ]+/g ) \|\| '', function ( word ) {`
	`282`	`+ var a = $.map( search.match( /"[^"]+"\|[^ ]+/g ) \|\| [''], function ( word ) {`
`283`	`283`	`if ( word.charAt(0) === '"' ) {`
`284`	`284`	`var m = word.match( /^"(.*)"$/ );`
`285`	`285`	`word = m ? m[1] : word;`