Upd. Recording and displaying an event from wp_spbc_auth_logs

CleanTalk · May 28, 2024 · bea7907 · bea7907
1 parent 4284a0e
commit bea7907
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/inc/spbc-admin.php b/inc/spbc-admin.php
@@ -653,10 +653,11 @@ function spbc_admin_log_action()
 
     if (isset($user->ID) && $user->ID > 0) {
         $roles = (is_array($user->roles) && ! empty($user->roles) ? reset($user->roles) : null); // Takes only first role.
+        $action = spbc_parse_action_from_admin_page_uri(Server::get('REQUEST_URI'));
 
         $log_id = spbc_auth_log(array(
             'username' => $user->get('user_login'),
-            'event'    => 'view',
+            'event'    => isset($action["parsed_action"]) ? $action["parsed_action"] : 'view',
             'page'     => Server::get('REQUEST_URI'),
             'blog_id'  => get_current_blog_id(),
             'roles'    => $roles

diff --git a/inc/spbc-settings.php b/inc/spbc-settings.php
@@ -1929,6 +1929,8 @@ function spbc_field_2fa__roles()
 
 function spbc_field_security_logs__prepare_data(&$table)
 {
+    global $wpdb;
+
     if ($table->items_count) {
         foreach ($table->rows as $row) {
             $ips_c[] = $row->auth_ip;
@@ -1971,10 +1973,9 @@ function spbc_field_security_logs__prepare_data(&$table)
                 $page = "<a href='" . $url . "' target='_blank'>" . $url . "</a>";
             }
 
-
-            $action = spbc_parse_action_from_admin_page_uri($url, $row->id);
-            $action = !empty($action['parsed_action']) ? $action['parsed_action'] : '';
-            $is_add_time = !empty($action['add_time']) ? $action['add_time'] : false;
+            $action = !empty($row->event) ? $row->event : '';
+            $action_for_add_time = spbc_parse_action_from_admin_page_uri($url, $row->id);
+            $is_add_time = !empty($action_for_add_time['add_time']) ? $action_for_add_time['add_time'] : false;
 
             switch ($row->event) {
                 case 'view':