From caf43f2259dd33fce80a9691118863aa83cd461b Mon Sep 17 00:00:00 2001
From: Peter Goodhall <peter@magicbug.co.uk>
Date: Fri, 21 Jun 2019 21:22:22 +0100
Subject: [PATCH 1/2] Added function for deleting API keys

Added a function to delete API keys in the API model, this will be exposed via the api.php controller with a frontend component
---
 application/models/Api_model.php | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/application/models/Api_model.php b/application/models/Api_model.php
index 045c746d..08946a21 100644
--- a/application/models/Api_model.php
+++ b/application/models/Api_model.php
@@ -19,6 +19,10 @@ class API_Model extends CI_Model {
     	return $this->db->get('api');
     }
 
+    function delete_key($key) {
+    	$this->db->where('id', $key);
+		$this->db->delete('api');
+    }
     // Generate API Key
     function generate_key($rights) {
     	

From d05717c1e6638564a5091d7c067a04d52c9cd007 Mon Sep 17 00:00:00 2001
From: Peter Goodhall <peter@magicbug.co.uk>
Date: Fri, 21 Jun 2019 21:33:21 +0100
Subject: [PATCH 2/2] You can now delete API keys via /api/help

---
 application/controllers/Api.php  | 14 ++++++++++++++
 application/models/Api_model.php |  2 +-
 application/views/api/help.php   | 12 ++++++++----
 3 files changed, 23 insertions(+), 5 deletions(-)

diff --git a/application/controllers/Api.php b/application/controllers/Api.php
index 126bdfef..c7592110 100644
--- a/application/controllers/Api.php
+++ b/application/controllers/Api.php
@@ -69,6 +69,20 @@ class API extends CI_Controller {
 		redirect('api/help');
 	}
 
+	function delete($key) {
+		$this->load->model('user_model');
+		if(!$this->user_model->authorize(99)) { $this->session->set_flashdata('notice', 'You\'re not allowed to do that!'); redirect('dashboard'); }
+
+
+		$this->load->model('api_model');
+
+		$this->api_model->delete_key($key);
+
+		$this->session->set_flashdata('notice', 'API Key <b>'.$key."</b> has been deleted");
+
+		redirect('api/help');
+	}
+
 	// Example of authing
 	function auth($key) {
 		$this->load->model('api_model');
diff --git a/application/models/Api_model.php b/application/models/Api_model.php
index 08946a21..4c41b8b2 100644
--- a/application/models/Api_model.php
+++ b/application/models/Api_model.php
@@ -20,7 +20,7 @@ class API_Model extends CI_Model {
     }
 
     function delete_key($key) {
-    	$this->db->where('id', $key);
+    	$this->db->where('key', $key);
 		$this->db->delete('api');
     }
     // Generate API Key
diff --git a/application/views/api/help.php b/application/views/api/help.php
index c49c3604..52a6cbde 100644
--- a/application/views/api/help.php
+++ b/application/views/api/help.php
@@ -2,9 +2,9 @@
 
 <br>
 <?php if($this->session->flashdata('notice')) { ?>
-	<div id="message" >
-    	<?php echo $this->session->flashdata('notice'); ?>
-	</div>
+<div class="alert alert-success" role="alert">
+	<?php echo $this->session->flashdata('notice'); ?>
+</div>
 <?php } ?>
 
 <div class="card">
@@ -43,7 +43,11 @@
 						?>
 
 					</td>
-					<td><span class="badge badge-pill badge-light"><?php echo ucfirst($row->status); ?></span>  <a href="<?php echo site_url('api/auth/'.$row->key); ?>" target="_blank" class="btn btn-primary btn-sm">Test</td>
+					<td><span class="badge badge-pill badge-light"><?php echo ucfirst($row->status); ?></span>
+						<a href="<?php echo site_url('api/auth/'.$row->key); ?>" target="_blank" class="btn btn-primary btn-sm">Test</a>
+
+						<a href="<?php echo site_url('api/delete/'.$row->key); ?>" class="btn btn-danger btn-sm" onclick="return confirm('Are you sure you want delete API Key <?php echo $row->key; ?>?');">Delete</a>
+					</td>
 				</tr>
 
 			<?php } ?>