Do not reset source and destination port range values when it's an associated rule created by nat port forward. It fixes #3778

1 files changed, 9 insertions, 5 deletions

diff --git a/usr/local/www/javascript/firewall_rules_edit/firewall_rules_edit.js b/usr/local/www/javascript/firewall_rules_edit/firewall_rules_edit.js
index 08f1d2b..f8d714d 100644
--- a/usr/local/www/javascript/firewall_rules_edit/firewall_rules_edit.js
+++ b/usr/local/www/javascript/firewall_rules_edit/firewall_rules_edit.js
@@ -6,25 +6,29 @@ function ext_change() {
 	if ((document.iform.srcbeginport.selectedIndex == 0) && portsenabled && editenabled) {
 		document.iform.srcbeginport_cust.disabled = 0;
 	} else {
-		document.iform.srcbeginport_cust.value = "";
+		if (editenabled)
+			document.iform.srcbeginport_cust.value = "";
 		document.iform.srcbeginport_cust.disabled = 1;
 	}
 	if ((document.iform.srcendport.selectedIndex == 0) && portsenabled && editenabled) {
 		document.iform.srcendport_cust.disabled = 0;
 	} else {
-		document.iform.srcendport_cust.value = "";
+		if (editenabled)
+			document.iform.srcendport_cust.value = "";
 		document.iform.srcendport_cust.disabled = 1;
 	}
 	if ((document.iform.dstbeginport.selectedIndex == 0) && portsenabled && editenabled) {
 		document.iform.dstbeginport_cust.disabled = 0;
 	} else {
-		document.iform.dstbeginport_cust.value = "";
+		if (editenabled)
+			document.iform.dstbeginport_cust.value = "";
 		document.iform.dstbeginport_cust.disabled = 1;
 	}
 	if ((document.iform.dstendport.selectedIndex == 0) && portsenabled && editenabled) {
 		document.iform.dstendport_cust.disabled = 0;
 	} else {
-		document.iform.dstendport_cust.value = "";
+		if (editenabled)
+			document.iform.dstendport_cust.value = "";
 		document.iform.dstendport_cust.disabled = 1;
 	}
 
@@ -218,4 +222,4 @@ function tcpflags_anyclick(obj) {
 		document.getElementById('tcpheader').style.display= "";
 	}
 }
-//]]>
\ No newline at end of file
+//]]>