Page MenuHomePhabricator

Diffusion - fix browse file bug
ClosedPublic

Authored by btrahan on Feb 11 2014, 4:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 9:36 AM
Unknown Object (File)
Wed, May 1, 9:05 PM
Unknown Object (File)
Sat, Apr 27, 11:36 PM
Unknown Object (File)
Sat, Apr 27, 11:35 PM
Unknown Object (File)
Sat, Apr 27, 11:35 PM
Unknown Object (File)
Sat, Apr 27, 11:35 PM
Unknown Object (File)
Thu, Apr 25, 2:48 AM
Unknown Object (File)
Mon, Apr 15, 4:11 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!