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.

CommentFileSizeAuthor
#2 2566489-twitter-global.patch441 byteswwhurley
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

wwhurley created an issue. See original summary.

wwhurley’s picture

FileSize
441 bytes
wwhurley’s picture

Status: Active » Needs review
glynster’s picture

This 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

// Twitter API params.
$twitter_account = NULL;
$access_global = TRUE;

However if you change line 177 to:

$twitter = twitter_connect(NULL, TRUE);

Everything works as it should.

Nodz’s picture

This 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.

Renrhaf’s picture

Yes a tagged release for this would be great.
Thanks for your patch !

rupertj’s picture

Status: Needs review » Fixed

Yes, 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.

rupertj’s picture

joachim’s picture

Done.

Renrhaf’s picture

Thanks a lot !

glynster’s picture

@rupertj I still had to apply this patch to the latest version so everything worked again.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.