Fixed 802.1p duplicating values for vlanprio and vlanprioset

For real this time. Friggin' github.

1 files changed, 13 insertions, 1 deletions

diff --git a/usr/local/www/firewall_rules_edit.php b/usr/local/www/firewall_rules_edit.php
index 7acef74..77808e4 100755
--- a/usr/local/www/firewall_rules_edit.php
+++ b/usr/local/www/firewall_rules_edit.php
@@ -1323,13 +1323,25 @@ $i--): ?>
 						else
 							$opts .= "<option value=\"{$vprio}\" {$selected}>" . strtoupper($vprio) . "</option>\n";
 					}
+
+					$optsset = "";
+					foreach($vlanprio as $vprioset) {
+						if ($vprioset == $pconfig['vlanprioset'])
+							$selected = " selected=\"selected\"";
+						else
+							$selected = "";
+						if ($vprioset == "none")
+							$optsset .= "<option value=\"\">{$vprioset}</option>\n";
+						else
+							$optsset .= "<option value=\"{$vprioset}\" {$selected}>" . strtoupper($vprioset) . "</option>\n";
+					}
 ?>
 					<select name='vlanprio'>
 					<?php echo $opts; ?>
 					</select>
 					<p><?=gettext("Choose 802.1p priority to match on");?></p>
 					<select name='vlanprioset'>
-					<?php echo $opts; ?>
+					<?php echo $optsset; ?>
 					</select>
 					<p><?=gettext("Choose 802.1p priority to apply");?></p>
 				</div>