Merge pull request #18 from bitinn/master

github user might not have a display name
1.0
Phil Sturgeon 2013-08-26 20:27:20 -07:00
commit 53c79b51fa
1 changed files with 2 additions and 2 deletions

View File

@ -26,7 +26,7 @@ class Github extends IdentityProvider
$user = new User; $user = new User;
$user->uid = $response->id; $user->uid = $response->id;
$user->nickname = $response->login; $user->nickname = $response->login;
$user->name = $response->name; $user->name = isset($response->name) ? $response->name : null;
$user->email = isset($response->email) ? $response->email : null; $user->email = isset($response->email) ? $response->email : null;
$user->urls = array( $user->urls = array(
'GitHub' => 'http://github.com/'.$user->login, 'GitHub' => 'http://github.com/'.$user->login,
@ -35,4 +35,4 @@ class Github extends IdentityProvider
return $user; return $user;
} }
} }