summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFlorian Pritz <bluewind@xinu.at>2017-04-17 13:21:29 +0200
committerFlorian Pritz <bluewind@xinu.at>2017-04-18 13:50:33 +0200
commit39d973c2988174e0c895d203b5dd7a7d3de8f1f9 (patch)
tree629c7bf06284f33f254bfee1a2dd9f31324e46ab
parent5e67488012527cc31777ce5d8835fdfd9547ba13 (diff)
Use === for some equality checks
Signed-off-by: Florian Pritz <bluewind@xinu.at>
-rw-r--r--application/controllers/user.php6
-rw-r--r--application/models/muser.php4
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?");
}