artdarek / oauth-4-laravel

OAuth Service Provider for Laravel 4
684 stars 217 forks source link

Emergency: 5 methods NOT implemented here - Class Artdarek\OAuth\TokenStorage #95

Closed asmshaon closed 9 years ago

asmshaon commented 9 years ago

Class Artdarek\OAuth\TokenStorage contains 5 abstract methods and must therefore be declared abstract or implement the remaining methods (OAuth\Common\Storage\TokenStorageInterface::storeAuthorizationState, OAuth\Common\Storage\TokenStorageInterface::hasAuthorizationState, OAuth\Common\Storage\TokenStorageInterface::retrieveAuthorizationState OAuth\Common\Storage\TokenStorageInterface:clearAuthorizationState, OAuth\Common\Storage\TokenStorageInterface::clearAllAuthorizationStates,

asmshaon commented 9 years ago

Need to implement these functions

public function storeAuthorizationState($service, $state)
{

}

public function hasAuthorizationState($service)
{

}

public function retrieveAuthorizationState($service){

}

public function clearAuthorizationState($service)
{

}

public function clearAllAuthorizationStates()
{

}
asmshaon commented 9 years ago

yes, please add the following 5 function for now. They have 10 functions, but you implemented only 5. https://github.com/Lusitanian/PHPoAuthLib/blob/master/src/OAuth/Common/Storage/TokenStorageInterface.php

On Wed, Aug 6, 2014 at 4:23 PM, lvandyk notifications@github.com wrote:

Any quick fix for this? Did a composer update and cannot continue.

— Reply to this email directly or view it on GitHub https://github.com/artdarek/oauth-4-laravel/issues/95#issuecomment-51317016 .

asmshaon commented 9 years ago

at least add those missing 5 functions for now in this class : Artdarek\OAuth\TokenStorage

otherwise i cant work.

On Wed, Aug 6, 2014 at 4:23 PM, lvandyk notifications@github.com wrote:

Any quick fix for this? Did a composer update and cannot continue.

— Reply to this email directly or view it on GitHub https://github.com/artdarek/oauth-4-laravel/issues/95#issuecomment-51317016 .