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
F13084663: D10128.diff
Wed, Apr 24, 11:15 PM
Unknown Object (File)
Mon, Apr 22, 8:53 AM
Unknown Object (File)
Thu, Apr 18, 4:56 PM
Unknown Object (File)
Thu, Apr 11, 7:49 AM
Unknown Object (File)
Fri, Mar 29, 6:59 PM
Unknown Object (File)
Thu, Mar 28, 2:48 PM
Unknown Object (File)
Mar 10 2024, 11:17 AM
Unknown Object (File)
Mar 10 2024, 11:17 AM
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).