Page MenuHomePhabricator

Diffusion - fix browse file bug
ClosedPublic

Authored by btrahan on Feb 11 2014, 4:31 AM.
Tags
None
Referenced Files
F19516414: D8194.diff
Wed, Jan 14, 11:48 PM
F19516413: D8194.diff
Wed, Jan 14, 11:48 PM
F19514260: D8194.diff
Wed, Jan 14, 12:03 AM
F19174296: D8194.diff
Dec 15 2025, 10:54 PM
F19064666: D8194.id.diff
Nov 29 2025, 10:19 PM
F18817153: D8194.diff
Oct 21 2025, 1:01 PM
F18814637: D8194.id.diff
Oct 20 2025, 11:39 PM
F18809204: D8194.diff
Oct 19 2025, 11:58 AM

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!