Page MenuHomePhabricator
Differential D20913

Give "PhabricatorUserEmail" a PHID
ClosedPublic
Actions

Authored by epriestley on Nov 14 2019, 7:31 PM.
Tags
None
Referenced Files
F18829535: D20913.id.diff
Fri, Oct 24, 10:41 PM
F18824683: D20913.diff
Thu, Oct 23, 5:21 PM
F18768088: D20913.id49857.diff
Wed, Oct 8, 1:40 AM
F18765888: D20913.diff
Tue, Oct 7, 1:37 PM
F18734112: D20913.id.diff
Sep 30 2025, 10:48 PM
F18656523: D20913.id.diff
Sep 22 2025, 10:47 PM
F18632084: D20913.diff
Sep 16 2025, 5:19 PM
F18608736: D20913.id49835.diff
Sep 14 2025, 1:37 AM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13444: Provide a standalone script entry point for resolving a repository identity
Commits
rP89dcf9792a0a: Give "PhabricatorUserEmail" a PHID
Summary

Ref T13444. To interact meaningfully with "DestructionEngine", objects need a PHID. The "UserEmail" object currently does not have one (or a real "Query").

Provide basic PHID support so "DestructionEngine" can interact with the object more powerfully.

Test Plan
  • Ran migrations, checked data in database, saw sensible PHIDs assigned.
  • Added a new email address to my account, saw it get a PHID.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Nov 14 2019, 7:31 PM
Owners added a subscriber: Restricted Owners Package.Nov 14 2019, 7:31 PM
Harbormaster completed remote builds in B23663: Diff 49835.Nov 14 2019, 7:32 PM
epriestley requested review of this revision.Nov 14 2019, 7:32 PM
This revision was not accepted when it landed; it landed in state Needs Review.Nov 19 2019, 5:41 PM
Closed by commit rP89dcf9792a0a: Give "PhabricatorUserEmail" a PHID (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP89dcf9792a0a: Give "PhabricatorUserEmail" a PHID.

Revision Contents
Changeset List

Diff 49857

View Options

resources/sql/autopatches/20191114.email.01.phid.sql

Loading...
View Options

resources/sql/autopatches/20191114.email.02.populate.php

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/people/phid/PhabricatorPeopleUserEmailPHIDType.php

Loading...
View Options

src/applications/people/query/PhabricatorPeopleUserEmailQuery.php

Loading...
View Options

src/applications/people/storage/PhabricatorUserEmail.php

Loading...
Phabricator · Built by Phacility