Replaced deprecated jQuery functions #458
Conversation
| @@ -446,10 +446,14 @@ $.extend(true, laravelValidation, { | |||
| * @return {boolean} | |||
| */ | |||
| In: function(value, element, params) { | |||
| if ($.isArray(value) && laravelValidation.helpers.hasRules(element, "Array")) { | |||
| if (laravelValidation.helpers.isArray(value) | |||
| && laravelValidation.helpers.hasRules(element, "Array") | |||
There was a problem hiding this comment.
Misleading line break before '&&'; readers may interpret this as an expression boundary.
Mixed double and single quotes.
| errors[ element.name ] = previous.message = $.isFunction( message ) ? message( value ) : message[0]; | ||
| errors[ element.name ] | ||
| = previous.message | ||
| = typeof message === "function" ? message( value ) : message[0]; |
There was a problem hiding this comment.
Line is too long.
Misleading line break before '='; readers may interpret this as an expression boundary.
Mixed double and single quotes.
| @@ -224,7 +226,9 @@ laravelValidation = { | |||
| } else { | |||
| errors = {}; | |||
| message = response || validator.defaultMessage( element, "remote" ); | |||
| errors[ element.name ] = previous.message = $.isFunction( message ) ? message( value ) : message[0]; | |||
| errors[ element.name ] | |||
| = previous.message | |||
There was a problem hiding this comment.
Misleading line break before '='; readers may interpret this as an expression boundary.
| errors[ element.name ] = previous.message = $.isFunction( message ) ? message( value ) : message; | ||
| errors[ element.name ] | ||
| = previous.message | ||
| = typeof message === "function" ? message( value ) : message; |
There was a problem hiding this comment.
Line is too long.
Misleading line break before '='; readers may interpret this as an expression boundary.
Mixed double and single quotes.
| @@ -104,7 +104,9 @@ laravelValidation = { | |||
| validator.showErrors(); | |||
| } else { | |||
| var errors = {}; | |||
| errors[ element.name ] = previous.message = $.isFunction( message ) ? message( value ) : message; | |||
| errors[ element.name ] | |||
| = previous.message | |||
There was a problem hiding this comment.
Misleading line break before '='; readers may interpret this as an expression boundary.
Description
Replaces deprecated jQuery 3 functions listed at https://api.jquery.com/category/deprecated/