Skip to content

Commit

Permalink
Core: Disable deprecated $.format
Browse files Browse the repository at this point in the history
Throwing error if $.format is called and updated all deprecated calls to use
$.validator.format.

Fixes gh-755
Closes gh-971
  • Loading branch information
bartsipes authored and jzaefferer committed Jan 14, 2014
1 parent 70ae69e commit bf3b350
Show file tree
Hide file tree
Showing 11 changed files with 39 additions and 38 deletions.
2 changes: 1 addition & 1 deletion src/additional/accept.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,4 +30,4 @@ jQuery.validator.addMethod("accept", function(value, element, param) {
// Either return true because we've validated each file, or because the
// browser does not support element.files and the FileList feature
return true;
}, jQuery.format("Please enter a value with a valid mimetype."));
}, jQuery.validator.format("Please enter a value with a valid mimetype."));
2 changes: 1 addition & 1 deletion src/additional/extension.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@
jQuery.validator.addMethod("extension", function(value, element, param) {
param = typeof param === "string" ? param.replace(/,/g, '|') : "png|jpe?g|gif";
return this.optional(element) || value.match(new RegExp(".(" + param + ")$", "i"));
}, jQuery.format("Please enter a value with a valid extension."));
}, jQuery.validator.format("Please enter a value with a valid extension."));
2 changes: 1 addition & 1 deletion src/additional/require_from_group.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,4 +34,4 @@ jQuery.validator.addMethod("require_from_group", function(value, element, option
$fields.data('being_validated', false);
}
return isValid;
}, jQuery.format("Please fill at least {0} of these fields."));
}, jQuery.validator.format("Please fill at least {0} of these fields."));
2 changes: 1 addition & 1 deletion src/additional/skip_or_fill_minimum.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,4 +40,4 @@ jQuery.validator.addMethod("skip_or_fill_minimum", function(value, element, opti
$fields.data('being_validated', false);
}
return isValid;
}, jQuery.format("Please either skip these fields or fill at least {0} of them."));
}, jQuery.validator.format("Please either skip these fields or fill at least {0} of them."));
5 changes: 3 additions & 2 deletions src/core.js
Original file line number Diff line number Diff line change
Expand Up @@ -1199,8 +1199,9 @@ $.extend($.validator, {

});

// deprecated, use $.validator.format instead
$.format = $.validator.format;
$.format = function deprecated() {
throw "$.format has been deprecated. Please use $.validator.format instead.";
};

}(jQuery));

12 changes: 6 additions & 6 deletions src/localization/messages_ja.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@
creditcard: "有効なクレジットカード番号を入力してください。",
equalTo: "同じ値をもう一度入力してください。",
extension: "有効な拡張子を含む値を入力してください。",
maxlength: $.format("{0} 文字以内で入力してください。"),
minlength: $.format("{0} 文字以上で入力してください。"),
rangelength: $.format("{0} 文字から {1} 文字までの値を入力してください。"),
range: $.format("{0} から {1} までの値を入力してください。"),
max: $.format("{0} 以下の値を入力してください。"),
min: $.format("{0} 以上の値を入力してください。")
maxlength: $.validator.format("{0} 文字以内で入力してください。"),
minlength: $.validator.format("{0} 文字以上で入力してください。"),
rangelength: $.validator.format("{0} 文字から {1} 文字までの値を入力してください。"),
range: $.validator.format("{0} から {1} までの値を入力してください。"),
max: $.validator.format("{0} 以下の値を入力してください。"),
min: $.validator.format("{0} 以上の値を入力してください。")
});
}(jQuery));
12 changes: 6 additions & 6 deletions src/localization/messages_kk.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@
creditcard: "Несие картасының нөмірін дұрыс енгізуіңізді сұраймыз.",
equalTo: "Осы мәнді қайта енгізуіңізді сұраймыз.",
extension: "Файлдың кеңейтуін дұрыс таңдаңыз.",
maxlength: $.format("Ұзындығы {0} символдан көр болмасын."),
minlength: $.format("Ұзындығы {0} символдан аз болмасын."),
rangelength: $.format("Ұзындығы {0}-{1} дейін мән енгізуіңізді сұраймыз."),
range: $.format("Пожалуйста, введите число от {0} до {1}. - {0} - {1} санын енгізуіңізді сұраймыз."),
max: $.format("{0} аз немесе тең санын енгізуіңіді сұраймыз."),
min: $.format("{0} көп немесе тең санын енгізуіңізді сұраймыз.")
maxlength: $.validator.format("Ұзындығы {0} символдан көр болмасын."),
minlength: $.validator.format("Ұзындығы {0} символдан аз болмасын."),
rangelength: $.validator.format("Ұзындығы {0}-{1} дейін мән енгізуіңізді сұраймыз."),
range: $.validator.format("Пожалуйста, введите число от {0} до {1}. - {0} - {1} санын енгізуіңізді сұраймыз."),
max: $.validator.format("{0} аз немесе тең санын енгізуіңіді сұраймыз."),
min: $.validator.format("{0} көп немесе тең санын енгізуіңізді сұраймыз.")
});
}(jQuery));
12 changes: 6 additions & 6 deletions src/localization/messages_ko.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@
creditcard: "신용카드 번호가 바르지 않습니다.",
equalTo: "같은 값을 다시 입력하세요.",
extension: "올바른 확장자가 아닙니다.",
maxlength: $.format("{0}자를 넘을 수 없습니다. "),
minlength: $.format("{0}자 이상 입력하세요."),
rangelength: $.format("문자 길이가 {0} 에서 {1} 사이의 값을 입력하세요."),
range: $.format("{0} 에서 {1} 사이의 값을 입력하세요."),
max: $.format("{0} 이하의 값을 입력하세요."),
min: $.format("{0} 이상의 값을 입력하세요.")
maxlength: $.validator.format("{0}자를 넘을 수 없습니다. "),
minlength: $.validator.format("{0}자 이상 입력하세요."),
rangelength: $.validator.format("문자 길이가 {0} 에서 {1} 사이의 값을 입력하세요."),
range: $.validator.format("{0} 에서 {1} 사이의 값을 입력하세요."),
max: $.validator.format("{0} 이하의 값을 입력하세요."),
min: $.validator.format("{0} 이상의 값을 입력하세요.")
});
}(jQuery));
12 changes: 6 additions & 6 deletions src/localization/messages_lt.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@
creditcard: "Prašau įvesti teisingą kreditinės kortelės numerį.",
equalTo: "Prašau įvestį tą pačią reikšmę dar kartą.",
extension: "Prašau įvesti reikšmę su teisingu plėtiniu.",
maxlength: $.format("Prašau įvesti ne daugiau kaip {0} simbolių."),
minlength: $.format("Prašau įvesti bent {0} simbolius."),
rangelength: $.format("Prašau įvesti reikšmes, kurių ilgis nuo {0} iki {1} simbolių."),
range: $.format("Prašau įvesti reikšmę intervale nuo {0} iki {1}."),
max: $.format("Prašau įvesti reikšmę mažesnę arba lygią {0}."),
min: $.format("Prašau įvesti reikšmę didesnę arba lygią {0}.")
maxlength: $.validator.format("Prašau įvesti ne daugiau kaip {0} simbolių."),
minlength: $.validator.format("Prašau įvesti bent {0} simbolius."),
rangelength: $.validator.format("Prašau įvesti reikšmes, kurių ilgis nuo {0} iki {1} simbolių."),
range: $.validator.format("Prašau įvesti reikšmę intervale nuo {0} iki {1}."),
max: $.validator.format("Prašau įvesti reikšmę mažesnę arba lygią {0}."),
min: $.validator.format("Prašau įvesti reikšmę didesnę arba lygią {0}.")
});
}(jQuery));
12 changes: 6 additions & 6 deletions src/localization/messages_vi.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@
creditcard: "Hãy nhập số thẻ tín dụng.",
equalTo: "Hãy nhập thêm lần nữa.",
extension: "Phần mở rộng không đúng.",
maxlength: $.format("Hãy nhập từ {0} kí tự trở xuống."),
minlength: $.format("Hãy nhập từ {0} kí tự trở lên."),
rangelength: $.format("Hãy nhập từ {0} đến {1} kí tự."),
range: $.format("Hãy nhập từ {0} đến {1}."),
max: $.format("Hãy nhập từ {0} trở xuống."),
min: $.format("Hãy nhập từ {1} trở lên.")
maxlength: $.validator.format("Hãy nhập từ {0} kí tự trở xuống."),
minlength: $.validator.format("Hãy nhập từ {0} kí tự trở lên."),
rangelength: $.validator.format("Hãy nhập từ {0} đến {1} kí tự."),
range: $.validator.format("Hãy nhập từ {0} đến {1}."),
max: $.validator.format("Hãy nhập từ {0} trở xuống."),
min: $.validator.format("Hãy nhập từ {1} trở lên.")
});
}(jQuery));
4 changes: 2 additions & 2 deletions test/methods.js
Original file line number Diff line number Diff line change
Expand Up @@ -885,7 +885,7 @@ test("pattern", function() {
function testCardTypeByNumber(number, cardname, expected) {
$("#cardnumber").val(number);
var actual = $("#ccform").valid();
equal(actual, expected, $.format("Expect card number {0} to validate to {1}, actually validated to ", number, expected));
equal(actual, expected, $.validator.format("Expect card number {0} to validate to {1}, actually validated to ", number, expected));
}

test('creditcardtypes, all', function() {
Expand Down Expand Up @@ -952,7 +952,7 @@ function fillFormWithValuesAndExpect(formSelector, inputValues, expected) {
$(formSelector + ' input:eq(' + i + ')').val(inputValues[i]);
}
var actual = $(formSelector).valid();
equal(actual, expected, $.format("Filled inputs of form '{0}' with {1} values ({2})", formSelector, inputValues.length, inputValues.toString()));
equal(actual, expected, $.validator.format("Filled inputs of form '{0}' with {1} values ({2})", formSelector, inputValues.length, inputValues.toString()));

}

Expand Down

0 comments on commit bf3b350

Please sign in to comment.