search

SleepWalker / hoauth

yii-hoauth provides simple integration with social network authorisation lib Hybridauth in Yii.
Other
59 stars 35 forks source link

Wrong call to hoauthAfterLogin while checking for hoauthProcessUser meth... #25

Closed oinegue closed 10 years ago

oinegue commented 10 years ago

...od

I didn't see hoauthProcessUser callback documented anywhere, but I think there was an error HOAuthAction.php, maybe due to a copy&paste. After checking existence of hoauthProcessUser, I'd expect to call that method, not hoauthAfterLogin. By the way, hoauthProcessUser should be documented because it's a powerful callback for those who don't use yii-user and don't want the default implementation of processUser.

kradam commented 10 years ago

I've just wanted to issue that.Is there a chance to accept this PR?

SleepWalker commented 10 years ago

Thanks for the bug fix! Sorry that it took so long, to answer here I have not much time at the moment :( But I will try to update documentation on this week.