HomePhabricator

Remove several pieces of audit-related code

Description

Remove several pieces of audit-related code

Summary: Ref T10978. This code (mostly related to the old ADD_AUDIT transaction and some to the "store English text in the database" audit reasons) is no longer reachable.

Test Plan:
Grepped for removed symbols:

  • withAuditStatus
  • getActionNameMap (unrelated callsites exist)
  • getActionName (unrelated callsites exist)
  • getActionPastTenseVerb
  • addAuditReason
  • getAuditReasons
  • auditReasonMap

Also audited some commits.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10978

Differential Revision: https://secure.phabricator.com/D17267

Details

Provenance
epriestleyAuthored on Jan 26 2017, 6:32 PM
epriestleyPushed on Jan 30 2017, 11:26 PM
Reviewer
chad
Differential Revision
D17267: Remove several pieces of audit-related code
Parents
rP2e9cc5e8e8a3: Make implicit audits by the Owners tool use modern code
Branches
Unknown
Tags
Unknown
Tasks
T10978: Modernize Audit
Build Status
Buildable 15423
Build 20323: Run Core Tests