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.
Recent versions of the Twitter module add a "global" flag for accounts. The Twitter Pull module is not passing this attribute to twitter_connect() to use those.
Comment | File | Size | Author |
---|---|---|---|
#2 | 2566489-twitter-global.patch | 441 bytes | wwhurley |
Comments
Comment #2
wwhurley CreditAttribution: wwhurley at Forum One commentedComment #3
wwhurley CreditAttribution: wwhurley at Forum One commentedComment #4
glynster CreditAttribution: glynster commentedThis patch fixes the issue even though the latest dev has fixed this. The line in question is 177 from twitter_pull.class.inc.
$twitter = twitter_connect($twitter_account, $access_global);
On line 130 - 132 is printed
However if you change line 177 to:
$twitter = twitter_connect(NULL, TRUE);
Everything works as it should.
Comment #5
Nodz CreditAttribution: Nodz commentedThis also solved it for me. But it was hard to find the fix and took me time. Perhaps it's time to release an alpha3 because as is alpha2 doesn't work as expected with the recent (security related) updates of the twitter module.
Comment #6
RenrhafYes a tagged release for this would be great.
Thanks for your patch !
Comment #7
rupertj CreditAttribution: rupertj at Torchbox commentedYes, please create a new release from the latest dev. Upgrading the twitter module recently broke one of our client's twitter streams, and it it took ages to track down the source of the fault and find this issue.
Comment #8
rupertj CreditAttribution: rupertj at Torchbox commentedThe actual fix for this was in #2562681: Twitter 5.9 module incompatible with Twitter Pull, btw.
Comment #9
joachim CreditAttribution: joachim commentedDone.
Comment #10
RenrhafThanks a lot !
Comment #11
glynster CreditAttribution: glynster commented@rupertj I still had to apply this patch to the latest version so everything worked again.