Page MenuHomePhabricator

Diffusion - fix browse file bug
ClosedPublic

Authored by btrahan on Feb 11 2014, 4:31 AM.
Tags
None
Referenced Files
F14063567: D8194.diff
Mon, Nov 18, 7:15 PM
F14044760: D8194.id18536.diff
Tue, Nov 12, 8:41 PM
F14006073: D8194.diff
Mon, Oct 28, 12:36 AM
F13989836: D8194.diff
Mon, Oct 21, 11:25 PM
Unknown Object (File)
Sep 26 2024, 6:45 AM
Unknown Object (File)
Sep 4 2024, 3:33 AM
Unknown Object (File)
Aug 30 2024, 5:14 AM
Unknown Object (File)
Aug 27 2024, 9:44 PM

Details

Summary

we were calling a member method on a diffusion hash. not sure why. Fixes T4402

Test Plan

clicked about, no fatals and seemed to move sensical backwards in time

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Oh, I think diffusion.commitparentsquery previously had some magic which wouldn't have survived real calls over the wire. D7808 fixed that to make it return identifiers instead, but I missed this callsite. Thanks!