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.
Comment | File | Size | Author |
---|---|---|---|
#23 | images_not_displaying-2656664-23.patch | 1.75 KB | tim-diels |
#22 | images_not_displaying-2656664-22.patch | 1.42 KB | Jelle_S |
#17 | images_not_displaying-2656664-17.patch | 3.48 KB | Jelle_S |
#15 | images_not_displaying-2656664-15.patch | 3.44 KB | mooazhammam |
Comments
Comment #2
bvakulin CreditAttribution: bvakulin as a volunteer 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 CreditAttribution: 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 CreditAttribution: mooazhammam commentedadd the options settings as checkbox to ease of use and to avoid typos
Comment #5
oliveyrc CreditAttribution: oliveyrc commentedNice Patch @mooazhammam. Applied okay and now pulls in the missing/required fields.
Comment #6
Freya CreditAttribution: Freya commentedAlso get the fields for a link (description + name)
Comment #7
Samba B CreditAttribution: Samba B commented#2 worked for me.
Comment #8
veverita CreditAttribution: veverita commentedI used #6 and it worked for me.
Thanks
Comment #9
GuyPaddock CreditAttribution: GuyPaddock at RedBottle Design, LLC for House at Work commented@Freya: Why are there spaces around some of the field names?
Comment #10
ph7 CreditAttribution: ph7 commentedI used #6 and it worked for me.
Thanks
Comment #11
mrprasa CreditAttribution: mrprasa as a volunteer 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 CreditAttribution: mrprasa as a volunteer 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 CreditAttribution: Gik000 as a volunteer commented#6 worked for me
Thank you
Comment #15
mooazhammam CreditAttribution: mooazhammam as a volunteer and 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 CreditAttribution: mooazhammam as a volunteer and 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 CreditAttribution: leooo_l as a volunteer 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 CreditAttribution: 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 CreditAttribution: daveferrara1 commented7.x-3.4 Includes a version of this fix.
Comment #26
daveferrara1 CreditAttribution: daveferrara1 commented