Page MenuHomePhabricator
Differential D21448

Wrap all direct access to author/committer properties on "CommitData"
ClosedPublic
Actions

Authored by epriestley on Aug 12 2020, 7:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 22 2026, 12:33 AM
Unknown Object (File)
Apr 2 2026, 2:51 AM
Unknown Object (File)
Apr 1 2026, 11:59 AM
Unknown Object (File)
Mar 14 2026, 7:01 PM
Unknown Object (File)
Mar 14 2026, 6:55 PM
Unknown Object (File)
Feb 6 2026, 12:01 AM
Unknown Object (File)
Jan 21 2026, 4:57 PM
Unknown Object (File)
Jan 11 2026, 7:43 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch
Commits
rPe454c3dafe99: Wrap all direct access to author/committer properties on "CommitData"
Summary

Ref T13552. Currently, various callers read raw properties off "CommitData" directly. Wrap these in accessors to support storage changes which persist "CommitRef" information instead.

Test Plan
  • Ran "diffusion.querycommits", saw the same data before and after.
  • Looked at a commit, saw authorship information and date.
  • Viewed tags in a repository, saw author information.
  • Ran "rebuild-identities", saw no net effect.
  • Grepped for callers to "getCommitDetail(...)".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 12 2020, 7:42 PM
Harbormaster completed remote builds in B24862: Diff 51047.Aug 12 2020, 7:44 PM
epriestley requested review of this revision.Aug 12 2020, 7:44 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 16 2020, 12:36 AM
Closed by commit rPe454c3dafe99: Wrap all direct access to author/committer properties on "CommitData" (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPe454c3dafe99: Wrap all direct access to author/committer properties on "CommitData".

Revision Contents
Changeset List

Diff 51084

View Options

src/applications/diffusion/conduit/DiffusionQueryCommitsConduitAPIMethod.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...
View Options

src/applications/diffusion/data/DiffusionPathChange.php

Loading...
View Options

src/applications/diffusion/view/DiffusionTagListView.php

Loading...
View Options

src/applications/repository/management/PhabricatorRepositoryManagementRebuildIdentitiesWorkflow.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryCommit.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryCommitData.php

Loading...
View Options

src/applications/repository/worker/PhabricatorRepositoryCommitPublishWorker.php

Loading...
Phabricator · Built by Phacility