Page MenuHomePhabricator

Let Herald activation depend on which transactions are being applied, and generate transactions
ClosedPublic

Authored by epriestley on Mar 5 2014, 3:03 AM.
Tags
None
Referenced Files
F19467832: D8404.diff
Sat, Jan 3, 6:13 PM
F19080561: D8404.diff
Dec 2 2025, 1:44 AM
F19077579: D8404.id.diff
Dec 1 2025, 3:28 PM
F19070835: D8404.diff
Nov 30 2025, 5:58 PM
F18853340: D8404.id.diff
Oct 31 2025, 3:55 PM
F18850579: D8404.diff
Oct 30 2025, 5:59 PM
F18731977: D8404.id.diff
Sep 30 2025, 3:53 PM
F18724748: D8404.id.diff
Sep 30 2025, 4:00 AM
Subscribers

Details

Summary

Ref T2222. Ref T4484. This is a stepping stone to getting Herald supported in the new Differental code. Generally:

  • Instead of an Editor either supporting or not supporting Herald, let it choose based on transactions. Specifically, Differential only runs rules on revision creation and diff updates.
  • Optionally, allow an Editor to return some transactions to apply instead of having to apply everything itself. This lets us make it clear why changes happend in the transaction log, and share more code.
  • I updated only one transaction type (owners in Maniphest) since it was the easiest and cleanest to update and test. Everything else still works like it used to, it just won't generate a transaction record yet.
  • The transaction records are a touch rough, but we can clean them up later.
Test Plan

{F122282}

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan edited edge metadata.
btrahan added inline comments.
src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
643

nice

This revision is now accepted and ready to land.Mar 5 2014, 6:50 PM