From cf576a12e54b51f563dccc77b783fce7de6b5768 Mon Sep 17 00:00:00 2001
From: Alexander Marx <amarx@ipfire.org>
Date: Fri, 18 Jan 2013 10:18:25 +0100
Subject: [PATCH] Forward Firewall: Bugfix: when editing an rule, the default
 ACTION (ACCEPT,DROP,REJECT) was set depending on Firewallmode. Now it checks,
 if a rule is edited and value is set to rule action

---
 html/cgi-bin/forwardfw.cgi | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/html/cgi-bin/forwardfw.cgi b/html/cgi-bin/forwardfw.cgi
index 8fae77123..b2976e1c3 100755
--- a/html/cgi-bin/forwardfw.cgi
+++ b/html/cgi-bin/forwardfw.cgi
@@ -1014,15 +1014,21 @@ print <<END;
 END
 	foreach ("ACCEPT","DROP","REJECT")
 	{
-		if($fwdfwsettings{'POLICY'} eq 'MODE2'){
-			$fwdfwsettings{'RULE_ACTION'} = 'DROP';
-		}
-
-		if ($_ eq $fwdfwsettings{'RULE_ACTION'})
-		{
-			print"<option selected>$_</option>";
+		if($fwdfwsettings{'updatefwrule'} eq 'on'){
+			print"<option ";
+			print "selected='selected'" if ($fwdfwsettings{'RULE_ACTION'} eq $_);
+			print">$_</option>";
 		}else{
-			print"<option>$_</option>";
+			if($fwdfwsettings{'POLICY'} eq 'MODE2'){
+				$fwdfwsettings{'RULE_ACTION'} = 'DROP';
+			}
+	
+			if ($_ eq $fwdfwsettings{'RULE_ACTION'})
+			{
+				print"<option selected>$_</option>";
+			}else{
+				print"<option>$_</option>";
+			}
 		}
 	}
 	print"</select></td></tr></table><hr>";