Page MenuHomePhabricator

Load and attach objects when loading application transactions
ClosedPublic

Authored by epriestley on Feb 18 2014, 11:58 PM.
Tags
None
Referenced Files
F14405424: D8273.id19686.diff
Mon, Dec 23, 7:46 AM
F14403791: D8273.diff
Mon, Dec 23, 3:27 AM
Unknown Object (File)
Wed, Dec 18, 7:19 PM
Unknown Object (File)
Tue, Dec 17, 7:26 PM
Unknown Object (File)
Mon, Dec 16, 8:13 PM
Unknown Object (File)
Thu, Dec 5, 12:20 PM
Unknown Object (File)
Nov 20 2024, 11:34 AM
Unknown Object (File)
Nov 16 2024, 9:50 AM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP3103ce33b82d: Load and attach objects when loading application transactions
Summary

Ref T3886. Fixes the removed TODO. This also implements the generally reasonable policy "you have to be able to see an object in order to see its transactions". That was implicit before (we never load transactions without loading an object first) but is now explicit.

This fixes bad (nonspecialized) rendering of custom field transactions in Projects, and shortly in Differential, where stories would read "alincoln edited this object." instead of a more specific string.

Test Plan

Viewed a project edit, saw a more specific string. Browed ApplicationTransaction applications.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped