Page MenuHomePhabricator

Remove all "originalTitle"/"originalName" fields from objects
ClosedPublic

Authored by epriestley on Feb 7 2018, 11:35 AM.
Tags
None
Referenced Files
F15478527: D19013.diff
Tue, Apr 8, 12:40 AM
F15465403: D19013.diff
Wed, Apr 2, 6:16 PM
F15462778: D19013.id45609.diff
Tue, Apr 1, 5:36 PM
F15438667: D19013.id45582.diff
Wed, Mar 26, 2:31 AM
F15433112: D19013.id45582.diff
Mon, Mar 24, 9:25 PM
F15430711: D19013.id.diff
Mon, Mar 24, 8:37 AM
F15429246: D19013.diff
Mon, Mar 24, 1:20 AM
F15412567: D19013.diff
Mar 19 2025, 1:43 PM
Subscribers
None

Details

Summary

Depends on D19012. Ref T13053. In D19012, I've changed "Thread-Topic" to always use PHIDs.

This change drops the selective on-object storage we have to track the original, human-readable title for objects.

Even if we end up backing out the "Thread-Topic" change, we'd be better off storing this in a table in the Mail app which just has <objectPHID, first subject we used when sending mail for that object>, since then we get the right behavior without needing every object to have this separate field.

Test Plan

Grepped for original, originalName, originalTitle, etc.

Diff Detail

Repository
rP Phabricator
Branch
mailer11
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19376
Build 26202: Run Core Tests
Build 26201: arc lint + arc unit