diff options
author | NOYB <Al_Stu@Frontier.com> | 2016-02-12 00:03:50 -0800 |
---|---|---|
committer | NOYB <Al_Stu@Frontier.com> | 2016-02-12 00:03:50 -0800 |
commit | a4f41878a579bfdd1e3234efe93a144149f2326d (patch) | |
tree | b6ecba2c46606564f78693000fc1fc022a1c448a /src/usr/local/www/firewall_nat.php | |
parent | 99a05a0a051c7722f800b5ed0f6560ec8d5fbda0 (diff) | |
download | pfsense-a4f41878a579bfdd1e3234efe93a144149f2326d.zip pfsense-a4f41878a579bfdd1e3234efe93a144149f2326d.tar.gz |
Firewall / Rules - Consolidate Move Separators Function
Consolidate the moving of separators for add/copy/dup/delete into a generic function.
Diffstat (limited to 'src/usr/local/www/firewall_nat.php')
-rw-r--r-- | src/usr/local/www/firewall_nat.php | 19 |
1 files changed, 6 insertions, 13 deletions
diff --git a/src/usr/local/www/firewall_nat.php b/src/usr/local/www/firewall_nat.php index 25e95c8..bc82698 100644 --- a/src/usr/local/www/firewall_nat.php +++ b/src/usr/local/www/firewall_nat.php @@ -142,13 +142,9 @@ if ($_GET['act'] == "del") { // Update the separators $a_separators = &$config['nat']['separator']; - - for ($idx=0; isset($a_separators['sep' . $idx]); $idx++ ) { - $seprow = substr($a_separators['sep' . $idx]['row']['0'], 2); - if ($seprow > $_GET['id']) { - $a_separators['sep' . $idx]['row']['0'] = 'fr' . ($seprow - 1); - } - } + $ridx = $_GET['id']; + $mvnrows = -1; + move_separators($a_separators, $ridx, $mvnrows); if (write_config()) { mark_subsystem_dirty('natconf'); @@ -180,12 +176,9 @@ if (isset($_POST['del_x'])) { unset($a_nat[$rulei]); // Update the separators - for ($idx=0; isset($a_separators['sep' . $idx]); $idx++ ) { - $seprow = substr($a_separators['sep' . $idx]['row']['0'], 2); - if ($seprow > $rulei) { - $a_separators['sep' . $idx]['row']['0'] = 'fr' . ($seprow - 1); - } - } + $ridx = $rulei; + $mvnrows = -1; + move_separators($a_separators, $ridx, $mvnrows); } if (write_config()) { |