aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorScott Ullrich <sullrich@pfsense.org>2007-11-01 18:04:25 +0000
committerScott Ullrich <sullrich@pfsense.org>2007-11-01 18:04:25 +0000
commit0063c2c580ddb72f0459ffda17396d405358490b (patch)
tree3d14425dac004d3e2b5d96ae60f40746843e747f
parent8ce4c10af01cf34b4ae00f5643b64d703a278abd (diff)
downloadpfsense-packages-0063c2c580ddb72f0459ffda17396d405358490b.tar.gz
pfsense-packages-0063c2c580ddb72f0459ffda17396d405358490b.tar.bz2
pfsense-packages-0063c2c580ddb72f0459ffda17396d405358490b.zip
The first line in the function reads: "box = document.forms[1].selectbox;"
Microsoft's javascript system interprets that to say: "box = document.forms.1.selectbox;" which, of course, does not exist on the document being displayed in the browser - this means you can not select the Category for the Rules to view or edit. The "fix" is to remove the [1] in the line so IE-7 and cousins will interpret the line correctly - If this "breaks" other, well mannered browsers then you need to include code in the javascript function to determine "what" browser is performing the action and adjust the variable assignment accordingly .... Ticket #1483
-rw-r--r--packages/snort/snort_rules.php2
1 files changed, 1 insertions, 1 deletions
diff --git a/packages/snort/snort_rules.php b/packages/snort/snort_rules.php
index f1b3ba0f..acad85f8 100644
--- a/packages/snort/snort_rules.php
+++ b/packages/snort/snort_rules.php
@@ -273,7 +273,7 @@ include("head.inc");
<!--
function go()
{
- box = document.forms[1].selectbox;
+ box = document.forms.selectbox;
destination = box.options[box.selectedIndex].value;
if (destination) location.href = destination;
}