Page MenuHomePhabricator

Make "DifferentialDiff->properties" a proper "attachable" property
ClosedPublic

Authored by epriestley on Jul 26 2021, 6:44 PM.
Tags
None
Referenced Files
F13094370: D21710.id51717.diff
Thu, Apr 25, 7:15 AM
F13089848: D21710.diff
Thu, Apr 25, 2:02 AM
Unknown Object (File)
Tue, Apr 23, 11:48 PM
Unknown Object (File)
Wed, Apr 17, 7:13 AM
Unknown Object (File)
Sun, Apr 7, 12:12 AM
Unknown Object (File)
Thu, Mar 28, 11:16 PM
Unknown Object (File)
Thu, Mar 28, 7:04 PM
Unknown Object (File)
Wed, Mar 27, 12:48 PM
Subscribers
Restricted Owners Package

Details

Summary

See PHI498. This should be initialized to "self::ATTACHABLE" like other attachable properties, but is currently initialized to "array()".

Initialize it the normal way and try to catch all code paths which may have accessed it without actually loading and attaching it.

Also, remove UI for the very old "excuse" property, which "arc" has not written for well more than a year.

Test Plan

Grepped for affected symbols, loaded various revision pages. Somewhat tricky to be 100% sure that every pathway is caught, but it should be obvious if I missed anything once someone hits the code path.

Diff Detail

Repository
rP Phabricator
Branch
dprop1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 25520
Build 35283: Run Core Tests
Build 35282: arc lint + arc unit

Event Timeline

Owners added a subscriber: Restricted Owners Package.Jul 26 2021, 6:44 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 27 2021, 8:39 PM
This revision was automatically updated to reflect the committed changes.