Page MenuHomePhabricator

Don't require an actor in `PhabricatorFile::attachToObject()`
ClosedPublic

Authored by epriestley on Sep 4 2014, 4:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 11:03 PM
Unknown Object (File)
Wed, Apr 17, 2:01 PM
Unknown Object (File)
Fri, Apr 12, 6:49 AM
Unknown Object (File)
Wed, Mar 27, 6:58 AM
Unknown Object (File)
Mar 4 2024, 6:30 AM
Unknown Object (File)
Mar 4 2024, 6:30 AM
Unknown Object (File)
Feb 2 2024, 3:32 AM
Unknown Object (File)
Dec 27 2023, 8:10 PM

Details

Summary

Ref T6013. A very long time ago, edges were less clearly low-level infrastructure, and some user-aware stuff got built around edge edits.

This was kind of a mess and I eventually removed it, during or prior to T5245. The big issue was that control flow was really hard to figure out as things went all the way down to the deepest level of infrastructure and then came back up the stack to events and transactions. The new stuff is more top-down and generally seems a lot easier and cleaner.

Consequently, actors are no longer required for edge edits. Remove the parameter.

Test Plan

Poked around; ran unit tests.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Don't require an actor in `PhabricatorFile::attachToObject()`.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Sep 4 2014, 7:30 PM
epriestley updated this revision to Diff 25069.

Closed by commit rP3af442e4ac94 (authored by @epriestley).