If an item is imported to a user's activitystream and then subsequently deleted, it makes sense for activitystream to exclude that item from future imports.

Currently, a replacement node will be created the next time the feed / remote source is refreshed.

CommentFileSizeAuthor
#1 736406-dont_import_deleted_items-2.x.patch3.88 KBxurizaemon
#1 736406-dont_import_deleted_items-1.x.patch2.96 KBxurizaemon
#2 736406-dont_import_deleted_items-2.x.patch3.92 KBxurizaemon
#2 736406-dont_import_deleted_items-1.x.patch3 KBxurizaemon

Comments

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Version: 6.x-2.x-dev » 6.x-1.x-dev
StatusFileSize
new 736406-dont_import_deleted_items-1.x.patch2.96 KB
new 736406-dont_import_deleted_items-2.x.patch3.88 KB

PLEASE USE PATCHES IN NEXT COMMENT - NOT THESE TWO.

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Title: When activitystream-created nodes are deleted, don't re-import them on the next run » Don't import items if they've been previously imported and deleted
Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Active » Needs review
StatusFileSize
new 736406-dont_import_deleted_items-1.x.patch3 KB
new 736406-dont_import_deleted_items-2.x.patch3.92 KB

Corrected hook_update_N values to match in both patches.

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Version: 6.x-1.x-dev » 6.x-2.x-dev

Actually, looks like this was already a bug report - #667156: Deleted data reappears on running cron

But have marked that one duplicate (despite it being the first) because there's a patch here.

mokko’s picture

mokko commented

A work around that seems to work is to unpublish the respective items. Could be quite difficult to educate the users to do this if you have many of them. I will try the patch tonight.