diff options
author | Scott Ullrich <sullrich@pfsense.org> | 2007-11-01 18:04:25 +0000 |
---|---|---|
committer | Scott Ullrich <sullrich@pfsense.org> | 2007-11-01 18:04:25 +0000 |
commit | 0063c2c580ddb72f0459ffda17396d405358490b (patch) | |
tree | 3d14425dac004d3e2b5d96ae60f40746843e747f /packages/snort | |
parent | 8ce4c10af01cf34b4ae00f5643b64d703a278abd (diff) | |
download | pfsense-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
Diffstat (limited to 'packages/snort')
-rw-r--r-- | packages/snort/snort_rules.php | 2 |
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; } |