Page MenuHomePhabricator
Differential D12563

Rename protocol adapter classes
ClosedPublic
Actions

Authored by joshuaspence on Apr 27 2015, 9:20 AM.
Tags
None
Referenced Files
F14013301: D12563.diff
Sat, Nov 2, 4:04 AM
F14009206: D12563.id31000.diff
Wed, Oct 30, 12:23 PM
F14005828: D12563.diff
Sun, Oct 27, 8:24 PM
F13985981: D12563.id30160.diff
Sun, Oct 20, 11:56 PM
F13975773: D12563.diff
Oct 18 2024, 11:33 AM
F13956439: D12563.id.diff
Oct 14 2024, 5:39 AM
Unknown Object (File)
Oct 13 2024, 5:13 PM
Unknown Object (File)
Oct 9 2024, 1:28 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