From fd6e3f34954e641fc444b865f5ec862dbc02b83f Mon Sep 17 00:00:00 2001 From: REJack Date: Sat, 14 May 2016 20:24:32 +0200 Subject: [PATCH] removed config var `update_last_login_attempt` changed `update_login_attempts()` (removed config var `update_last_login_attempt` & optimized if's) --- application/config/aauth.php | 1 - application/libraries/Aauth.php | 22 +++++----------------- 2 files changed, 5 insertions(+), 18 deletions(-) diff --git a/application/config/aauth.php b/application/config/aauth.php index 58668dd..fd9a1bb 100644 --- a/application/config/aauth.php +++ b/application/config/aauth.php @@ -115,7 +115,6 @@ $config_aauth["default"] = array( 'max_login_attempt' => 10, 'max_login_attempt_per_minutes' => 5, - 'update_last_login_attempt' => true, 'login_with_name' => false, diff --git a/application/libraries/Aauth.php b/application/libraries/Aauth.php index c1be968..f3d5456 100644 --- a/application/libraries/Aauth.php +++ b/application/libraries/Aauth.php @@ -635,30 +635,18 @@ class Aauth { $query = $this->aauth_db->get( $this->config_vars['users'] ); $row = $query->row(); - $data = array(); + $data['last_login_attempt'] = date("Y-m-d H:i:s"); - $minutes = $this->config_vars['max_login_attempt_per_minutes']; - if (strtotime($row->last_login_attempt) > strtotime("-$minutes minutes")) { + if (strtotime($row->last_login_attempt) > strtotime("-".$this->config_vars['max_login_attempt_per_minutes'])) { $data['login_attempts'] = $row->login_attempts + 1; - - if($this->config_vars['update_last_login_attempt']){ - $data['last_login_attempt'] = date("Y-m-d H:i:s"); - } - - $query = $this->aauth_db->where('id', $user_id); - $this->aauth_db->update($this->config_vars['users'], $data); - } else { - - $data['last_login_attempt'] = date("Y-m-d H:i:s"); $data['login_attempts'] = 1; - - $this->aauth_db->where('id', $user_id); - $this->aauth_db->update($this->config_vars['users'], $data); - } + $this->aauth_db->where('id', $user_id); + $this->aauth_db->update($this->config_vars['users'], $data); + if ( $data['login_attempts'] > $this->config_vars['max_login_attempt'] ) { return FALSE; } else {