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.
After updating to latest CORE I am getting the following warning intermittently:
Warning: Invalid argument supplied for foreach() in _socialmedia_widgets_get_facebook_locales() (line 754 of /home/felix/public_html/sites/all/modules/socialmedia/socialmedia.widgets.inc).
Please advise.
Comments
Comment #2
downwriter CreditAttribution: downwriter commentedAny updates on this issue? I just ran into this problem as well.
Comment #3
mohankumar.r CreditAttribution: mohankumar.r commentedHi dude, I think, its php version issue. It does not occur on a server that runs PHP 5.4.10.
Comment #4
kyletaylored CreditAttribution: kyletaylored at LevelTen Interactive commentedI can confirm this is actually an issue - Facebook stopped supporting their locales XML feed and those now run through some app interface. Luckily, there is a JSON version on Github we can refer to (I don't think these will change very much), so I'll add my patch here.
Comment #5
kyletaylored CreditAttribution: kyletaylored at LevelTen Interactive commentedThis should be patched on dev now.
Comment #6
kyletaylored CreditAttribution: kyletaylored at LevelTen Interactive commentedComment #7
PROMESI can't find a new dev-version or a patch. The current dev-version is still from 2015-Mar-13...
???????
Comment #8
kyletaylored CreditAttribution: kyletaylored at LevelTen Interactive commentedYea, this is weird. Usually, the commit shows up, but it doesn't seem to be generating a new tarball/zip. But you can definitely see it in the git log.
http://cgit.drupalcode.org/socialmedia/log/
Comment #9
junaidpvI assume commit should go to 7.x-1.x branch to get next dev tarball to get generated,
Also, this appears to be a problem with Facebook itself. It is already reported at Facebook's bug tracker: https://developers.facebook.com/bugs/1836827343245862/ . They accepted it as a valid one.
Comment #11
kyletaylored CreditAttribution: kyletaylored at LevelTen Interactive commented@junaidpv yep, you're right.
And thanks for letting me know! I added in another workaround, for now, that should also pull in and cache updates from a JSON file on Github. It's non-dynamic fix, but one nonetheless.