From 8dce7a9214eeb4fd334f7749f706bd1db9649245 Mon Sep 17 00:00:00 2001
From: Steve Beaver <sbeaver@netgate.com>
Date: Fri, 10 Feb 2017 12:02:03 -0500
Subject: GET/POST conversion interfaces_wireless*

---
 src/usr/local/www/interfaces_wireless.php      | 14 +++++++-------
 src/usr/local/www/interfaces_wireless_edit.php |  7 ++-----
 2 files changed, 9 insertions(+), 12 deletions(-)

diff --git a/src/usr/local/www/interfaces_wireless.php b/src/usr/local/www/interfaces_wireless.php
index 304eca2..70cb05a 100644
--- a/src/usr/local/www/interfaces_wireless.php
+++ b/src/usr/local/www/interfaces_wireless.php
@@ -52,13 +52,13 @@ function clone_inuse($num) {
 	return false;
 }
 
-if ($_GET['act'] == "del") {
+if ($_POST['act'] == "del") {
 	/* check if still in use */
-	if (clone_inuse($_GET['id'])) {
+	if (clone_inuse($_POST['id'])) {
 		$input_errors[] = gettext("This wireless clone cannot be deleted because it is assigned as an interface.");
 	} else {
-		pfSense_interface_destroy($a_clones[$_GET['id']]['cloneif']);
-		unset($a_clones[$_GET['id']]);
+		pfSense_interface_destroy($a_clones[$_POST['id']]['cloneif']);
+		unset($a_clones[$_POST['id']]);
 
 		write_config();
 
@@ -120,8 +120,8 @@ foreach ($a_clones as $clone) {
 							<?=htmlspecialchars($clone['descr'])?>
 						</td>
 						<td>
-							<a class="fa fa-pencil"	title="<?=gettext('Edit WiFi interface')?>"	href="interfaces_wireless_edit.php?id=<?=$i?>"></a>
-							<a class="fa fa-trash"	title="<?=gettext('Delete WiFi interface')?>"	href="interfaces_wireless.php?act=del&amp;id=<?=$i?>"></a>
+							<a class="fa fa-pencil"	title="<?=gettext('Edit WiFi interface')?>"	href="interfaces_wireless_edit.php?id=<?=$i?>" usepost></a>
+							<a class="fa fa-trash"	title="<?=gettext('Delete WiFi interface')?>"	href="interfaces_wireless.php?act=del&amp;id=<?=$i?>" usepost></a>
 						</td>
 					</tr>
 <?php
@@ -135,7 +135,7 @@ foreach ($a_clones as $clone) {
 </div>
 
 <nav class="action-buttons">
-	<a href="interfaces_wireless_edit.php" class="btn btn-success btn-sm">
+	<a href="interfaces_wireless_edit.php" class="btn btn-success btn-sm" usepost>
 		<i class="fa fa-plus icon-embed-btn"></i>
 		<?=gettext("Add")?></a>
 </nav>
diff --git a/src/usr/local/www/interfaces_wireless_edit.php b/src/usr/local/www/interfaces_wireless_edit.php
index a37e817..d6d4172 100644
--- a/src/usr/local/www/interfaces_wireless_edit.php
+++ b/src/usr/local/www/interfaces_wireless_edit.php
@@ -58,10 +58,7 @@ function clone_compare($a, $b) {
 
 $portlist = get_interface_list();
 
-if (is_numericint($_GET['id'])) {
-	$id = $_GET['id'];
-}
-if (isset($_POST['id']) && is_numericint($_POST['id'])) {
+if (is_numericint($_POST['id'])) {
 	$id = $_POST['id'];
 }
 
@@ -72,7 +69,7 @@ if (isset($id) && $a_clones[$id]) {
 	$pconfig['descr'] = $a_clones[$id]['descr'];
 }
 
-if ($_POST) {
+if ($_POST['save']) {
 	unset($input_errors);
 	$pconfig = $_POST;
 
-- 
cgit v1.1