Paths

Table of Contentst

Differential D18418

Migrate revision storage to modern status constants ("accepted") instead of legacy numeric values ("2")
ClosedPublic
Actions

Authored by epriestley on Aug 11 2017, 11:34 PM.

Tags

None

Referenced Files

	F19505238: D18418.diff
	Fri, Jan 9, 10:26 AM

	F19072122: D18418.id.diff
	Nov 30 2025, 9:28 PM

	F19060469: D18418.diff
	Nov 29 2025, 9:21 AM

	F18934419: D18418.id44262.diff
	Nov 10 2025, 3:02 PM

	F18846840: D18418.id.diff
	Oct 29 2025, 9:24 PM

	F18841472: D18418.diff
	Oct 28 2025, 9:23 AM

	F18815397: D18418.id44253.diff
	Oct 21 2025, 2:00 AM

	F18812350: D18418.diff
	Oct 20 2025, 8:45 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential

Commits

rP7b695aa43bd1: Migrate revision storage to modern status constants ("accepted") instead of…

Summary

Ref T2543. Rewrites all the storage to use constants.

Note that transactions still use legacy values, I'll migrate and update them separately.

Test Plan

Ran migration.
Browsed around, changed revision states, viewed dashboard, etc.
Selected DISTINCT() and GROUP_CONCAT() of the status field in the database, saw sane/expected before and after values.
Verified that old Conduit methods still return numeric constants for compatibility.

Diff Detail

Repository

Branch

status19

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 18045
Build 24240: Run Core Tests
Build 24239: arc lint + arc unit

Event Timeline

epriestley created this revision.Aug 11 2017, 11:34 PM

Harbormaster completed remote builds in B18045: Diff 44253.Aug 11 2017, 11:35 PM

epriestley edited the summary of this revision. (Show Details)Aug 11 2017, 11:36 PM

chad accepted this revision.Aug 11 2017, 11:53 PM

This revision is now accepted and ready to land.Aug 11 2017, 11:53 PM

Closed by commit rP7b695aa43bd1: Migrate revision storage to modern status constants ("accepted") instead of… (authored by epriestley). · Explain WhyAug 12 2017, 11:02 AM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential.Aug 12 2017, 11:17 AM

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20170811.differential.02.modernstatus.sql

17 lines

src/

applications/

differential/

constants/

DifferentialLegacyQuery.php

15 lines

query/

DifferentialRevisionQuery.php

4 lines

storage/

DifferentialRevision.php

19 lines

xaction/

DifferentialRevisionStatusTransaction.php

3 lines

Diff 44253

resources/sql/autopatches/20170811.differential.02.modernstatus.sql

Loading...

src/applications/differential/constants/DifferentialLegacyQuery.php

Loading...

src/applications/differential/query/DifferentialRevisionQuery.php

Loading...

src/applications/differential/storage/DifferentialRevision.php

Loading...

src/applications/differential/xaction/DifferentialRevisionStatusTransaction.php

Loading...