Closed (fixed)
Project:
Facebook Pull
Version:
7.x-3.1
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Jan 2016 at 06:54 UTC
Updated:
18 Sep 2017 at 17:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
bvakulin commentedI have this bug too. I fixed it by adding fields to the options in _facebook_pull_render_block function.
before:
$options = array(
'limit' => $limit,
'locale' => $language->language,
);
after:
$options = array(
'limit' => $limit,
'locale' => $language->language,
'fields' => 'picture,type,message,created_time,from,link',
);
Comment #3
ShijunLiu commentedI think it is because of facebook upgraded their api and doesn't return the certain fields any more, but fortunately you can still get them by providing them explicitly. Thus bvakulin's code solved this problem.
I also made a patch based on his code.
Comment #4
mooazhammam commentedadd the options settings as checkbox to ease of use and to avoid typos
Comment #5
oliveyrc commentedNice Patch @mooazhammam. Applied okay and now pulls in the missing/required fields.
Comment #6
Freya commentedAlso get the fields for a link (description + name)
Comment #7
samba b commented#2 worked for me.
Comment #8
veverita commentedI used #6 and it worked for me.
Thanks
Comment #9
guypaddock commented@Freya: Why are there spaces around some of the field names?
Comment #10
ph7 commentedI used #6 and it worked for me.
Thanks
Comment #11
mrprasa commented#6 worked for me also. It pulls the thumbnail image though, and I need the full size image.
I tried changing the "picture" paramater to "full_picture" and do not get any results although this should work according to tests in the facebook developer graphs api at https://developers.facebook.com/tools/explorer
Below is the code I used.
$options = array(
'limit' => $limit,
'locale' => $language->language,
'fields' => 'full_picture,type,message,created_time,from,link',
Comment #12
mrprasa commentedResolved the issue by modifying the template facebook_pull-feed.tpl.php by changing $item->picture to $item->full_picture
if (in_array($item->type, array('photo', 'video'))):echo l('<img src="' . $item->full_picture . '" />', $item->link, array('html' => true, 'attributes' => array('target' => '_blank')));Comment #13
truls1502Why not have these patch be included to the next release version? Especially patch number 4 in comment #4 which give us able to choose of the fields.
However I am not able to get this works with Boxes-module
Comment #14
Gik000 commented#6 worked for me
Thank you
Comment #15
mooazhammam commentedUpdated patch to 7.x-3.1
This Patch Do the following
1- fix the missing from property
2- allow the user to select the needed property to display
3- fix missing images and properties
Thanks
Comment #16
mooazhammam commentedComment #17
jelle_sNew patch, also includes the 'name' and 'description' fields for when the item is a link.
Comment #18
fenstrat#17 does indeed fix the issues noted here.
Comment #19
leooo_l commentedHello, I'm trying to let images work properly following your patches and advises. For the moment I get thumbnails. But they are not aligned and a little messy. Moreover I tried to make changes suggested by mrprasa in order to get full pictures and not thumbnails, but I did not let the suggested code work properly (I got a syntax error somewhere in here '" />' after the 'full_picture wording....)
May someone help me out:
1) to get full size pictures
2) to get those picture aligned inside the page and not messy all around
?
Many thanks in advance for your help
Comment #20
fenstrat@leooo_l Please see https://www.drupal.org/patch/apply on how to apply the patch in #17. If you require a full size image you will probably need to override facebook_pull-feed.tpl.php in your own theme see https://www.drupal.org/docs/7/theming/overriding-themable-output/about-o... - As for being "messy" you'll probably have to try some custom CSS to get what you're after.
Comment #21
daveferrara1 commentedThe patch provided #17 does fix the issue, however it is not working if you check and uncheck particular boxes. Some more work is needed for this feature.
Comment #22
jelle_sReroll of #17
Comment #23
tim-dielsReroll of #22 for version 3.3 and also added object_id as field so you can use it to retrieve better images.
Comment #25
daveferrara1 commented7.x-3.4 Includes a version of this fix.
Comment #26
daveferrara1 commented