Commit 1e3b92e9 authored by Ad Schellevis's avatar Ad Schellevis

(legacy) refactor system_routes.php

parent 48acede9
...@@ -34,44 +34,6 @@ require_once("system.inc"); ...@@ -34,44 +34,6 @@ require_once("system.inc");
require_once("pfsense-utils.inc"); require_once("pfsense-utils.inc");
require_once("rrd.inc"); require_once("rrd.inc");
if (!is_array($config['staticroutes'])) {
$config['staticroutes'] = array();
}
if (!is_array($config['staticroutes']['route'])) {
$config['staticroutes']['route'] = array();
}
$a_routes = &$config['staticroutes']['route'];
$a_gateways = return_gateways_array(true, true, true);
if ($_POST) {
$pconfig = $_POST;
if ($_POST['apply']) {
$retval = 0;
if (file_exists('/tmp/.system_routes.apply')) {
$toapplylist = unserialize(file_get_contents('/tmp/.system_routes.apply'));
foreach ($toapplylist as $toapply) {
mwexec("{$toapply}");
}
@unlink('/tmp/.system_routes.apply');
}
$retval = system_routing_configure();
$retval |= filter_configure();
/* reconfigure our gateway monitor */
setup_gateways_monitor();
$savemsg = get_std_save_message();
if ($retval == 0) {
clear_subsystem_dirty('staticroutes');
}
}
}
function delete_static_route($id) function delete_static_route($id)
{ {
global $config, $a_routes; global $config, $a_routes;
...@@ -105,296 +67,271 @@ function delete_static_route($id) ...@@ -105,296 +67,271 @@ function delete_static_route($id)
unset($targets); unset($targets);
} }
if ($_GET['act'] == "del") { if (!isset($config['staticroutes']['route']) || !is_array($config['staticroutes']['route'])) {
if ($a_routes[$_GET['id']]) { $a_routes = array();
delete_static_route($_GET['id']); } else {
unset($a_routes[$_GET['id']]); $a_routes = &$config['staticroutes']['route'];
write_config();
header("Location: system_routes.php");
exit;
}
} }
if (isset($_POST['del_x'])) { if ($_SERVER['REQUEST_METHOD'] === 'POST') {
/* delete selected routes */ $pconfig = $_POST;
if (is_array($_POST['route']) && count($_POST['route'])) { if (isset($_POST['id']) && isset($a_routes[$_POST['id']])) {
foreach ($_POST['route'] as $routei) { $id = $_POST['id'];
delete_static_route($routei);
unset($a_routes[$routei]);
}
write_config();
header("Location: system_routes.php");
exit;
} }
if (!empty($_POST['act'])) {
} elseif ($_GET['act'] == "toggle") { $act = $_POST['act'];
if ($a_routes[$_GET['id']]) {
if (isset($a_routes[$_GET['id']]['disabled'])) {
unset($a_routes[$_GET['id']]['disabled']);
} else { } else {
delete_static_route($_GET['id']); $act = null;
$a_routes[$_GET['id']]['disabled'] = true;
} }
write_config(); if (!empty($pconfig['apply'])) {
mark_subsystem_dirty('staticroutes'); // todo: remove this ugly hook
header("Location: system_routes.php"); if (file_exists('/tmp/.system_routes.apply')) {
exit; $toapplylist = unserialize(file_get_contents('/tmp/.system_routes.apply'));
} foreach ($toapplylist as $toapply) {
} else { mwexec("{$toapply}");
/* yuck - IE won't send value attributes for image buttons, while Mozilla does - so we use .x/.y to find move button clicks instead... */
unset($movebtn);
foreach ($_POST as $pn => $pd) {
if (preg_match("/move_(\d+)_x/", $pn, $matches)) {
$movebtn = $matches[1];
break;
} }
@unlink('/tmp/.system_routes.apply');
} }
/* move selected routes before this route */
if (isset($movebtn) && is_array($_POST['route']) && count($_POST['route'])) {
$a_routes_new = array();
/* copy all routes < $movebtn and not selected */ $retval = system_routing_configure();
for ($i = 0; $i < $movebtn; $i++) { $retval |= filter_configure();
if (!in_array($i, $_POST['route'])) { /* reconfigure our gateway monitor */
$a_routes_new[] = $a_routes[$i]; setup_gateways_monitor();
}
}
/* copy all selected routes */ if ($retval == 0) {
for ($i = 0; $i < count($a_routes); $i++) { clear_subsystem_dirty('staticroutes');
if ($i == $movebtn) {
continue;
} }
if (in_array($i, $_POST['route'])) { } elseif (isset($id) && $act == 'del') {
$a_routes_new[] = $a_routes[$i]; delete_static_route($id);
unset($a_routes[$id]);
write_config();
} elseif ($act == 'del_x' && isset($pconfig['route'])) {
/* delete selected routes */
if (is_array($pconfig['route'])) {
foreach ($_POST['route'] as $routei) {
delete_static_route($routei);
unset($a_routes[$routei]);
} }
write_config();
} }
} elseif (isset($id) && $act == "toggle") {
/* copy $movebtn route */ if (isset($a_routes[$id]['disabled'])) {
if ($movebtn < count($a_routes)) { unset($a_routes[$id]['disabled']);
$a_routes_new[] = $a_routes[$movebtn]; } else {
delete_static_route($id);
$a_routes[$id]['disabled'] = true;
} }
/* copy all routes > $movebtn and not selected */ write_config();
for ($i = $movebtn+1; $i < count($a_routes); $i++) { mark_subsystem_dirty('staticroutes');
if (!in_array($i, $_POST['route'])) { } elseif ( $act == 'move' && isset($pconfig['route']) && count($pconfig['route']) > 0) {
$a_routes_new[] = $a_routes[$i]; // move selected rules
} if (!isset($id)) {
} // if rule not set/found, move to end
if (count($a_routes_new) > 0) { $id = count($a_routes);
$a_routes = $a_routes_new;
} }
$a_routes = legacy_move_config_list_items($a_routes, $id, $pconfig['route']);
if (write_config()) { if (write_config()) {
mark_subsystem_dirty('staticroutes'); mark_subsystem_dirty('staticroutes');
} }
}
header("Location: system_routes.php"); header("Location: system_routes.php");
exit; exit;
}
} }
$a_gateways = return_gateways_array(true, true, true);
legacy_html_escape_form_data($a_routes);
legacy_html_escape_form_data($a_gateways);
$pgtitle = array(gettext('System'), gettext('Routes')); $pgtitle = array(gettext('System'), gettext('Routes'));
$shortcut_section = "routing"; $shortcut_section = "routing";
include("head.inc");
$main_buttons = array( $main_buttons = array(
array('label'=> gettext('Add route'), 'href'=>'system_routes_edit.php'), array('label'=> gettext('Add route'), 'href'=>'system_routes_edit.php'),
); );
include("head.inc");
?> ?>
<script type="text/javascript">
$( document ).ready(function() {
// link remove route
$(".act-del-route").click(function(event){
var id = $(this).data('id');
event.preventDefault();
BootstrapDialog.show({
type:BootstrapDialog.TYPE_INFO,
title: "<?= gettext("Route");?>",
message: '<?=gettext("Do you really want to delete this route?");?>',
buttons: [{
label: "<?= gettext("No");?>",
action: function(dialogRef) {
dialogRef.close();
}}, {
label: "<?= gettext("Yes");?>",
action: function(dialogRef) {
$("#id").val(id);
$("#act").val("del");
$("#iform").submit();
}
}]
});
});
// link remove list of routes
$("#del_x").click(function(event){
event.preventDefault();
BootstrapDialog.show({
type:BootstrapDialog.TYPE_INFO,
title: "<?= gettext("Route");?>",
message: '<?=gettext("Do you really want to delete the selected routes?");?>',
buttons: [{
label: "<?= gettext("No");?>",
action: function(dialogRef) {
dialogRef.close();
}}, {
label: "<?= gettext("Yes");?>",
action: function(dialogRef) {
$("#id").val(id);
$("#act").val("del_x");
$("#iform").submit();
}
}]
});
});
// link toggle buttons
$(".act_toggle").click(function(event){
event.preventDefault();
var id = $(this).data("id");
$("#id").val(id);
$("#act").val("toggle");
$("#iform").submit();
});
// link move buttons
$(".act_move").click(function(){
var id = $(this).data("id");
$("#id").val(id);
$("#act").val("move");
$("#iform").submit();
});
});
</script>
<body> <body>
<?php include("fbegin.inc"); ?> <?php include("fbegin.inc"); ?>
<script type="text/javascript" src="/javascript/row_toggle.js"></script>
<section class="page-content-main"> <section class="page-content-main">
<div class="container-fluid"> <div class="container-fluid">
<div class="row"> <div class="row">
<?php
<?php if (isset($savemsg)) { if (is_subsystem_dirty('staticroutes')):?><p>
print_info_box($savemsg);
} ?>
<?php if (is_subsystem_dirty('staticroutes')) :
?><p>
<?php print_info_box_apply(sprintf(gettext("The static route configuration has been changed.%sYou must apply the changes in order for them to take effect."), "<br />"));?><br /></p> <?php print_info_box_apply(sprintf(gettext("The static route configuration has been changed.%sYou must apply the changes in order for them to take effect."), "<br />"));?><br /></p>
<?php <?php
endif; ?> endif; ?>
<section class="col-xs-12"> <section class="col-xs-12">
<div class="tab-content content-box col-xs-12"> <div class="tab-content content-box col-xs-12">
<form action="system_routes.php" method="post" name="iform" id="iform"> <form action="system_routes.php" method="post" name="iform" id="iform">
<input type="hidden" id="act" name="act" value="" />
<input type="hidden" id="id" name="id" value="" />
<div class="table-responsive"> <div class="table-responsive">
<table class="table table-striped table-sort"> <table class="table table-striped table-sort">
<tr>
<tr id="frheader"> <td></td>
<td width="2%" class="list">&nbsp;</td> <td></td>
<td width="2%" class="list">&nbsp;</td> <td><?=gettext("Network");?></td>
<td width="22%" class="listhdrr"><?=gettext("Network");?></td> <td><?=gettext("Gateway");?></td>
<td width="20%" class="listhdrr"><?=gettext("Gateway");?></td> <td><?=gettext("Interface");?></td>
<td width="15%" class="listhdrr"><?=gettext("Interface");?></td> <td><?=gettext("Description");?></td>
<td width="29%" class="listhdr"><?=gettext("Description");?></td> <td></td>
<td width="10%" class="list">
</td>
</tr> </tr>
<?php $i = 0; foreach ($a_routes as $route) : <?php
?> $i = 0;
<tr valign="top" id="fr<?=$i;?>"> foreach ($a_routes as $route) :?>
<?php <tr class="<?=isset($route['disabled']) ? "text-muted" : "";?>">
$iconfn = "glyphicon glyphicon-play text-success"; <td>
if (isset($route['disabled'])) { <input type="checkbox" name="route[]" value="<?=$i;?>"/>
$textss = "<span class=\"gray\">";
$textse = "</span>";
$iconfn .= "glyphicon glyphicon-play text-muted";
} else {
$textss = $textse = "";
}
?>
<td class="listt">
<input type="checkbox" id="frc<?=$i;
?>" name="route[]" value="<?=$i;
?>" onclick="fr_bgcolor('<?=$i;?>')" />
</td> </td>
<td class="listt" align="center"> <td>
<a href="?act=toggle&amp;id=<?=$i;?>"> <a href="#" class="act_toggle" data-id="<?=$i;?>">
<span class="<?=$iconfn; <span class="glyphicon glyphicon-play <?=isset($route['disabled']) ? "text-muted" : "text-success" ;?>"
?>" title="<?=gettext("click to toggle enabled/disabled status");?>" alt="icon"></span> title="<?=gettext("click to toggle enabled/disabled status");?>" alt="icon">
</span>
</a> </a>
</td> </td>
<td class="listlr" onclick="fr_toggle(<?=$i; <td>
?>)" id="frd<?=$i; <?=strtolower($route['network']);?>
?>" ondblclick="document.location='system_routes_edit.php?id=<?=$i;?>';">
<?=$textss;
?><?=strtolower($route['network']);
?><?=$textse;?>
</td>
<td class="listr" onclick="fr_toggle(<?=$i;
?>)" id="frd<?=$i;
?>" ondblclick="document.location='system_routes_edit.php?id=<?=$i;?>';">
<?=$textss;?>
<?php
echo htmlentities($a_gateways[$route['gateway']]['name']) . " - " . htmlentities($a_gateways[$route['gateway']]['gateway']);
?>
<?=$textse;?>
</td> </td>
<td class="listr" onclick="fr_toggle(<?=$i; <td>
?>)" id="frd<?=$i; <?=$a_gateways[$route['gateway']]['name'] . " - " . $a_gateways[$route['gateway']]['gateway'];?>
?>" ondblclick="document.location='system_routes_edit.php?id=<?=$i;?>';">
<?=$textss;?>
<?php
echo convert_friendly_interface_to_friendly_descr($a_gateways[$route['gateway']]['friendlyiface']) . " ";
?>
<?=$textse;?>
</td> </td>
<td class="listbg" onclick="fr_toggle(<?=$i; <td>
?>)" ondblclick="document.location='system_routes_edit.php?id=<?=$i;?>';"> <?=convert_friendly_interface_to_friendly_descr($a_gateways[$route['gateway']]['friendlyiface']);?>
<?=$textss;
?><?=htmlspecialchars($route['descr']);
?>&nbsp;<?=$textse;?>
</td> </td>
<td class="list nowrap" valign="middle"> <td>
<table border="0" cellspacing="0" cellpadding="1" summary="move"> <?=$route['descr'];?>
<tr>
<td align="center" valign="middle">
<button onmouseover="fr_insline(<?=$i;
?>, true)" onmouseout="fr_insline(<?=$i;
?>, false)" name="move_<?=$i;?>"
class="btn btn-default btn-xs"
title="<?=gettext("move selected rules before this rule");?>"
type="submit"><span class="glyphicon glyphicon-arrow-left"></span></button>
</td> </td>
<td> <td>
<a data-id="<?=$i;?>" data-toggle="tooltip" data-placement="left" title="<?=gettext("move selected rules before this rule");?>" class="act_move btn btn-default btn-xs">
<span class="glyphicon glyphicon-arrow-left"></span>
</a>
<a class="btn btn-default btn-xs" href="system_routes_edit.php?id=<?=$i;?>"> <a class="btn btn-default btn-xs" href="system_routes_edit.php?id=<?=$i;?>">
<span class="glyphicon glyphicon-pencil" title="<?=gettext("edit rule");?>" alt="edit"></span> <span class="glyphicon glyphicon-pencil" title="<?=gettext("edit rule");?>" alt="edit"></span>
</a> </a>
</td> <button type="button" class="btn btn-default btn-xs act-del-route"
</tr> data-id="<?=$i?>" title="<?=gettext("delete route");?>" data-toggle="tooltip"
<tr> data-placement="left" ><span class="glyphicon glyphicon-remove"></span>
<td align="center" valign="middle"> </button>
<a class="btn btn-default btn-xs" href="system_routes.php?act=del&amp;id=<?=$i;
?>" onclick="return confirm('<?=gettext("Do you really want to delete this rule?");?>')">
<span class="glyphicon glyphicon-remove" title="<?=gettext("delete rule");?>" alt="delete"></span>
</a>
</td>
<td>
<a class="btn btn-default btn-xs" href="system_routes_edit.php?dup=<?=$i;?>"> <a class="btn btn-default btn-xs" href="system_routes_edit.php?dup=<?=$i;?>">
<span class="glyphicon glyphicon-plus" title="<?=gettext("add a new rule based on this one");?>" alt="duplicate"></span> <span class="glyphicon glyphicon-plus" title="<?=gettext("add a new rule based on this one");?>" alt="duplicate"></span>
</a> </a>
</td> </td>
</tr> </tr>
</table>
</td>
</tr>
<?php $i++;
endforeach; ?> <?php
<tr> $i++;
<td class="list" colspan="6"></td> endforeach; ?>
<td class="list nowrap" valign="middle">
<table border="0" cellspacing="0" cellpadding="1" summary="edit">
<tr> <tr>
<td colspan="6"></td>
<td> <td>
<?php <?php
if ($i == 0) : if ($i == 0) :?>
?>
<span class="glyphicon glyphicon-arrow-left text-muted" <span class="glyphicon glyphicon-arrow-left text-muted"
title="<?=gettext("move selected routes to end");?>" alt="move" /> title="<?=gettext("move selected routes to end");?>" alt="move" />
<?php <?php
else : else :?>
?> <button type="submit" data-id="<?=$i;?>" data-toggle="tooltip" data-placement="left" title="<?=gettext("move selected rules to end");?>" class="act_move btn btn-default btn-xs">
<button name="move_<?=$i;?>" type="submit" class="btn btn-default btn-xs" <span class="glyphicon glyphicon-arrow-left"></span>
title="<?=gettext("move selected routes to end");?>"><span class="glyphicon glyphicon-arrow-left"></span></button> </button>
<?php <?php
endif; endif;?>
?>
</td>
<td>
<a href="system_routes_edit.php" class="btn btn-default btn-xs"><span class="glyphicon glyphicon-plus"></span></a> <a href="system_routes_edit.php" class="btn btn-default btn-xs"><span class="glyphicon glyphicon-plus"></span></a>
</td> <?php
</tr> if ($i == 0) :?>
<tr>
<td>
<?php
if ($i == 0) :
?>
<span class="btn btn-default btn-xs"> <span class="btn btn-default btn-xs">
<span class="glyphicon glyphicon-remove text-muted"></span> <span class="glyphicon glyphicon-remove text-muted"></span>
</span> </span>
<?php <?php
else : else :?>
?> <button id="del_x" title="<?=gettext("delete selected routes");?>" class="btn btn-default btn-xs"><span class="glyphicon glyphicon-remove"></span></button>
<button name="del_x" type="submit" title="<?=gettext("delete selected routes");?>" <?php
onclick="return confirm('<?=gettext("Do you really want to delete the selected routes?");?>')" class="btn btn-default btn-xs"><span class="glyphicon glyphicon-remove"></span></button> endif;?>
<?php
endif;
?>
</td> </td>
</tr> </tr>
</table> <tr>
<td colspan="7">
<strong><?= gettext('Note:') ?></strong><br/>
<?=gettext("Do not enter static routes for networks assigned on any interface of this firewall. Static routes are only used for networks reachable via a different router, and not reachable via your default gateway.");?>
</td> </td>
</tr> </tr>
<tr><td colspan="7">
<?= gettext('Note:') ?>
<?=gettext("Do not enter static routes for networks assigned on any interface of this firewall. Static routes are only used for networks reachable via a different router, and not reachable via your default gateway.");?>
</td></tr>
</table> </table>
</div> </div>
</form> </form>
</div> </div>
</section> </section>
</div> </div>
</div> </div>
</section> </section>
<?php include("foot.inc"); <?php include("foot.inc");
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment