Homec4science

Kill PhabricatorObjectDataHandle

Authored by Bob Trahan <bob.trahan@gmail.com> on Sep 11 2013, 21:27.

Description

Kill PhabricatorObjectDataHandle

Summary: Ref T603. Killing this class is cool because the classes that replace it are policy-aware. Tried to keep my wits about me as I did this and fixed a few random things along the way. (Ones I remember right now are pulling a query outside of a foreach loop in Releeph and fixing the text in UIExample to note that the ace of hearts if "a powerful" card and not the "most powerful" card (Q of spades gets that honor IMO))

Test Plan: tested the first few changes (execute, executeOne X handle, object) then got real mechanical / careful with the other changes.

Reviewers: epriestley

Reviewed By: epriestley

CC: Korvin, aran, FacebookPOC

Maniphest Tasks: T603

Differential Revision: https://secure.phabricator.com/D6941

Details

Event Timeline

Bob Trahan <bob.trahan@gmail.com> committed rPHb902005bed52: Kill PhabricatorObjectDataHandle (authored by Bob Trahan <bob.trahan@gmail.com>).Sep 11 2013, 21:27