Page MenuHomePhabricator
Differential D12563

Rename protocol adapter classes
ClosedPublic
Actions

Authored by joshuaspence on Apr 27 2015, 9:20 AM.
Tags
None
Referenced Files
F15423813: D12563.id30626.diff
Sat, Mar 22, 6:32 PM
F15404143: D12563.id31000.diff
Tue, Mar 18, 6:25 AM
F15282778: D12563.diff
Mar 4 2025, 2:15 AM
Unknown Object (File)
Feb 26 2025, 8:40 AM
Unknown Object (File)
Feb 18 2025, 1:02 PM
Unknown Object (File)
Feb 13 2025, 9:14 PM
Unknown Object (File)
Feb 8 2025, 3:59 AM
Unknown Object (File)
Feb 8 2025, 3:59 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T5655: Standardize naming conventions
Commits
Restricted Diffusion Commit
rP427cc221499d: Rename protocol adapter classes
Summary

Ref T5655. Rename protocol adapter classes to remove Base from the class name.

Test Plan

Unit tests still pass.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 30160.Apr 27 2015, 9:20 AM
joshuaspence retitled this revision from to Rename protocol adapter classes.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
joshuaspence added a task: T5655: Standardize naming conventions.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 27 2015, 9:20 AM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
joshuaspence updated this object.May 7 2015, 10:34 AM
joshuaspence edited edge metadata.
joshuaspence updated this revision to Diff 30626.May 7 2015, 10:38 AM

Rebase

epriestley accepted this revision.May 17 2015, 2:49 PM
epriestley edited edge metadata.

Oh, I thought this was a mess but this doesn't rename any non-abstract adapters which might actually appear in config files.

This revision is now accepted and ready to land.May 17 2015, 2:49 PM
Closed by commit rP427cc221499d: Rename protocol adapter classes (authored by joshuaspence, committed by joshuaspence). · Explain WhyMay 17 2015, 10:21 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 31000

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/infrastructure/daemon/bot/adapter/PhabricatorBaseProtocolAdapter.php

Loading...
View Options

src/infrastructure/daemon/bot/adapter/PhabricatorBotBaseStreamingProtocolAdapter.php

Loading...
View Options

src/infrastructure/daemon/bot/adapter/PhabricatorBotFlowdockProtocolAdapter.php

Loading...
View Options

src/infrastructure/daemon/bot/adapter/PhabricatorCampfireProtocolAdapter.php

Loading...
View Options

src/infrastructure/daemon/bot/adapter/PhabricatorIRCProtocolAdapter.php

Loading...
View Options

src/infrastructure/daemon/bot/adapter/PhabricatorProtocolAdapter.php

Loading...
View Options

src/infrastructure/daemon/bot/adapter/PhabricatorStreamingProtocolAdapter.php

Loading...
Phabricator · Built by Phacility