Add delete confirmation

application/controllers/users.php

@@ -156,12 +156,14 @@ class Users extends MY_Controller {
 	 * @param integer $id
 	 */
 	public function delete($id = '') {
-		if (!is_array($this->user->getUserByID())) {
-			show_404();
-		}
+		$user = $this->user->getUserByID($id);
 
-		$this->user->delete($id);
+		if (!isset($user) || !is_array($user) || !isset($user['id'])) {
+			show_404();
+		} else {
+			$this->user->delete($user['id']);
 			$this->messages->add('The selected user was deleted', 'success');
 			redirect('users', 200);
 		}
 	}
+}

application/views/admin/users/index.php

@@ -22,7 +22,7 @@ foreach ($users as $user):
 				<tr>
 					<td><?=$user['username'];?></td>
 					<td><?=$user['firstname'];?> <?=$user['lastname'];?></td>
-					<td><?=anchor('users/edit/' . $user['id'], lang('user_edit'));?> | <?=anchor('users/delete/' . $user['id'], lang('user_delete'));?></td>
+					<td><?=anchor('users/edit/' . $user['id'], lang('user_edit'));?> | <a href="javascript:deleteConfirm('<?=site_url('users/delete/' . $user['id']);?>');" title="User"><?=lang('user_delete');?></a></td>
 				</tr>
 <?php
 endforeach;

assets/js/scattport.js

@@ -1,5 +1,5 @@
 /**
- *
+ * Gets the queued notifications.
  */
 function getNotifications() {
 	$.get(SITE_URL + 'ajax/get_notifications', function(data) {
@@ -9,6 +9,19 @@ function getNotifications() {
 	});
 }
 
+/**
+ * Asks the user if he is sure to delete an item.
+ *
+ * @param url
+ */
+function deleteConfirm(url) {
+	var answer = confirm('Are you sure?');
+
+	if (answer) {
+		window.location = url;
+	}
+}
+
 /**
  * Saves the changes done by in-place edit.
  *