Page MenuHomePhabricator

Remove legacy transaction editor "getDisableEmail()" method
ClosedPublic

Authored by epriestley on Jan 19 2018, 3:48 PM.
Tags
None
Referenced Files
F13221702: D18881.diff
Sun, May 19, 2:58 AM
F13207685: D18881.id45273.diff
Thu, May 16, 2:27 AM
F13206302: D18881.diff
Wed, May 15, 6:52 AM
F13204167: D18881.diff
Wed, May 15, 12:23 AM
F13186990: D18881.diff
Sat, May 11, 4:15 AM
Unknown Object (File)
Tue, May 7, 7:20 AM
Unknown Object (File)
Fri, May 3, 5:56 AM
Unknown Object (File)
Thu, May 2, 8:28 AM
Subscribers
None

Details

Summary

Ref T13042. This is a very, very old policy-violating option from yesteryear which supported build systems publishing updates by adding comments to revisions, without sending email about it.

Harbormaster has served this role for a long time and this is policy-violating in the general case (it allows attackers to act in secret).

Test Plan

Grepped for affected symbols.

Diff Detail

Repository
rP Phabricator
Branch
bulk16
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19057
Build 25719: Run Core Tests
Build 25718: arc lint + arc unit