Page MenuHomePhabricator
Differential D21015

Make AuthProvider, ExternalAccount, and ExternalAccountIdentifier all Destructible
ClosedPublic
Actions

Authored by epriestley on Feb 22 2020, 3:19 AM.
Tags
None
Referenced Files
F15452254: D21015.id50078.diff
Sat, Mar 29, 3:22 AM
F15442230: D21015.id50078.diff
Wed, Mar 26, 10:13 PM
F15436939: D21015.id50078.diff
Tue, Mar 25, 4:48 PM
F15431795: D21015.id50060.diff
Mon, Mar 24, 2:33 PM
F15425538: D21015.diff
Sun, Mar 23, 4:53 AM
F15419166: D21015.id50078.diff
Fri, Mar 21, 4:09 AM
F15418962: D21015.id50078.diff
Fri, Mar 21, 2:44 AM
F15415167: D21015.diff
Thu, Mar 20, 4:32 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13493: JIRA API has changed identifiers from "key" to "accountId"
Commits
rP0872051bfa28: Make AuthProvider, ExternalAccount, and ExternalAccountIdentifier all…
Summary

Depends on D21014. Ref T13493. Make these objects all use destructible interfaces and destroy sub-objects appropriately.

Test Plan
  • Used bin/remove destroy --trace ... to destroy a provider, a user, and an external account.
  • Observed destruction of sub-objects, including external account identifiers.

Diff Detail

Repository
rP Phabricator
Branch
auth7
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23884
Build 32864: Run Core Tests
Build 32863: arc lint + arc unit

Revision Contents
Changeset List

Diff 50060

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/auth/storage/PhabricatorAuthProviderConfig.php

Loading...
View Options

src/applications/people/storage/PhabricatorExternalAccount.php

Loading...
View Options

src/applications/people/storage/PhabricatorExternalAccountIdentifier.php

Loading...
View Options

src/applications/people/storage/PhabricatorUser.php

Loading...
View Options

src/infrastructure/query/policy/PhabricatorCursorPagedPolicyAwareQuery.php

Loading...
Phabricator · Built by Phacility