Page MenuHomePhabricator

Merge TYPE_PROJECT_COLUMNS and TYPE_COLUMN transactions into a more general TYPE_COLUMNS transaction
ClosedPublic

Authored by epriestley on Apr 6 2016, 10:10 AM.
Tags
None
Referenced Files
F19867764: D15634.id.diff
Mar 15 2026, 12:56 AM
F19825393: D15634.id37681.diff
Mar 8 2026, 11:01 AM
F19819189: D15634.diff
Mar 6 2026, 4:01 AM
F19812588: D15634.diff
Mar 4 2026, 5:33 AM
F19811287: D15634.diff
Mar 3 2026, 11:37 PM
F19788638: D15634.diff
Feb 25 2026, 4:25 PM
F19718766: D15634.id37673.diff
Feb 12 2026, 4:20 PM
F19712104: D15634.id37673.diff
Feb 11 2026, 8:08 PM
Subscribers
None

Details

Summary

Ref T6027. We currently have two different transaction types:

  • TYPE_PROJECT_COLUMNS does most of the work, but has a sort of weird structure and isn't really suitable for API use.
  • TYPE_COLUMN is this weird, junk transaction which mostly just creates the other transaction.

Merge them into a single higher-level TYPE_COLUMNS transaction which works properly and has a sensible structure and comprehensive error checking.

Remaining work here:

  • I've removed the old rendering logic, but not yet added new logic. I need to migrate the old transaction types and add new rendering logic.
  • Although the internal representation is now suitable for use in the API, it isn't properly exposed yet.
Test Plan
  • Created tasks into a column.
  • Ran unit tests.
  • Moved tasks between columns.
  • Will perform additional testing in followups.

Diff Detail

Repository
rP Phabricator
Branch
column1
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/maniphest/editor/ManiphestTransactionEditor.php:1149XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 11539
Build 14408: Run Core Tests
Build 14407: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Merge TYPE_PROJECT_COLUMNS and TYPE_COLUMN transactions into a more general TYPE_COLUMNS transaction.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Apr 6 2016, 3:10 PM
This revision was automatically updated to reflect the committed changes.