mirror of
https://github.com/apache/cloudstack.git
synced 2025-11-03 04:12:31 +01:00
replace validateNumber() with validateInteger() on all integer fields on all pages.
This commit is contained in:
parent
cd0f5e7c20
commit
32c1fb77f4
@ -684,11 +684,11 @@ function doResourceLimitsForAccount($actionLink, $detailsTab, $midmenuItem1) {
|
||||
"Save": function() {
|
||||
// validate values
|
||||
var isValid = true;
|
||||
isValid &= validateNumber("Instance Limit", $("#dialog_resource_limits #limits_vm"), $("#dialog_resource_limits #limits_vm_errormsg"), -1, 32000, false);
|
||||
isValid &= validateNumber("Public IP Limit", $("#dialog_resource_limits #limits_ip"), $("#dialog_resource_limits #limits_ip_errormsg"), -1, 32000, false);
|
||||
isValid &= validateNumber("Disk Volume Limit", $("#dialog_resource_limits #limits_volume"), $("#dialog_resource_limits #limits_volume_errormsg"), -1, 32000, false);
|
||||
isValid &= validateNumber("Snapshot Limit", $("#dialog_resource_limits #limits_snapshot"), $("#dialog_resource_limits #limits_snapshot_errormsg"), -1, 32000, false);
|
||||
isValid &= validateNumber("Template Limit", $("#dialog_resource_limits #limits_template"), $("#dialog_resource_limits #limits_template_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Instance Limit", $("#dialog_resource_limits #limits_vm"), $("#dialog_resource_limits #limits_vm_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Public IP Limit", $("#dialog_resource_limits #limits_ip"), $("#dialog_resource_limits #limits_ip_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Disk Volume Limit", $("#dialog_resource_limits #limits_volume"), $("#dialog_resource_limits #limits_volume_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Snapshot Limit", $("#dialog_resource_limits #limits_snapshot"), $("#dialog_resource_limits #limits_snapshot_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Template Limit", $("#dialog_resource_limits #limits_template"), $("#dialog_resource_limits #limits_template_errormsg"), -1, 32000, false);
|
||||
if (!isValid) return;
|
||||
|
||||
var instanceLimit = trim($("#dialog_resource_limits #limits_vm").val());
|
||||
|
||||
@ -537,11 +537,11 @@ function doEditResourceLimits2($actionLink, $detailsTab, $midmenuItem1, $readonl
|
||||
var $resourceLimitsTab = $("#right_panel_content #tab_content_resource_limits");
|
||||
|
||||
var isValid = true;
|
||||
isValid &= validateNumber("Instance Limit", $resourceLimitsTab.find("#limits_vm_edit"), $resourceLimitsTab.find("#limits_vm_edit_errormsg"), -1, 32000, false);
|
||||
isValid &= validateNumber("Public IP Limit", $resourceLimitsTab.find("#limits_ip_edit"), $resourceLimitsTab.find("#limits_ip_edit_errormsg"), -1, 32000, false);
|
||||
isValid &= validateNumber("Disk Volume Limit", $resourceLimitsTab.find("#limits_volume_edit"), $resourceLimitsTab.find("#limits_volume_edit_errormsg"), -1, 32000, false);
|
||||
isValid &= validateNumber("Snapshot Limit", $resourceLimitsTab.find("#limits_snapshot_edit"), $resourceLimitsTab.find("#limits_snapshot_edit_errormsg"), -1, 32000, false);
|
||||
isValid &= validateNumber("Template Limit", $resourceLimitsTab.find("#limits_template_edit"), $resourceLimitsTab.find("#limits_template_edit_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Instance Limit", $resourceLimitsTab.find("#limits_vm_edit"), $resourceLimitsTab.find("#limits_vm_edit_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Public IP Limit", $resourceLimitsTab.find("#limits_ip_edit"), $resourceLimitsTab.find("#limits_ip_edit_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Disk Volume Limit", $resourceLimitsTab.find("#limits_volume_edit"), $resourceLimitsTab.find("#limits_volume_edit_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Snapshot Limit", $resourceLimitsTab.find("#limits_snapshot_edit"), $resourceLimitsTab.find("#limits_snapshot_edit_errormsg"), -1, 32000, false);
|
||||
isValid &= validateInteger("Template Limit", $resourceLimitsTab.find("#limits_template_edit"), $resourceLimitsTab.find("#limits_template_edit_errormsg"), -1, 32000, false);
|
||||
if (!isValid)
|
||||
return;
|
||||
|
||||
|
||||
@ -122,8 +122,8 @@ function afterLoadIpJSP() {
|
||||
$createPortForwardingRow.find("#add_link").bind("click", function(event){
|
||||
var isValid = true;
|
||||
isValid &= validateDropDownBox("Instance", $createPortForwardingRow.find("#vm"), $createPortForwardingRow.find("#vm_errormsg"));
|
||||
isValid &= validateNumber("Public Port", $createPortForwardingRow.find("#public_port"), $createPortForwardingRow.find("#public_port_errormsg"), 1, 65535);
|
||||
isValid &= validateNumber("Private Port", $createPortForwardingRow.find("#private_port"), $createPortForwardingRow.find("#private_port_errormsg"), 1, 65535);
|
||||
isValid &= validateInteger("Public Port", $createPortForwardingRow.find("#public_port"), $createPortForwardingRow.find("#public_port_errormsg"), 1, 65535);
|
||||
isValid &= validateInteger("Private Port", $createPortForwardingRow.find("#private_port"), $createPortForwardingRow.find("#private_port_errormsg"), 1, 65535);
|
||||
if (!isValid)
|
||||
return;
|
||||
|
||||
@ -220,8 +220,8 @@ function afterLoadIpJSP() {
|
||||
// validate values
|
||||
var isValid = true;
|
||||
isValid &= validateString("Name", createLoadBalancerRow.find("#name"), createLoadBalancerRow.find("#name_errormsg"));
|
||||
isValid &= validateNumber("Public Port", createLoadBalancerRow.find("#public_port"), createLoadBalancerRow.find("#public_port_errormsg"), 1, 65535);
|
||||
isValid &= validateNumber("Private Port", createLoadBalancerRow.find("#private_port"), createLoadBalancerRow.find("#private_port_errormsg"), 1, 65535);
|
||||
isValid &= validateInteger("Public Port", createLoadBalancerRow.find("#public_port"), createLoadBalancerRow.find("#public_port_errormsg"), 1, 65535);
|
||||
isValid &= validateInteger("Private Port", createLoadBalancerRow.find("#private_port"), createLoadBalancerRow.find("#private_port_errormsg"), 1, 65535);
|
||||
if (!isValid) return;
|
||||
|
||||
var $template = $("#load_balancer_template").clone();
|
||||
@ -1306,7 +1306,7 @@ function portForwardingJsonToTemplate(jsonObj, $template) {
|
||||
$template.find("#save_link").unbind("click").bind("click", function(event){
|
||||
// validate values
|
||||
var isValid = true;
|
||||
isValid &= validateNumber("Private Port", $rowContainerEdit.find("#private_port"), $rowContainerEdit.find("#private_port_errormsg"), 1, 65535);
|
||||
isValid &= validateInteger("Private Port", $rowContainerEdit.find("#private_port"), $rowContainerEdit.find("#private_port_errormsg"), 1, 65535);
|
||||
if (!isValid) return;
|
||||
|
||||
var $spinningWheel = $rowContainerEdit.find("#spinning_wheel");
|
||||
|
||||
@ -644,7 +644,7 @@ function doCreateVMFromIso($actionLink, $detailsTab, $midmenuItem1) {
|
||||
isValid &= validateString("Name", thisDialog.find("#name"), thisDialog.find("#name_errormsg"), true);
|
||||
isValid &= validateString("Group", thisDialog.find("#group"), thisDialog.find("#group_errormsg"), true);
|
||||
if(thisDialog.find("#size_container").css("display") != "none")
|
||||
isValid &= validateNumber("Size", thisDialog.find("#size"), thisDialog.find("#size_errormsg"));
|
||||
isValid &= validateInteger("Size", thisDialog.find("#size"), thisDialog.find("#size_errormsg"));
|
||||
if (!isValid)
|
||||
return;
|
||||
|
||||
|
||||
@ -1316,7 +1316,7 @@ function addZoneWizardValidatePublicIPRange($thisWizard) {
|
||||
isValid &= validateString("Account", $thisWizard.find("#step4").find("#add_publicip_vlan_account"), $thisWizard.find("#step4").find("#add_publicip_vlan_account_errormsg"), true); //optional
|
||||
|
||||
if (isTagged) {
|
||||
isValid &= validateNumber("VLAN", $thisWizard.find("#step4").find("#add_publicip_vlan_vlan"), $thisWizard.find("#step4").find("#add_publicip_vlan_vlan_errormsg"), 2, 4095);
|
||||
isValid &= validateInteger("VLAN", $thisWizard.find("#step4").find("#add_publicip_vlan_vlan"), $thisWizard.find("#step4").find("#add_publicip_vlan_vlan_errormsg"), 2, 4095);
|
||||
}
|
||||
|
||||
isValid &= validateIp("Gateway", $thisWizard.find("#step4").find("#add_publicip_vlan_gateway"), $thisWizard.find("#step4").find("#add_publicip_vlan_gateway_errormsg"), false); //required
|
||||
|
||||
@ -192,12 +192,12 @@ function initAddIngressRuleDialog() {
|
||||
var isValid = true;
|
||||
|
||||
if(protocol == "ICMP") {
|
||||
isValid &= validateNumber("Type", $thisDialog.find("#icmp_type"), $thisDialog.find("#icmp_type_errormsg"), -1, 40, false); //required
|
||||
isValid &= validateNumber("Code", $thisDialog.find("#icmp_code"), $thisDialog.find("#icmp_code_errormsg"), -1 , 15, false); //required
|
||||
isValid &= validateInteger("Type", $thisDialog.find("#icmp_type"), $thisDialog.find("#icmp_type_errormsg"), -1, 40, false); //required
|
||||
isValid &= validateInteger("Code", $thisDialog.find("#icmp_code"), $thisDialog.find("#icmp_code_errormsg"), -1 , 15, false); //required
|
||||
}
|
||||
else { //TCP, UDP
|
||||
isValid &= validateNumber("Start Port", $thisDialog.find("#start_port"), $thisDialog.find("#start_port_errormsg"), 1, 65535, false); //required
|
||||
isValid &= validateNumber("End Port", $thisDialog.find("#end_port"), $thisDialog.find("#end_port_errormsg"), 1, 65535, false); //required
|
||||
isValid &= validateInteger("Start Port", $thisDialog.find("#start_port"), $thisDialog.find("#start_port_errormsg"), 1, 65535, false); //required
|
||||
isValid &= validateInteger("End Port", $thisDialog.find("#end_port"), $thisDialog.find("#end_port_errormsg"), 1, 65535, false); //required
|
||||
}
|
||||
|
||||
if($thisDialog.find("input[name='ingress_rule_type']:checked").val() == "cidr") {
|
||||
|
||||
@ -676,7 +676,7 @@ function doCreateVMFromTemplate($actionLink, $detailsTab, $midmenuItem1) {
|
||||
isValid &= validateString("Name", thisDialog.find("#name"), thisDialog.find("#name_errormsg"), true);
|
||||
isValid &= validateString("Group", thisDialog.find("#group"), thisDialog.find("#group_errormsg"), true);
|
||||
if(thisDialog.find("#size_container").css("display") != "none")
|
||||
isValid &= validateNumber("Size", thisDialog.find("#size"), thisDialog.find("#size_errormsg"));
|
||||
isValid &= validateInteger("Size", thisDialog.find("#size"), thisDialog.find("#size_errormsg"));
|
||||
if (!isValid)
|
||||
return;
|
||||
|
||||
|
||||
@ -132,7 +132,7 @@ function afterLoadVolumeJSP() {
|
||||
var isValid = true;
|
||||
isValid &= validateString("Name", thisDialog.find("#add_volume_name"), thisDialog.find("#add_volume_name_errormsg"));
|
||||
if(thisDialog.find("#size_container").css("display") != "none")
|
||||
isValid &= validateNumber("Size", thisDialog.find("#size"), thisDialog.find("#size_errormsg"));
|
||||
isValid &= validateInteger("Size", thisDialog.find("#size"), thisDialog.find("#size_errormsg"));
|
||||
if (!isValid) return;
|
||||
|
||||
thisDialog.dialog("close");
|
||||
@ -783,7 +783,7 @@ function doRecurringSnapshot($actionLink, $detailsTab, $midmenuItem1) {
|
||||
return false;
|
||||
case "0":
|
||||
var isValid = true;
|
||||
isValid &= validateNumber("Keep # of snapshots", bottomPanel.find("#edit_max"), bottomPanel.find("#edit_max_errormsg"));
|
||||
isValid &= validateInteger("Keep # of snapshots", bottomPanel.find("#edit_max"), bottomPanel.find("#edit_max_errormsg"));
|
||||
if (!isValid) return;
|
||||
intervalType = "hourly";
|
||||
minute = bottomPanel.find("#edit_minute").val();
|
||||
@ -794,7 +794,7 @@ function doRecurringSnapshot($actionLink, $detailsTab, $midmenuItem1) {
|
||||
|
||||
case "1":
|
||||
var isValid = true;
|
||||
isValid &= validateNumber("Keep # of snapshots", bottomPanel.find("#edit_max"), bottomPanel.find("#edit_max_errormsg"));
|
||||
isValid &= validateInteger("Keep # of snapshots", bottomPanel.find("#edit_max"), bottomPanel.find("#edit_max_errormsg"));
|
||||
if (!isValid) return;
|
||||
intervalType = "daily";
|
||||
minute = bottomPanel.find("#edit_minute").val();
|
||||
@ -811,7 +811,7 @@ function doRecurringSnapshot($actionLink, $detailsTab, $midmenuItem1) {
|
||||
|
||||
case "2":
|
||||
var isValid = true;
|
||||
isValid &= validateNumber("Keep # of snapshots", bottomPanel.find("#edit_max"), bottomPanel.find("#edit_max_errormsg"));
|
||||
isValid &= validateInteger("Keep # of snapshots", bottomPanel.find("#edit_max"), bottomPanel.find("#edit_max_errormsg"));
|
||||
if (!isValid) return;
|
||||
intervalType = "weekly";
|
||||
minute = bottomPanel.find("#edit_minute").val();
|
||||
@ -830,7 +830,7 @@ function doRecurringSnapshot($actionLink, $detailsTab, $midmenuItem1) {
|
||||
|
||||
case "3":
|
||||
var isValid = true;
|
||||
isValid &= validateNumber("Keep # of snapshots", bottomPanel.find("#edit_max"), bottomPanel.find("#edit_max_errormsg"));
|
||||
isValid &= validateInteger("Keep # of snapshots", bottomPanel.find("#edit_max"), bottomPanel.find("#edit_max_errormsg"));
|
||||
if (!isValid) return;
|
||||
intervalType = "monthly";
|
||||
minute = bottomPanel.find("#edit_minute").val();
|
||||
|
||||
@ -474,7 +474,7 @@ function bindAddVLANButton($button, $leftmenuItem1) {
|
||||
isValid &= validateString("Account", $thisDialog.find("#add_publicip_vlan_account"), $thisDialog.find("#add_publicip_vlan_account_errormsg"), true); //optional
|
||||
|
||||
if (isTagged) {
|
||||
isValid &= validateNumber("VLAN", $thisDialog.find("#add_publicip_vlan_vlan"), $thisDialog.find("#add_publicip_vlan_vlan_errormsg"), 2, 4095);
|
||||
isValid &= validateInteger("VLAN", $thisDialog.find("#add_publicip_vlan_vlan"), $thisDialog.find("#add_publicip_vlan_vlan_errormsg"), 2, 4095);
|
||||
}
|
||||
if (isDirect) {
|
||||
isValid &= validateString("Network Name", $thisDialog.find("#add_publicip_vlan_network_name"), $thisDialog.find("#add_publicip_vlan_network_name_errormsg"));
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user