Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The code looks nice and clean. Any plans for a Drupal 7 version? I definitely would use this more simplified version over the other facebook tools.
I could even take a quick stab at it the initial port, if you didn't mind maintaining both branches :)
Comments
Comment #1
TripX CreditAttribution: TripX commentedsubscribe
this would be awesome
Comment #2
aidanlis CreditAttribution: aidanlis commentedI don't think it will even need porting, just change the .info file and it should work in D7. I'll release a D7 version when I get some free time (not soon).
Comment #3
rickmanelius CreditAttribution: rickmanelius commented2 changes required. First, the .info changes
And also, hook theme goes from arguments to variables
Comment #4
Rechi CreditAttribution: Rechi commentedDidn't work for me completely.
The $items variable in the facebook_pull-feed.tpl.php doesn't get assigned correctly.
If I add
$items[0] = $variables[0];
in the facebook_pull-feed.tpl.php I get at least the first facebook post.
I also have the problem that with the generated https://graph.facebook.com/... call
I only get my first post (tried the generated link in the browser)
Drupal Core Verision 7.7
Comment #5
Rechi CreditAttribution: Rechi commentedOk first problem solved:
change:
return theme('facebook_pull_feed', $data);
to
return theme('facebook_pull_feed',array('items' => $data));
Comment #6
Juan C CreditAttribution: Juan C commentedSubscribe
Comment #7
aidanlis CreditAttribution: aidanlis commentedPorted to D7!