screen name instead of user id

[glanchow]

I use a custom provider as described in the documentation, and there is a problem with this function :
public function loadUserByUsername($username)
The problem is that (despite it's name) $username should be the twitter user id, because the rest of the code waits for it, and because the user can change its screen name in twitter, so the twitter id is the only reliable username.

Line 95 of Security / Authentication / Provider / TwitterProvider.php
$user = $this->userProvider->loadUserByUsername($accessToken['screen_name']);

I'm new to this module but I suppose we need to change this line.

[net02]

yup, i was following the docs to integrate it with FOSUserBundle, and in the suggested security provider it uses twitterID, yet the value passed is the screen_name. And as you said, it's not reliable.

Line 95 of Security / Authentication / Provider / TwitterProvider.php
$user = $this->userProvider->loadUserByUsername($accessToken['user_id']);

will fix it.
I'll attach a pull request asap ;)

[glanchow]

thank you net02 but there was a pullrequest on that duplicate thread.