Merge pull request hybridauth#519 from guarly/master

[Twitter] Add email option to Twitter API
CityofOttawa · Aug 26, 2015 · 5dc0a5a · 5dc0a5a
2 parents 4e0d993 + b3da1fe
commit 5dc0a5a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/hybridauth/Hybrid/Providers/Twitter.php b/hybridauth/Hybrid/Providers/Twitter.php
@@ -109,7 +109,8 @@ function loginFinish() {
 	 * {@inheritdoc}
 	 */
 	function getUserProfile() {
-		$response = $this->api->get('account/verify_credentials.json');
+		$includeEmail = isset($this->config['includeEmail']) ? (bool) $this->config['includeEmail'] : false;
+		$response = $this->api->get('account/verify_credentials.json'. ($includeEmail ? '?include_email=true' : ''));
 
 		// check the last HTTP status code returned
 		if ($this->api->http_code != 200) {
@@ -129,6 +130,7 @@ function getUserProfile() {
 		$this->user->profile->profileURL = (property_exists($response, 'screen_name')) ? ("http://twitter.com/" . $response->screen_name) : "";
 		$this->user->profile->webSiteURL = (property_exists($response, 'url')) ? $response->url : "";
 		$this->user->profile->region = (property_exists($response, 'location')) ? $response->location : "";
+		if($includeEmail) $this->user->profile->email = (property_exists($response, 'email')) ? $response->email : "";
 
 		return $this->user->profile;
 	}

diff --git a/hybridauth/config.php b/hybridauth/config.php
@@ -35,7 +35,8 @@
 				),
 				"Twitter" => array(
 					"enabled" => true,
-					"keys" => array("key" => "", "secret" => "")
+					"keys" => array("key" => "", "secret" => ""),
+					"includeEmail" => false
 				),
 				// windows live
 				"Live" => array(