Page MenuHomePhabricator

Fix bad tables in diffusion blame
ClosedPublic

Authored by chad on Aug 25 2017, 2:53 AM.
Tags
None
Referenced Files
F18112303: D18474.diff
Tue, Aug 12, 4:20 PM
F18090027: D18474.id.diff
Wed, Aug 6, 1:46 PM
F18089868: D18474.id44382.diff
Wed, Aug 6, 1:03 PM
F18089358: D18474.id44381.diff
Wed, Aug 6, 11:18 AM
F18087116: D18474.diff
Wed, Aug 6, 2:23 AM
F17680111: D18474.diff
Jul 13 2025, 3:11 PM
Unknown Object (File)
Jun 18 2025, 6:19 PM
Unknown Object (File)
Jun 5 2025, 2:58 PM
Subscribers

Details

Summary

My fake data was 100%, and not all tables have full revision history. This leads to a broken table. Instead check if we have any revisions at all, then always show the column, with or without a link inside.

Test Plan

going on a limb this is the correct fix and test on secure... again ...

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable