HomePhorge

Switch File deletion to use ModularTransactions

Description

Switch File deletion to use ModularTransactions

Summary: Fixes T12587. Adds a new PhabricatorFileDeleteTransaction that enqueues File delete tasks.

Test Plan:

  • hack PhabricatorFileQuery to ignore isDeleted state
  • stop daemons
  • upload a file, delete it from the UI
  • check that the DB has updated isDeleted = 1
  • check timeline rendering in File detail view
  • start daemons
  • confirm rows are deleted from DB

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin, thoughtpolice

Maniphest Tasks: T12587

Differential Revision: https://secure.phabricator.com/D17723

Details

Provenance
Austin McKinleyAuthored on Apr 18 2017, 12:48 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPab2aa74d6e29: Fix several duplication/replay behaviors in Aphlict
Branches
Unknown
Tags
Unknown

Event Timeline