Deleted tweets on twitter account are not deleted on the site and there is currently no way to delete it. any suggestion?

CommentFileSizeAuthor
#9 twitter-n2279893-9-7.x-5.x.patch5.57 KBdamienmckenna
#4 twitter-n2279893-4.patch4.1 KBdamienmckenna
#2 twitter-deleted_post_on_twitter_are_not_delted_on_site-2279893-2.patch3.05 KBr-mo

Comments

Anonymous’s picture

Anonymous (not verified) commented

It would be nice to have an option to re-sync tweets.

r-mo’s picture

r-mo commented
StatusFileSize
new twitter-deleted_post_on_twitter_are_not_delted_on_site-2279893-2.patch3.05 KB

Quick patch to check the most recent 10 statuses for deleted tweets, on each cron.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-5.8 » 7.x-5.x-dev
Status: Active » Needs work
Issue tags: -delete tweets

Needs a little work to adhere to the Drupal coding standards.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review
StatusFileSize
new twitter-n2279893-4.patch4.1 KB

This has been rerolled, updated to the Drupal coding standards, and I added an item to hook_uninstall for the new variable.

alfthecat’s picture

alfthecat commented

Patch applies cleanly and it works brilliantly too! Just need to set "remove deleted twitter statuses of site users" on the main settings page.

Thanks so much!!

hkirsman’s picture

hkirsman commented

#4 works!

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Category: Bug report » Feature request
damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Closed a duplicate: #2150773: Tweets deleted on twitter.com are still showing in block after cron run

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new twitter-n2279893-9-7.x-5.x.patch5.57 KB

Rerolled. I've also tidied up the code a little in twitter_remove_deleted_statuses() and expanded its check with a new setting that allows the site builder to control how many tweets will be checked at once.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Oh, I also changed from using db_query() to db_query_range().

DamienMcKenna queued 9: twitter-n2279893-9-7.x-5.x.patch for re-testing.

rbrandon’s picture

rbrandon commented

Patch looks good. as long as the re-fetch is occurring it might be nice to check for updated tweets as well. Any word on this getting rolled into the module?