Page MenuHomePhabricator

Apply normal Audit actions directly with Transaction editor
ClosedPublic

Authored by epriestley on Aug 2 2014, 7:29 PM.
Tags
None
Referenced Files
F14060754: D10128.diff
Mon, Nov 18, 3:17 AM
F14047327: D10128.diff
Thu, Nov 14, 3:32 AM
F14037377: D10128.diff
Sun, Nov 10, 3:31 PM
F14021373: D10128.diff
Wed, Nov 6, 8:03 AM
F13978644: D10128.diff
Oct 19 2024, 12:16 AM
F13973973: D10128.diff
Oct 18 2024, 3:28 AM
F13964416: D10128.id24365.diff
Oct 15 2024, 8:19 PM
F13959047: D10128.id.diff
Oct 14 2024, 5:37 PM
Subscribers

Details

Summary

Ref T4896. This converts the last "CommentEditor" to a transaction editor and removes a large part of the old code.

Test Plan
  • Added comments.
  • Accepted / added auditors.
  • Added inline comments.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Apply normal Audit actions directly with Transaction editor.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: btrahan, joshuaspence.
btrahan edited edge metadata.

awesometown

src/applications/audit/controller/PhabricatorAuditAddCommentController.php
86–88

surprised this logic isn't a method on commit or something.

This revision is now accepted and ready to land.Aug 2 2014, 9:09 PM
epriestley updated this revision to Diff 24365.

Closed by commit rPb5750412c75b (authored by @epriestley).