summaryrefslogtreecommitdiff
path: root/actions/register.php
diff options
context:
space:
mode:
authorBrion Vibber <brion@pobox.com>2010-02-04 13:08:34 -0800
committerBrion Vibber <brion@pobox.com>2010-02-04 13:08:34 -0800
commit239b88025ef1368bb871871ee903d6b078493f76 (patch)
treec21a4650f9a2ee9fa63b7289b4e3874c2e1ee5ff /actions/register.php
parent5a1cbdc6f1e32be7b8430924a1125422d8457584 (diff)
Should fix spurious 'nickname taken' and 'email taken' errors on registration. Form's checks for existing nicks & emails would incorrectly return true on the second lookup due to bad interaction with negative caching.
(was checking $obj !== false but we return null now on negative cache hits, with false for cache misses)
Diffstat (limited to 'actions/register.php')
-rw-r--r--actions/register.php4
1 files changed, 2 insertions, 2 deletions
diff --git a/actions/register.php b/actions/register.php
index 698137346..ccab76cf0 100644
--- a/actions/register.php
+++ b/actions/register.php
@@ -280,7 +280,7 @@ class RegisterAction extends Action
function nicknameExists($nickname)
{
$user = User::staticGet('nickname', $nickname);
- return ($user !== false);
+ return is_object($user);
}
/**
@@ -300,7 +300,7 @@ class RegisterAction extends Action
return false;
}
$user = User::staticGet('email', $email);
- return ($user !== false);
+ return is_object($user);
}
// overrrided to add entry-title class