Page MenuHomePhabricator

Transactions - fix bug in differential where older inlines weren't appearing
ClosedPublic

Authored by btrahan on Dec 5 2014, 6:19 PM.
Tags
None
Referenced Files
F13049123: D10935.id26267.diff
Fri, Apr 19, 1:34 AM
Unknown Object (File)
Thu, Apr 11, 9:05 AM
Unknown Object (File)
Wed, Apr 10, 9:08 AM
Unknown Object (File)
Mon, Apr 1, 4:00 AM
Unknown Object (File)
Fri, Mar 22, 5:26 PM
Unknown Object (File)
Mar 12 2024, 4:28 PM
Unknown Object (File)
Feb 5 2024, 8:28 PM
Unknown Object (File)
Feb 1 2024, 4:56 PM
Subscribers

Details

Summary

Fixes T6699. We need to "loadInlineComments" consistently, though for an unexpected reason - this mutates the $changesets to include all $changesets that have an associated inline comment, which is necessary to make them render properly.

Test Plan

Took a diff with inline comments and updated it, noting the inline comments disappeared. applied this patch and the inlines reappeared.

Diff Detail

Repository
rP Phabricator
Branch
T6699
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3185
Build 3191: [Placeholder Plan] Wait for 30 Seconds