The call that is being made to fetch the user timeline using Twitter API, is already fetching the retweet count. It's an useful data so why not store it as well?

CommentFileSizeAuthor
#7 store_retweet_count_with_tweets-2841765-7.patch2.65 KBqzmenko
#2 store_retweet_count_with_tweets-2841765-1.patch2.54 KBSGhosh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SGhosh created an issue. See original summary.

SGhosh’s picture

SGhosh CreditAttribution: SGhosh as a volunteer and at Innoraft commented
FileSize
store_retweet_count_with_tweets-2841765-1.patch2.54 KB

The attached path does the following -

  1. Add a new column to the table twitter(which stores tweets and their related info) - retweet_count, which will store the retweet count data for the tweet.
  2. Adds the additional code to the twitter API library and it's drupal associated code, that extracts the data from the twitter API response, and stores it in the DB. Now to d
    the same to the retweet count data.
  3. Adds a retweet count views data handler to be used in the twitter view.
SGhosh’s picture

SGhosh CreditAttribution: SGhosh as a volunteer and at Innoraft commented
Status: Active » Needs review
SGhosh’s picture

SGhosh CreditAttribution: SGhosh as a volunteer and at Innoraft commented
Assigned: SGhosh » Unassigned
andrey.troeglazov’s picture

andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commented
Version: 7.x-5.x-dev » 7.x-6.x-dev
Assigned: Unassigned » andrey.troeglazov
andrey.troeglazov’s picture

andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commented

Hello SGhosh,
Thanks for your patch, if you can reroll it against 7.6-dev branch it will be good.

qzmenko’s picture

qzmenko
Primary language Russian
Location Novosibirsk
CreditAttribution: qzmenko as a volunteer and at DrupalJedi commented
FileSize
store_retweet_count_with_tweets-2841765-7.patch2.65 KB

Rerolled for latest dev version with changes in twitter_schema.

andrey.troeglazov’s picture

andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commented
Assigned: andrey.troeglazov » Unassigned
Status: Needs review » Fixed

Commited into dev branch. Thank you.

Status: Fixed » Closed (fixed)

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

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: » #2580277: Plan for Twitter v7.x-6.3 release