diff --git a/modules/authfacebook/lib/Auth/Source/Facebook.php b/modules/authfacebook/lib/Auth/Source/Facebook.php index 78c90c5ebcbfa6bee288a6d1099919ae3f7cf99d..b922dd05aec61263d5148438f98608e74fc98ec8 100644 --- a/modules/authfacebook/lib/Auth/Source/Facebook.php +++ b/modules/authfacebook/lib/Auth/Source/Facebook.php @@ -2,6 +2,8 @@ namespace SimpleSAML\Module\authfacebook\Auth\Source; +use SimpleSAML\Module; + /** * Authenticate using Facebook Platform. * @@ -62,14 +64,15 @@ class Facebook extends \SimpleSAML\Auth\Source * @param array $info Information about this authentication source. * @param array $config Configuration. */ - public function __construct($info, $config) { + public function __construct($info, $config) + { assert(is_array($info)); assert(is_array($config)); // Call the parent constructor first, as required by the interface parent::__construct($info, $config); - $cfgParse = \SimpleSAML\Configuration::loadFromArray($config, 'authsources[' . var_export($this->authId, TRUE) . ']'); + $cfgParse = \SimpleSAML\Configuration::loadFromArray($config, 'authsources['.var_export($this->authId, true).']'); $this->api_key = $cfgParse->getString('api_key'); $this->secret = $cfgParse->getString('secret'); @@ -83,17 +86,18 @@ class Facebook extends \SimpleSAML\Auth\Source * * @param array &$state Information about the current authentication. */ - public function authenticate(&$state) { + public function authenticate(&$state) + { assert(is_array($state)); // We are going to need the authId in order to retrieve this authentication source later $state[self::AUTHID] = $this->authId; \SimpleSAML\Auth\State::saveState($state, self::STAGE_INIT); - $facebook = new \SimpleSAML\Module\authfacebook\Facebook(array('appId' => $this->api_key, 'secret' => $this->secret), $state); + $facebook = new Module\authfacebook\Facebook(array('appId' => $this->api_key, 'secret' => $this->secret), $state); $facebook->destroySession(); - $linkback = \SimpleSAML\Module::getModuleURL('authfacebook/linkback.php'); + $linkback = Module::getModuleURL('authfacebook/linkback.php'); $url = $facebook->getLoginUrl(array('redirect_uri' => $linkback, 'scope' => $this->req_perms)); \SimpleSAML\Auth\State::saveState($state, self::STAGE_INIT); @@ -101,15 +105,16 @@ class Facebook extends \SimpleSAML\Auth\Source } - public function finalStep(&$state) { + public function finalStep(&$state) + { assert(is_array($state)); - $facebook = new \SimpleSAML\Module\authfacebook\Facebook(array('appId' => $this->api_key, 'secret' => $this->secret), $state); + $facebook = new Module\authfacebook\Facebook(array('appId' => $this->api_key, 'secret' => $this->secret), $state); $uid = $facebook->getUser(); if (isset($uid) && $uid) { try { - $info = $facebook->api("/" . $uid . ($this->user_fields ? "?fields=" . $this->user_fields : "")); + $info = $facebook->api("/".$uid.($this->user_fields ? "?fields=".$this->user_fields : "")); } catch (\FacebookApiException $e) { throw new \SimpleSAML\Error\AuthSource($this->authId, 'Error getting user profile.', $e); } @@ -120,26 +125,25 @@ class Facebook extends \SimpleSAML\Auth\Source } $attributes = array(); - foreach($info AS $key => $value) { + foreach ($info as $key => $value) { if (is_string($value) && !empty($value)) { - $attributes['facebook.' . $key] = array((string)$value); + $attributes['facebook.'.$key] = array((string)$value); } } if (array_key_exists('third_party_id', $info)) { - $attributes['facebook_user'] = array($info['third_party_id'] . '@facebook.com'); + $attributes['facebook_user'] = array($info['third_party_id'].'@facebook.com'); } else { - $attributes['facebook_user'] = array($uid . '@facebook.com'); + $attributes['facebook_user'] = array($uid.'@facebook.com'); } $attributes['facebook_targetedID'] = array('http://facebook.com!' . $uid); $attributes['facebook_cn'] = array($info['name']); - \SimpleSAML\Logger::debug('Facebook Returned Attributes: '. implode(", ", array_keys($attributes))); + \SimpleSAML\Logger::debug('Facebook Returned Attributes: '.implode(", ", array_keys($attributes))); $state['Attributes'] = $attributes; $facebook->destroySession(); } - }