Page MenuHomePhabricator
Differential D8330

Remove "dateCommitted" field from DifferentialRevision
ClosedPublic
Actions

Authored by epriestley on Feb 25 2014, 3:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 30, 11:17 PM
Unknown Object (File)
Sun, Apr 28, 5:21 AM
Unknown Object (File)
Sun, Apr 28, 5:21 AM
Unknown Object (File)
Sun, Apr 28, 5:21 AM
Unknown Object (File)
Sat, Apr 27, 12:02 PM
Unknown Object (File)
Sat, Apr 27, 11:10 AM
Unknown Object (File)
Mon, Apr 22, 10:34 PM
Unknown Object (File)
Sat, Apr 20, 6:30 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T2222: Implement ApplicationTransactions in Differential
Commits
Restricted Diffusion Commit
rP0f0673b9e526: Remove "dateCommitted" field from DifferentialRevision
Summary

Ref T2222. This is obsolete and no longer used. We could deduce it from transactions or commits in modern Phabricator if we wanted it. We may implement a more general mechanism for T4434.

Test Plan

grep

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
src/
applications/
differential/
conduit/
1 line
editor/
4 lines
5 lines
storage/
2 lines
repository/
worker/
commitmessageparser/
1 line

Diff 19828

View Options

resources/sql/autopatches/20140224.dxclean.1.datecommitted.sql

Loading...
View Options

src/applications/differential/conduit/ConduitAPI_differential_close_Method.php

Loading...
View Options

src/applications/differential/editor/DifferentialCommentEditor.php

Loading...
View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
View Options

src/applications/differential/storage/DifferentialRevision.php

Loading...
View Options

src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php

Loading...
Phabricator · Built by Phacility