Page MenuHomePhabricator

Transactions - don't bother checking for fancy transactions on comment create
ClosedPublic

Authored by btrahan on Dec 4 2014, 7:58 PM.
Tags
None
Referenced Files
F15378242: D10928.id26248.diff
Thu, Mar 13, 1:30 PM
F15378241: D10928.diff
Thu, Mar 13, 1:30 PM
F15377865: D10928.diff
Thu, Mar 13, 11:06 AM
F15343098: D10928.id26248.diff
Sun, Mar 9, 10:12 PM
F15343097: D10928.id.diff
Sun, Mar 9, 10:12 PM
F15343095: D10928.diff
Sun, Mar 9, 10:12 PM
F15338325: D10928.diff
Sun, Mar 9, 10:29 AM
Unknown Object (File)
Fri, Feb 21, 8:28 PM
Subscribers

Details

Summary

Only necessary for edits, only bother if the comment version is greater than 1. Ref T6690. This is another way to fix T6690 -- this check will never run since you can't edit a conpherence comment -- but the fix already applied should happen too to future proof Conpherence.

Test Plan

made a comment on a diff - success. edited the comment and mentions were generated.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Transactions - don't bother checking for fancy transactions on comment create.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Dec 4 2014, 8:01 PM
This revision was automatically updated to reflect the committed changes.