Merge pull request #253 from DasunThathsara/dasun

fix: admin deletion functionality for superadmin
DasunThathsara · Jan 31, 2024 · eccb6b4 · eccb6b4
2 parents 93f11af + 69bf03f
commit eccb6b4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/app/models/UserModel.php b/app/models/UserModel.php
@@ -401,7 +401,7 @@ public function getAdminCount(){
 
     // View all admins
     public function viewAdmins(){
-        $this->db->query('SELECT * FROM user WHERE userType = "admin"');
+        $this->db->query('SELECT * FROM user WHERE userType = "admin" AND status != 10');
 
         $results = $this->db->resultSet();
 
@@ -420,7 +420,7 @@ public function viewAdmin($admin_id){
 
     // Remove admin
     public function removeAdmin($admin_id){
-        $this->db->query('DELETE FROM user WHERE id = :id');
+        $this->db->query('UPDATE user SET status = 10 WHERE id = :id');
         $this->db->bind(':id', $admin_id);
 
         // Execute

diff --git a/app/views/superAdmin/admins.php b/app/views/superAdmin/admins.php
@@ -5,10 +5,7 @@
 <!--  SIDE NAVIGATION  -->
 <?php
 $section = 'lands';
-//require APPROOT.'/views/inc/components/sidenavbar.php';
-
-//print_r(session_id());
-session_destroy();
+require APPROOT.'/views/inc/components/sidenavbar.php';
 ?>
 
 <main class="page-container">