Skip to content

Commit fe14d00

Browse files
committed
Core: Allow 0 as value for autoCreateRanges
Closes jquery-validation#1293
1 parent 4cecd73 commit fe14d00

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

src/core.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1057,12 +1057,12 @@ $.extend( $.validator, {
10571057

10581058
if ( $.validator.autoCreateRanges ) {
10591059
// auto-create ranges
1060-
if ( rules.min && rules.max ) {
1060+
if ( rules.min != null && rules.max != null ) {
10611061
rules.range = [ rules.min, rules.max ];
10621062
delete rules.min;
10631063
delete rules.max;
10641064
}
1065-
if ( rules.minlength && rules.maxlength ) {
1065+
if ( rules.minlength != null && rules.maxlength != null ) {
10661066
rules.rangelength = [ rules.minlength, rules.maxlength ];
10671067
delete rules.minlength;
10681068
delete rules.maxlength;

test/rules.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -84,19 +84,19 @@ test("rules(), merge min/max to range, minlength/maxlength to rangelength", func
8484
$("#testForm1clean").validate({
8585
rules: {
8686
firstnamec: {
87-
min: 5,
88-
max: 12
87+
min: -15,
88+
max: 0
8989
},
9090
lastname: {
91-
minlength: 2,
92-
maxlength: 8
91+
minlength: 0,
92+
maxlength: 10
9393
}
9494
}
9595
});
9696

97-
deepEqual( $("#firstnamec").rules(), { range: [ 5, 12 ] } );
97+
deepEqual( $("#firstnamec").rules(), { range: [ -15, 0 ] } );
98+
deepEqual( $("#lastnamec").rules(), { rangelength: [ 0, 10 ] } );
9899

99-
deepEqual( $("#lastnamec").rules(), { rangelength: [ 2, 8 ] } );
100100
jQuery.validator.autoCreateRanges = false;
101101
});
102102

0 commit comments

Comments
 (0)