Homec4science

Hide Audit comment table reads behind an API

Authored by epriestley <git@epriestley.com> on Jul 25 2014, 03:00.

Description

Hide Audit comment table reads behind an API

Summary: Ref T4896. Buries all direct access to the table so we can limit the surface area affected by the migration.

Test Plan:

  • Grepped for PhabricatorAuditComment.
  • Grepped for audit_comment.
  • Viewed a bunch of comments.
  • Added a comment.
  • Reindexed a commit.
  • Searched for unique term in new comment.

Reviewers: btrahan, joshuaspence

Reviewed By: joshuaspence

Subscribers: epriestley

Maniphest Tasks: T4896

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

Details

Committed
epriestley <git@epriestley.com>Jul 25 2014, 03:00
Pushed
aubortJan 31 2017, 17:16
Parents
rPHc01aa794c1f2: Migrate Audit inline comments to new storage
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHdc5c87f74cda: Hide Audit comment table reads behind an API (authored by epriestley <git@epriestley.com>).Jul 25 2014, 03:00