diff options
author | Florian Pritz <bluewind@xinu.at> | 2017-04-17 13:21:29 +0200 |
---|---|---|
committer | Florian Pritz <bluewind@xinu.at> | 2017-04-18 13:50:33 +0200 |
commit | 39d973c2988174e0c895d203b5dd7a7d3de8f1f9 (patch) | |
tree | 629c7bf06284f33f254bfee1a2dd9f31324e46ab | |
parent | 5e67488012527cc31777ce5d8835fdfd9547ba13 (diff) |
Use === for some equality checks
Signed-off-by: Florian Pritz <bluewind@xinu.at>
-rw-r--r-- | application/controllers/user.php | 6 | ||||
-rw-r--r-- | application/models/muser.php | 4 |
2 files changed, 5 insertions, 5 deletions
diff --git a/application/controllers/user.php b/application/controllers/user.php index 71f83b527..d87b544c7 100644 --- a/application/controllers/user.php +++ b/application/controllers/user.php @@ -206,7 +206,7 @@ class User extends MY_Controller { $error[]= "Invalid email."; } - if (!$password || $password != $password_confirm) { + if (!$password || $password !== $password_confirm) { $error[]= "No password or passwords don't match."; } @@ -385,7 +385,7 @@ class User extends MY_Controller { $password = $this->input->post("password"); $password_confirm = $this->input->post("password_confirm"); - if (!$password || $password != $password_confirm) { + if (!$password || $password !== $password_confirm) { $error[]= "No password or passwords don't match."; } @@ -620,7 +620,7 @@ class User extends MY_Controller { $this->data["password"] = $password; if ($process !== false) { - if (!$password || $password != $password_confirm) { + if (!$password || $password !== $password_confirm) { $error[]= "No password or passwords don't match."; } else { $this->data["hash"] = $this->muser->hash_password($password); diff --git a/application/models/muser.php b/application/models/muser.php index 9866f0c86..1ee6c259a 100644 --- a/application/models/muser.php +++ b/application/models/muser.php @@ -55,7 +55,7 @@ class Muser extends CI_Model { function logged_in() { if ($this->has_session()) { - return $this->session->userdata('logged_in') == true; + return $this->session->userdata('logged_in') === true; } return false; @@ -303,7 +303,7 @@ class Muser extends CI_Model { ->where('action', $action) ->get()->row_array(); - if (!isset($query["key"]) || $key != $query["key"]) { + if (!isset($query["key"]) || $key !== $query["key"]) { throw new \exceptions\UserInputException("user/get_action/invalid-action", "Invalid action key. Has the key been used already?"); } |