Project

General

Profile

« Previous | Next » 

Revision 9411d6ab

Added by timdufrane over 12 years ago

Fixed 802.1p duplicating values for vlanprio and vlanprioset

For real this time. Friggin' github.

View differences:

usr/local/www/firewall_rules_edit.php
1323 1323
						else
1324 1324
							$opts .= "<option value=\"{$vprio}\" {$selected}>" . strtoupper($vprio) . "</option>\n";
1325 1325
					}
1326

  
1327
					$optsset = "";
1328
					foreach($vlanprio as $vprioset) {
1329
						if ($vprioset == $pconfig['vlanprioset'])
1330
							$selected = " selected=\"selected\"";
1331
						else
1332
							$selected = "";
1333
						if ($vprioset == "none")
1334
							$optsset .= "<option value=\"\">{$vprioset}</option>\n";
1335
						else
1336
							$optsset .= "<option value=\"{$vprioset}\" {$selected}>" . strtoupper($vprioset) . "</option>\n";
1337
					}
1326 1338
?>
1327 1339
					<select name='vlanprio'>
1328 1340
					<?php echo $opts; ?>
1329 1341
					</select>
1330 1342
					<p><?=gettext("Choose 802.1p priority to match on");?></p>
1331 1343
					<select name='vlanprioset'>
1332
					<?php echo $opts; ?>
1344
					<?php echo $optsset; ?>
1333 1345
					</select>
1334 1346
					<p><?=gettext("Choose 802.1p priority to apply");?></p>
1335 1347
				</div>

Also available in: Unified diff