Page MenuHomePhabricator

Remove all application-specific reply handler class overrides
ClosedPublic

Authored by epriestley on Apr 1 2015, 12:01 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 1:36 AM
Unknown Object (File)
Thu, Apr 11, 9:18 AM
Unknown Object (File)
Tue, Apr 9, 12:01 AM
Unknown Object (File)
Apr 2 2024, 6:56 AM
Unknown Object (File)
Mar 29 2024, 6:02 AM
Unknown Object (File)
Mar 28 2024, 1:46 AM
Unknown Object (File)
Mar 27 2024, 10:03 PM
Unknown Object (File)
Mar 10 2024, 4:02 PM
Subscribers

Details

Summary

Ref T7199. In the vein of D12231, these options were a bad idea.

  • They once served a very narrow, Facebook-specific need (see T1992), except even Facebook only used the Differential setting AFAIK.
  • Outside of that special case, they are unused and essentially unusable (generally speaking, they do not meaningfully implement anything modular or replaceable).
  • I have no knowledge of any install ever changing these settings, and can imagine no reason why they would.

Moving forward:

  • If they really need to, they can fork locally and chagne one line.
  • I expect "!actions" to make mail at least somewhat more modular soon, anyway.
  • Any derived handlers would break after T7199 and need to be rewritten anyway, so this is just taking advantage of a BC break to do cleanup.
Test Plan
  • Grepped for removed configuration.
  • Sent some mail from applications, verified the reply handlers set proper reply addresses.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Remove all application-specific reply handler class overrides.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
This revision is now accepted and ready to land.Apr 1 2015, 12:10 AM
This revision was automatically updated to reflect the committed changes.