CB » Facebook Connect CB Plugin

As discussed here:

[4:44:40 μμ] Nicholas G. Antimisiaris: great
[4:52:43 μμ] Nicholas G. Antimisiaris: ok - starting fb tests
[4:52:47 μμ] Nicholas G. Antimisiaris: first question
[4:53:44 μμ] Nicholas G. Antimisiaris: Registration: [first_name]_[last_name]
[4:55:31 μμ] Kyle (krileon): ?
[4:55:46 μμ] Nicholas G. Antimisiaris: Username facebook users are given on synchronization. Usernames should have some unique element such as fb_ to avoid conflicts. Substitions of [first_name], [last_name], and [user_id] will add a users facebook first name, last name, and id. Leave blank for no change of username.
[4:56:19 μμ] Nicholas G. Antimisiaris: suggest that you should follow your suggestion for default case
[4:57:24 μμ] Nicholas G. Antimisiaris: #1 - Cb sites might not have first name / last name fields enabled
#2 - even if these are enabled, the combination [first_name]_[last_name] does not guarantee uniqueness
[4:57:52 μμ] Nicholas G. Antimisiaris: propose to change this to something really unique that does not by default have [first_name] [last_name] in it
[4:58:22 μμ] Nicholas G. Antimisiaris: something like:
fb_[user_id]_[name]
[4:58:34 μμ] Kyle (krileon): that's way too long of a username..
[4:59:08 μμ] Nicholas G. Antimisiaris: well then fb_[user_id]
[4:59:19 μμ] Kyle (krileon): that'll work.. is what most facebook connect sites use
[4:59:21 μμ] Nicholas G. Antimisiaris: why, do we have a username length limit?
[4:59:49 μμ] Nicholas G. Antimisiaris: if there is a limit, you should mention it in tooltip
[5:00:11 μμ] Kyle (krileon): there is no limit, it would just seam absurdly long for: fb_1326262_kyle_larose
[5:00:28 μμ] Nicholas G. Antimisiaris: agree
[5:00:35 μμ] Nicholas G. Antimisiaris: but it will be unique
[5:00:46 μμ] Nicholas G. Antimisiaris: no issue with fb_[user_id]
[5:00:52 μμ] Kyle (krileon): fb_[user_id] should be sufficient

Commit r761 & r763

Good point about the uniqueness question. However it makes the CB's SEO worthless.

Could we do: [name] if that's not already existing and then add "_[user_id]" to it if it's already existing ?

That way the SEO would be clean, except in seldom cases where a homonym already registered.

Also I recommending spaces " " be converted to "_" in the username.

Commit r796 & r797