Hi Ice,

In class: "facebook_status_views_handler_field_status extends views_handler_field" 'function render' passes a empty status to facebook_status_render_alter hook, due to the fact that the keys in the $values array are different from what are used.

I found that appending the keys with the module name (i.e. changing $values->sid to $values->facebook_status_sid) resolves the issue.

Another off the topic issue, the function facebook_status_save_status returns the saved status containing the sid, but if it happens to be the case when you are pretending to save the status (when $new_status == $status), function returns the status without any sid. Is it left by chance or you have reasons for this?

Thanks.

Comments

icecreamyou’s picture

Status: Active » Fixed

Nice catch on both accounts, thanks! Committed fixes to CVS. (With regard to the second point, the returned status should now always have the SID property.)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.