First I'll cover a small bug I came across:

fb_views_handler_profile_pic.inc line 11 currently reads:
if ($fbu = $values->fb_user_app_fbu) {
but should read
if ($fbu = $values->{$this->field_alias}) {
(in case you're using a relationship for the user)

but, I've come to realize that possibly the more correct fix needed is for fb_views to have a relationship handler that loads the fbu of a user and then have fields like name, picture, etc. that use the value supplied by the relationship handler (also important for cases where there are multiple apps or users with no fbu). I don't think I know enough about views to try to author something like that right now (maybe I will soon), but would be super cool if someone could help write something like this!

Comments

nighthawk117’s picture

oops forgot the same change needs to be made on the next line too (ie. replace $values->fb_user_app_fbu with $values->{$this->field_alias}

Dave Cohen’s picture

Thanks for the tweaks. Its a struggle to keep up with views api, and I've had no time to do it. Need some community support for this.

I trust your relationship idea is correct. I want to add similar features not just for the currently logged in users, but also the user who's page you may be visiting at the moment. (A feature specific to 'tabs' on facebook pages).