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
F17371022: D10128.id24355.diff
Sat, Jun 28, 3:29 PM
F17371019: D10128.id24365.diff
Sat, Jun 28, 3:29 PM
Unknown Object (File)
Wed, Jun 4, 10:16 AM
Unknown Object (File)
May 28 2025, 9:13 AM
Unknown Object (File)
May 28 2025, 8:19 AM
Unknown Object (File)
May 27 2025, 7:18 AM
Unknown Object (File)
May 19 2025, 8:19 AM
Unknown Object (File)
May 17 2025, 4:02 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).