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
F14353537: D10412.diff
Thu, Dec 19, 4:40 PM
Unknown Object (File)
Thu, Nov 28, 1:44 AM
Unknown Object (File)
Nov 19 2024, 1:47 AM
Unknown Object (File)
Nov 8 2024, 11:09 PM
Unknown Object (File)
Nov 5 2024, 4:37 AM
Unknown Object (File)
Oct 30 2024, 10:13 PM
Unknown Object (File)
Oct 22 2024, 1:25 PM
Unknown Object (File)
Oct 22 2024, 10:51 AM

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
Branch
file3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 2429
Build 2433: [Placeholder Plan] Wait for 30 Seconds

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).