Added validation for voucher generation field (#373)

* Check voucher count input field before generating

* changed voucher-bulk-codes-num to numeric type

* Added Jan Felix to AUTHORS

* replaced alert by validation colors

* added another formatting reset

...so that it does not stay red if someone submits something valid
This commit is contained in:
Jan Felix Wiebe
2017-01-15 15:42:15 +01:00
committed by Raphael Michel
parent d4b3bf4370
commit 63bdb397e7
3 changed files with 17 additions and 7 deletions

View File

@@ -13,7 +13,7 @@
<div class="form-group">
<div class="col-md-7 col-sm-12 col-md-offset-3">
<div class="input-group">
<input type="text" class="form-control input-xs"
<input type="number" class="form-control input-xs"
id="voucher-bulk-codes-num"
placeholder="{% trans "Number" %}">
<div class="input-group-btn">

View File

@@ -96,12 +96,21 @@ $(function () {
// Vouchers
$("#voucher-bulk-codes-generate").click(function () {
var url = $(this).attr("data-rng-url"),
num = $("#voucher-bulk-codes-num").val();
$("#id_codes").html("Generating...");
$.getJSON(url + '?num=' + num, function (data) {
$("#id_codes").val(data.codes.join("\n"));
});
var num = $("#voucher-bulk-codes-num").val();
if (num != "") {
$(".form-group:has(#voucher-bulk-codes-num)").addClass("has-error");
var url = $(this).attr("data-rng-url");
$("#id_codes").html("Generating...");
$.getJSON(url + '?num=' + num, function (data) {
$("#id_codes").val(data.codes.join("\n"));
});
} else {
$(".form-group:has(#voucher-bulk-codes-num)").addClass("has-error");
$("#voucher-bulk-codes-num").focus();
setTimeout(function() {
$(".form-group:has(#voucher-bulk-codes-num)").removeClass("has-error");
}, 3000);
}
});
$("#ajaxerr").on("click", ".ajaxerr-close", ajaxErrDialog.hide);