Changed method names to be clearer that we're setting params

This commit is contained in:
Alex Bilbie 2014-09-30 22:28:38 +01:00
parent 1e3a192920
commit a3f5d20592
6 changed files with 17 additions and 17 deletions

View File

@ -230,15 +230,15 @@ class AuthCodeGrant extends AbstractGrant
$accessToken->associateScope($scope);
}
$this->server->getTokenType()->set('access_token', $accessToken->getId());
$this->server->getTokenType()->set('expires_in', $this->getAccessTokenTTL());
$this->server->getTokenType()->setParam('access_token', $accessToken->getId());
$this->server->getTokenType()->setParam('expires_in', $this->getAccessTokenTTL());
// Associate a refresh token if set
if ($this->server->hasGrantType('refresh_token')) {
$refreshToken = new RefreshTokenEntity($this->server);
$refreshToken->setId(SecureKey::generate());
$refreshToken->setExpireTime($this->server->getGrantType('refresh_token')->getRefreshTokenTTL() + time());
$this->server->getTokenType()->set('refresh_token', $refreshToken->getId());
$this->server->getTokenType()->setParam('refresh_token', $refreshToken->getId());
}
// Expire the auth code

View File

@ -112,8 +112,8 @@ class ClientCredentialsGrant extends AbstractGrant
$accessToken->setSession($session);
$accessToken->save($this->server->getStorage('access_token'));
$this->server->getTokenType()->set('access_token', $accessToken->getId());
$this->server->getTokenType()->set('expires_in', $this->getAccessTokenTTL());
$this->server->getTokenType()->setParam('access_token', $accessToken->getId());
$this->server->getTokenType()->setParam('expires_in', $this->getAccessTokenTTL());
return $this->server->getTokenType()->generateResponse();
}

View File

@ -148,15 +148,15 @@ class PasswordGrant extends AbstractGrant
$accessToken->associateScope($scope);
}
$this->server->getTokenType()->set('access_token', $accessToken->getId());
$this->server->getTokenType()->set('expires_in', $this->getAccessTokenTTL());
$this->server->getTokenType()->setParam('access_token', $accessToken->getId());
$this->server->getTokenType()->setParam('expires_in', $this->getAccessTokenTTL());
// Associate a refresh token if set
if ($this->server->hasGrantType('refresh_token')) {
$refreshToken = new RefreshTokenEntity($this->server);
$refreshToken->setId(SecureKey::generate());
$refreshToken->setExpireTime($this->server->getGrantType('refresh_token')->getRefreshTokenTTL() + time());
$this->server->getTokenType()->set('refresh_token', $refreshToken->getId());
$this->server->getTokenType()->setParam('refresh_token', $refreshToken->getId());
}
// Save everything

View File

@ -139,8 +139,8 @@ class RefreshTokenGrant extends AbstractGrant
$oldAccessToken->expire($this->server->getStorage('access_token'));
$newAccessToken->save($this->server->getStorage('access_token'));
$this->server->getTokenType()->set('access_token', $newAccessToken->getId());
$this->server->getTokenType()->set('expires_in', $this->getAccessTokenTTL());
$this->server->getTokenType()->setParam('access_token', $newAccessToken->getId());
$this->server->getTokenType()->setParam('expires_in', $this->getAccessTokenTTL());
// Expire the old refresh token
$oldRefreshToken->expire($this->server->getStorage('refresh_token'));
@ -152,7 +152,7 @@ class RefreshTokenGrant extends AbstractGrant
$newRefreshToken->setAccessToken($newAccessToken);
$newRefreshToken->save($this->server->getStorage('refresh_token'));
$this->server->getTokenType()->set('refresh_token', $newRefreshToken->getId());
$this->server->getTokenType()->setParam('refresh_token', $newRefreshToken->getId());
return $this->server->getTokenType()->generateResponse();
}

View File

@ -42,7 +42,7 @@ abstract class AbstractTokenType
* @param string $key
* @param mixed $value
*/
public function set($key, $value)
public function setParam($key, $value)
{
$this->response[$key] = $value;
}
@ -52,7 +52,7 @@ abstract class AbstractTokenType
* @param string $key
* @return mixed
*/
public function get($key)
public function getParam($key)
{
return isset($this->response[$key]) ? $this->response[$key] : null;
}

View File

@ -21,13 +21,13 @@ class Bearer extends AbstractTokenType implements TokenTypeInterface
public function generateResponse()
{
$return = [
'access_token' => $this->get('access_token'),
'access_token' => $this->getParam('access_token'),
'token_type' => 'Bearer',
'expires_in' => $this->get('expires_in')
'expires_in' => $this->getParam('expires_in')
];
if (!is_null($this->get('refresh_token'))) {
$return['refresh_token'] = $this->get('refresh_token');
if (!is_null($this->getParam('refresh_token'))) {
$return['refresh_token'] = $this->getParam('refresh_token');
}
return $return;