Merge pull request #389 from inoshperera/master

fixing EMM-1788 calDAV policy validation issue
This commit is contained in:
Milan Perera 2016-10-26 16:57:02 +05:30 committed by GitHub
commit d368cffa99
4 changed files with 35 additions and 45 deletions

View File

@ -1843,14 +1843,8 @@ validateStep["policy-profile"] = function () {
if (continueToCheckNextInputs) { if (continueToCheckNextInputs) {
var calendarAccountPort = $("input#calendar-account-port").val(); var calendarAccountPort = $("input#calendar-account-port").val();
if (!calendarAccountPort) { if (calendarAccountPort) {
validationStatus = { if (!$.isNumeric(calendarAccountPort)) {
"error": true,
"subErrorMsg": "Account Port is empty. You cannot proceed.",
"erroneousFeature": operation
};
continueToCheckNextInputs = false;
} else if (!$.isNumeric(calendarAccountPort)) {
validationStatus = { validationStatus = {
"error": true, "error": true,
"subErrorMsg": "Account Port requires a number input.", "subErrorMsg": "Account Port requires a number input.",
@ -1867,6 +1861,7 @@ validateStep["policy-profile"] = function () {
continueToCheckNextInputs = false; continueToCheckNextInputs = false;
} }
} }
}
// at-last, if the value of continueToCheckNextInputs is still true // at-last, if the value of continueToCheckNextInputs is still true
// this means that no error is found // this means that no error is found

View File

@ -3236,7 +3236,7 @@
</label> </label>
<input id="calendar-account-port" type="text" class="form-control operationDataKeys" <input id="calendar-account-port" type="text" class="form-control operationDataKeys"
data-key="calendarAccountPort" maxlength="5" data-default="" data-key="calendarAccountPort" maxlength="5" data-default=""
placeholder="[ Required field which requires Number Input ]"/> placeholder="[ Optional field which requires Number Input ]"/>
</div> </div>
<div class="wr-input-control"> <div class="wr-input-control">

View File

@ -1601,14 +1601,8 @@ validateStep["policy-profile"] = function () {
if (continueToCheckNextInputs) { if (continueToCheckNextInputs) {
var calendarAccountPort = $("input#calendar-account-port").val(); var calendarAccountPort = $("input#calendar-account-port").val();
if (!calendarAccountPort) { if (calendarAccountPort) {
validationStatus = { if (!$.isNumeric(calendarAccountPort)) {
"error": true,
"subErrorMsg": "Account Port is empty. You cannot proceed.",
"erroneousFeature": operation
};
continueToCheckNextInputs = false;
} else if (!$.isNumeric(calendarAccountPort)) {
validationStatus = { validationStatus = {
"error": true, "error": true,
"subErrorMsg": "Account Port requires a number input.", "subErrorMsg": "Account Port requires a number input.",
@ -1625,6 +1619,7 @@ validateStep["policy-profile"] = function () {
continueToCheckNextInputs = false; continueToCheckNextInputs = false;
} }
} }
}
// at-last, if the value of continueToCheckNextInputs is still true // at-last, if the value of continueToCheckNextInputs is still true
// this means that no error is found // this means that no error is found

View File

@ -3339,7 +3339,7 @@
</label> </label>
<input id="calendar-account-port" type="text" class="form-control operationDataKeys" <input id="calendar-account-port" type="text" class="form-control operationDataKeys"
data-key="calendarAccountPort" maxlength="5" data-default="" data-key="calendarAccountPort" maxlength="5" data-default=""
placeholder="[ Required field which requires Number Input ]"/> placeholder="[ Optional field which requires Number Input ]"/>
</div> </div>
<div class="wr-input-control"> <div class="wr-input-control">