revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-19 03:50:54 +01:00
Code Issues Releases Wiki Activity

Fix GitHub OAuth Registration for users without a name

Browse source 
Summary:
Github allows you to have an account without a real name. The OAuth controller
actually handles this fine, mostly, except that it calls a bogus method. Also
there is some null vs empty string confusion.

Test Plan:
Deleted my name on Github and then registered for an account on Phabricator.

Reviewed By: tuomaspelkonen
Reviewers: jungejason, tuomaspelkonen, aran
CC: anjali, aran, tuomaspelkonen
Differential Revision: 247
This commit is contained in:
epriestley 2011-05-06 17:01:00 -07:00
parent 4e70c434fd
commit 870f4bfe73
3 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/applications/auth/controller/oauthregistration/default/PhabricatorOAuthDefaultRegistrationController.php
View file

@ -60,9 +60,9 @@ class PhabricatorOAuthDefaultRegistrationController
}
}
if ($user->getRealName() === null) {
if (!strlen($user->getRealName())) {
$user->setRealName($request->getStr('realname'));
if (!strlen($user->getStr('realname'))) {
if (!strlen($user->getRealName())) {
$e_realname = 'Required';
$errors[] = 'Real name is required.';
} else {

4
src/applications/auth/oauth/provider/github/PhabricatorOAuthProviderGithub.php
View file

@ -78,7 +78,7 @@ class PhabricatorOAuthProviderGithub extends PhabricatorOAuthProvider {
}
public function retrieveUserEmail() {
return $this->userData['email'];
return idx($this->userData, 'email');
}
public function retrieveUserAccountName() {
@ -103,7 +103,7 @@ class PhabricatorOAuthProviderGithub extends PhabricatorOAuthProvider {
}
public function retrieveUserRealName() {
return $this->userData['name'];
return idx($this->userData, 'name');
}
}

2
src/applications/auth/oauth/provider/github/__init__.php
View file

@ -9,5 +9,7 @@
phutil_require_module('phabricator', 'applications/auth/oauth/provider/base');
phutil_require_module('phabricator', 'infrastructure/env');
phutil_require_module('phutil', 'utils');
phutil_require_source('PhabricatorOAuthProviderGithub.php');