Revision 6c07db48
Added by Phil Davis about 10 years ago
usr/local/www/firewall_shaper.php | ||
---|---|---|
52 | 52 |
exit; |
53 | 53 |
} |
54 | 54 |
|
55 |
$pgtitle = array(gettext("Firewall"),gettext("Traffic Shaper")); |
|
55 |
$pgtitle = array(gettext("Firewall"), gettext("Traffic Shaper"));
|
|
56 | 56 |
$shortcut_section = "trafficshaper"; |
57 | 57 |
|
58 | 58 |
$shaperIFlist = get_configured_interface_with_descr(); |
... | ... | |
163 | 163 |
$q = new cbq_queue(); |
164 | 164 |
break; |
165 | 165 |
default: |
166 |
/* XXX: Happens when sched==NONE?! */
|
|
166 |
/* XXX: Happens when sched == NONE?! */
|
|
167 | 167 |
$q = new altq_root_queue(); |
168 | 168 |
break; |
169 | 169 |
} |
... | ... | |
194 | 194 |
if ($queue) { |
195 | 195 |
$queue->SetEnabled("on"); |
196 | 196 |
$output_form .= $queue->build_form(); |
197 |
if (write_config()) |
|
197 |
if (write_config()) {
|
|
198 | 198 |
mark_subsystem_dirty('shaper'); |
199 |
} |
|
199 | 200 |
} else { |
200 | 201 |
$input_errors[] = gettext("Queue not found!"); |
201 | 202 |
} |
... | ... | |
333 | 334 |
} |
334 | 335 |
read_altq_config(); |
335 | 336 |
$output_form .= $queue->build_form(); |
336 |
} else {
|
|
337 |
} else { |
|
337 | 338 |
$output_form .= $default_shaper_msg; |
338 | 339 |
$dontshow = true; |
339 | 340 |
} |
... | ... | |
405 | 406 |
$output_form .= "</table>"; |
406 | 407 |
} |
407 | 408 |
|
408 |
$output = "<table summary=\"output form\">";
|
|
409 |
$output = "<table summary=\"output form\">"; |
|
409 | 410 |
$output .= $output_form; |
410 | 411 |
|
411 | 412 |
//$pgtitle = "Firewall: Shaper: By Interface View"; |
Also available in: Unified diff
Code spacing
and other random stuff I noticed.
I think this finishes messing with code style. The codebase should match
the developer style guide closely enough that 99.9% of changes will not
feel the need to also massage the formatting.