Extend from Phobject

Summary: All classes should extend from some other class. See D13275 for some explanation.

Test Plan: `arc unit`

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

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

src/aphront/AphrontRequest.php

@@ -5,7 +5,7 @@
  * @task cookie   Managing Cookies
  * @task cluster  Working With a Phabricator Cluster
  */
-final class AphrontRequest {
+final class AphrontRequest extends Phobject {
 
   // NOTE: These magic request-type parameters are automatically included in
   // certain requests (e.g., by phabricator_form(), JX.Request,
@@ -27,6 +27,7 @@ final class AphrontRequest {
   private $user;
   private $applicationConfiguration;
   private $uriData;
+  private $cookiePrefix;
 
   public function __construct($host, $path) {
     $this->host = $host;

src/aphront/AphrontURIMapper.php

@@ -1,6 +1,6 @@
 <?php
 
-final class AphrontURIMapper {
+final class AphrontURIMapper extends Phobject {
 
   private $map;
 

src/aphront/configuration/AphrontApplicationConfiguration.php

@@ -3,7 +3,7 @@
 /**
  * @task  routing URI Routing
  */
-abstract class AphrontApplicationConfiguration {
+abstract class AphrontApplicationConfiguration extends Phobject {
 
   private $request;
   private $host;

src/aphront/response/AphrontResponse.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class AphrontResponse {
+abstract class AphrontResponse extends Phobject {
 
   private $request;
   private $cacheable = false;

src/aphront/sink/AphrontHTTPSink.php

@@ -11,7 +11,7 @@
  * @task write  Writing Response Components
  * @task emit   Emitting the Response
  */
-abstract class AphrontHTTPSink {
+abstract class AphrontHTTPSink extends Phobject {
 
 
 /* -(  Writing Response Components  )---------------------------------------- */

src/applications/aphlict/query/AphlictDropdownDataQuery.php

@@ -1,6 +1,6 @@
 <?php
 
-final class AphlictDropdownDataQuery {
+final class AphlictDropdownDataQuery extends Phobject {
 
   private $viewer;
   private $notificationData;

src/applications/audit/constants/PhabricatorAuditActionConstants.php

@@ -1,6 +1,6 @@
 <?php
 
-final class PhabricatorAuditActionConstants {
+final class PhabricatorAuditActionConstants extends Phobject {
 
   const CONCERN   = 'concern';
   const ACCEPT    = 'accept';

src/applications/audit/constants/PhabricatorAuditCommitStatusConstants.php

@@ -1,6 +1,6 @@
 <?php
 
-final class PhabricatorAuditCommitStatusConstants {
+final class PhabricatorAuditCommitStatusConstants extends Phobject {
 
   const NONE                = 0;
   const NEEDS_AUDIT         = 1;

src/applications/audit/constants/PhabricatorAuditStatusConstants.php

@@ -1,6 +1,6 @@
 <?php
 
-final class PhabricatorAuditStatusConstants {
+final class PhabricatorAuditStatusConstants extends Phobject {
 
   const NONE                    = '';
   const AUDIT_NOT_REQUIRED      = 'audit-not-required';

src/applications/audit/storage/PhabricatorAuditInlineComment.php

@@ -1,6 +1,7 @@
 <?php
 
 final class PhabricatorAuditInlineComment
+  extends Phobject
   implements PhabricatorInlineCommentInterface {
 
   private $proxy;

src/applications/auth/data/PhabricatorAuthHighSecurityToken.php

@@ -1,3 +1,3 @@
 <?php
 
-final class PhabricatorAuthHighSecurityToken {}
+final class PhabricatorAuthHighSecurityToken extends Phobject {}

src/applications/auth/provider/PhabricatorAuthProvider.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class PhabricatorAuthProvider {
+abstract class PhabricatorAuthProvider extends Phobject {
 
   private $providerConfig;
 

src/applications/base/PhabricatorApplication.php

@@ -8,7 +8,9 @@
  * @task  fact  Fact Integration
  * @task  meta  Application Management
  */
-abstract class PhabricatorApplication implements PhabricatorPolicyInterface {
+abstract class PhabricatorApplication
+  extends Phobject
+  implements PhabricatorPolicyInterface {
 
   const MAX_STATUS_ITEMS      = 100;
 

src/applications/cache/PhabricatorCaches.php

@@ -7,7 +7,7 @@
  * @task setup      Setup Cache
  * @task compress   Compression
  */
-final class PhabricatorCaches {
+final class PhabricatorCaches extends Phobject {
 
   private static $requestCache;
 

src/applications/calendar/util/CalendarTimeUtil.php

@@ -10,7 +10,7 @@
  * a Sunday -> Saturday list, whilest the profile view shows a more simple
  * seven day rolling list of events.
  */
-final class CalendarTimeUtil {
+final class CalendarTimeUtil extends Phobject {
 
   public static function getCalendarEventEpochs(
     PhabricatorUser $user,

src/applications/celerity/CelerityAPI.php

@@ -4,7 +4,7 @@
  * Indirection layer which provisions for a terrifying future where we need to
  * build multiple resource responses per page.
  */
-final class CelerityAPI {
+final class CelerityAPI extends Phobject {
 
   private static $response;
 

src/applications/celerity/CelerityResourceMap.php

@@ -6,7 +6,7 @@
  * not need to invoke it directly; instead, you call higher-level Celerity APIs
  * and it uses the resource map to satisfy your requests.
  */
-final class CelerityResourceMap {
+final class CelerityResourceMap extends Phobject {
 
   private static $instances = array();
 
@@ -16,6 +16,7 @@ final class CelerityResourceMap {
   private $packageMap;
   private $nameMap;
   private $hashMap;
+  private $componentMap;
 
   public function __construct(CelerityResources $resources) {
     $this->resources = $resources;

src/applications/celerity/CelerityResourceMapGenerator.php

@@ -1,6 +1,6 @@
 <?php
 
-final class CelerityResourceMapGenerator {
+final class CelerityResourceMapGenerator extends Phobject {
 
   private $debug = false;
   private $resources;

src/applications/celerity/CelerityResourceTransformer.php

@@ -1,6 +1,6 @@
 <?php
 
-final class CelerityResourceTransformer {
+final class CelerityResourceTransformer extends Phobject {
 
   private $minify;
   private $rawURIMap;

src/applications/celerity/CeleritySpriteGenerator.php

@@ -1,6 +1,6 @@
 <?php
 
-final class CeleritySpriteGenerator {
+final class CeleritySpriteGenerator extends Phobject {
 
   public function buildMenuSheet() {
     $sprites = array();

src/applications/celerity/CelerityStaticResourceResponse.php

@@ -5,7 +5,7 @@
  * @{function:require_celerity_resource}, and then builds appropriate HTML or
  * Ajax responses.
  */
-final class CelerityStaticResourceResponse {
+final class CelerityStaticResourceResponse extends Phobject {
 
   private $symbols = array();
   private $needsResolve = true;

src/applications/celerity/resources/CelerityResources.php

@@ -3,7 +3,7 @@
 /**
  * Defines the location of static resources.
  */
-abstract class CelerityResources {
+abstract class CelerityResources extends Phobject {
 
   private $map;
 

src/applications/conduit/call/ConduitCall.php

@@ -8,9 +8,10 @@
  *   $result = $call->execute();
  *
  */
-final class ConduitCall {
+final class ConduitCall extends Phobject {
 
   private $method;
+  private $handler;
   private $request;
   private $user;
 

src/applications/conduit/protocol/ConduitAPIRequest.php

@@ -1,6 +1,6 @@
 <?php
 
-final class ConduitAPIRequest {
+final class ConduitAPIRequest extends Phobject {
 
   protected $params;
   private $user;

src/applications/conduit/protocol/ConduitAPIResponse.php

@@ -1,6 +1,6 @@
 <?php
 
-final class ConduitAPIResponse {
+final class ConduitAPIResponse extends Phobject {
 
   private $result;
   private $errorCode;

src/applications/config/check/PhabricatorSetupCheck.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class PhabricatorSetupCheck {
+abstract class PhabricatorSetupCheck extends Phobject {
 
   private $issues;
 

src/applications/config/custom/PhabricatorConfigOptionType.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class PhabricatorConfigOptionType {
+abstract class PhabricatorConfigOptionType extends Phobject {
 
   public function validateOption(PhabricatorConfigOption $option, $value) {
     return;

src/applications/config/issue/PhabricatorSetupIssue.php

@@ -1,6 +1,6 @@
 <?php
 
-final class PhabricatorSetupIssue {
+final class PhabricatorSetupIssue extends Phobject {
 
   private $issueKey;
   private $name;

src/applications/config/json/PhabricatorConfigJSON.php

@@ -1,6 +1,6 @@
 <?php
 
-final class PhabricatorConfigJSON {
+final class PhabricatorConfigJSON extends Phobject {
   /**
    * Properly format a JSON value.
    *

src/applications/conpherence/ConpherenceTransactionRenderer.php

@@ -1,6 +1,6 @@
 <?php
 
-final class ConpherenceTransactionRenderer {
+final class ConpherenceTransactionRenderer extends Phobject {
 
   public static function renderTransactions(
     PhabricatorUser $user,

src/applications/conpherence/constants/ConpherenceConstants.php

@@ -1,3 +1,3 @@
 <?php
 
-abstract class ConpherenceConstants {}
+abstract class ConpherenceConstants extends Phobject {}

src/applications/console/core/DarkConsoleCore.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DarkConsoleCore {
+final class DarkConsoleCore extends Phobject {
 
   private $plugins = array();
   const STORAGE_VERSION = 1;

src/applications/console/plugin/DarkConsolePlugin.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class DarkConsolePlugin {
+abstract class DarkConsolePlugin extends Phobject {
 
   private $data;
   private $request;

src/applications/console/plugin/errorlog/DarkConsoleErrorLogPluginAPI.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DarkConsoleErrorLogPluginAPI {
+final class DarkConsoleErrorLogPluginAPI extends Phobject {
 
   private static $errors = array();
 

src/applications/console/plugin/xhprof/DarkConsoleXHProfPluginAPI.php

@@ -4,7 +4,7 @@
  * @phutil-external-symbol function xhprof_enable
  * @phutil-external-symbol function xhprof_disable
  */
-final class DarkConsoleXHProfPluginAPI {
+final class DarkConsoleXHProfPluginAPI extends Phobject {
 
   private static $profilerStarted;
   private static $profilerRunning;

src/applications/dashboard/layoutconfig/PhabricatorDashboardLayoutConfig.php

@@ -1,6 +1,6 @@
 <?php
 
-final class PhabricatorDashboardLayoutConfig {
+final class PhabricatorDashboardLayoutConfig extends Phobject {
 
   const MODE_FULL                = 'layout-mode-full';
   const MODE_HALF_AND_HALF       = 'layout-mode-half-and-half';

src/applications/differential/DifferentialGetWorkingCopy.php

@@ -4,7 +4,7 @@
  * Can't find a good place for this, so I'm putting it in the most notably
  * wrong place.
  */
-final class DifferentialGetWorkingCopy {
+final class DifferentialGetWorkingCopy extends Phobject {
 
   /**
    * Creates and/or cleans a workspace for the requested repo.

src/applications/differential/constants/DifferentialAction.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialAction {
+final class DifferentialAction extends Phobject {
 
   const ACTION_CLOSE          = 'commit';
   const ACTION_COMMENT        = 'none';

src/applications/differential/constants/DifferentialChangeType.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialChangeType {
+final class DifferentialChangeType extends Phobject {
 
   const TYPE_ADD        = 1;
   const TYPE_CHANGE     = 2;

src/applications/differential/constants/DifferentialLintStatus.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialLintStatus {
+final class DifferentialLintStatus extends Phobject {
 
   const LINT_NONE             = 0;
   const LINT_OKAY             = 1;

src/applications/differential/constants/DifferentialReviewerStatus.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialReviewerStatus {
+final class DifferentialReviewerStatus extends Phobject {
 
   const STATUS_BLOCKING = 'blocking';
   const STATUS_ADDED = 'added';

src/applications/differential/constants/DifferentialRevisionControlSystem.php

@@ -1,7 +1,7 @@
 <?php
 
 // TODO: Unify with similar Repository constants
-final class DifferentialRevisionControlSystem {
+final class DifferentialRevisionControlSystem extends Phobject {
 
   const SVN         = 'svn';
   const GIT         = 'git';

src/applications/differential/constants/DifferentialRevisionStatus.php

@@ -6,7 +6,7 @@
  * application.
  */
 
-final class DifferentialRevisionStatus {
+final class DifferentialRevisionStatus extends Phobject {
 
   const COLOR_STATUS_DEFAULT = 'status';
   const COLOR_STATUS_DARK = 'status-dark';

src/applications/differential/constants/DifferentialUnitStatus.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialUnitStatus {
+final class DifferentialUnitStatus extends Phobject {
 
   const UNIT_NONE             = 0;
   const UNIT_OKAY             = 1;

src/applications/differential/constants/DifferentialUnitTestResult.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialUnitTestResult {
+final class DifferentialUnitTestResult extends Phobject {
 
   const RESULT_PASS         = 'pass';
   const RESULT_FAIL         = 'fail';

src/applications/differential/landing/DifferentialLandingStrategy.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class DifferentialLandingStrategy {
+abstract class DifferentialLandingStrategy extends Phobject {
 
   abstract public function processLandRequest(
     AphrontRequest $request,

src/applications/differential/parser/DifferentialChangesetParser.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialChangesetParser {
+final class DifferentialChangesetParser extends Phobject {
 
   const HIGHLIGHT_BYTE_LIMIT = 262144;
 
@@ -55,6 +55,8 @@ final class DifferentialChangesetParser {
   private $rangeEnd;
   private $mask;
 
+  private $highlightEngine;
+
   public function setRange($start, $end) {
     $this->rangeStart = $start;
     $this->rangeEnd = $end;

src/applications/differential/parser/DifferentialCommitMessageParser.php

@@ -19,7 +19,7 @@
  * @task support  Support Methods
  * @task internal Internals
  */
-final class DifferentialCommitMessageParser {
+final class DifferentialCommitMessageParser extends Phobject {
 
   private $labelMap;
   private $titleKey;

src/applications/differential/parser/DifferentialHunkParser.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialHunkParser {
+final class DifferentialHunkParser extends Phobject {
 
   private $oldLines;
   private $newLines;

src/applications/differential/render/DifferentialChangesetRenderer.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class DifferentialChangesetRenderer {
+abstract class DifferentialChangesetRenderer extends Phobject {
 
   private $user;
   private $changeset;

src/applications/differential/render/DifferentialRawDiffRenderer.php

@@ -1,5 +1,6 @@
 <?php
-final class DifferentialRawDiffRenderer {
+
+final class DifferentialRawDiffRenderer extends Phobject {
 
   private $changesets;
   private $format = 'unified';

src/applications/differential/storage/DifferentialInlineComment.php

@@ -1,6 +1,7 @@
 <?php
 
 final class DifferentialInlineComment
+  extends Phobject
   implements PhabricatorInlineCommentInterface {
 
   private $proxy;

src/applications/differential/storage/DifferentialReviewer.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialReviewer {
+final class DifferentialReviewer extends Phobject {
 
   private $reviewerPHID;
   private $status;

src/applications/differential/view/DifferentialChangesetFileTreeSideNavBuilder.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DifferentialChangesetFileTreeSideNavBuilder {
+final class DifferentialChangesetFileTreeSideNavBuilder extends Phobject {
 
   private $title;
   private $baseURI;

src/applications/diffusion/DiffusionLintSaveRunner.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionLintSaveRunner {
+final class DiffusionLintSaveRunner extends Phobject {
   private $arc = 'arc';
   private $severity = ArcanistLintSeverity::SEVERITY_ADVICE;
   private $all = false;

src/applications/diffusion/data/DiffusionBrowseResultSet.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionBrowseResultSet {
+final class DiffusionBrowseResultSet extends Phobject {
 
   const REASON_IS_FILE              = 'is-file';
   const REASON_IS_DELETED           = 'is-deleted';

src/applications/diffusion/data/DiffusionFileContent.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionFileContent {
+final class DiffusionFileContent extends Phobject {
 
   private $corpus;
   private $blameDict;

src/applications/diffusion/data/DiffusionGitBranch.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionGitBranch {
+final class DiffusionGitBranch extends Phobject {
 
   const DEFAULT_GIT_REMOTE = 'origin';
 

src/applications/diffusion/data/DiffusionPathChange.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionPathChange {
+final class DiffusionPathChange extends Phobject {
 
   private $path;
   private $commitIdentifier;

src/applications/diffusion/data/DiffusionRepositoryPath.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionRepositoryPath {
+final class DiffusionRepositoryPath extends Phobject {
 
   private $fullPath;
   private $path;

src/applications/diffusion/data/DiffusionRepositoryTag.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionRepositoryTag {
+final class DiffusionRepositoryTag extends Phobject {
 
   private $author;
   private $epoch;

src/applications/diffusion/protocol/DiffusionMercurialWireProtocol.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionMercurialWireProtocol {
+final class DiffusionMercurialWireProtocol extends Phobject {
 
   public static function getCommandArgs($command) {
     // We need to enumerate all of the Mercurial wire commands because the

src/applications/diffusion/query/DiffusionPathQuery.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionPathQuery {
+final class DiffusionPathQuery extends Phobject {
 
   private $pathIDs;
 

src/applications/diffusion/query/DiffusionRenameHistoryQuery.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionRenameHistoryQuery {
+final class DiffusionRenameHistoryQuery extends Phobject {
 
   private $oldCommit;
   private $wasCreated;

src/applications/diffusion/query/pathchange/DiffusionPathChangeQuery.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionPathChangeQuery {
+final class DiffusionPathChangeQuery extends Phobject {
 
   private $request;
   private $limit;

src/applications/diffusion/query/pathid/DiffusionPathIDQuery.php

@@ -3,7 +3,7 @@
 /**
  * @task pathutil Path Utilities
  */
-final class DiffusionPathIDQuery {
+final class DiffusionPathIDQuery extends Phobject {
 
   public function __construct(array $paths) {
     $this->paths = $paths;

src/applications/diffusion/request/DiffusionRequest.php

@@ -7,7 +7,7 @@
  * @task new Creating Requests
  * @task uri Managing Diffusion URIs
  */
-abstract class DiffusionRequest {
+abstract class DiffusionRequest extends Phobject {
 
   protected $callsign;
   protected $path;

src/applications/diffusion/ssh/DiffusionSubversionServeSSHWorkflow.php

@@ -4,7 +4,6 @@
  * This protocol has a good spec here:
  *
  *   http://svn.apache.org/repos/asf/subversion/trunk/subversion/libsvn_ra_svn/protocol
- *
  */
 final class DiffusionSubversionServeSSHWorkflow
   extends DiffusionSubversionSSHWorkflow {

src/applications/diffusion/symbol/DiffusionExternalSymbolQuery.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DiffusionExternalSymbolQuery {
+final class DiffusionExternalSymbolQuery extends Phobject {
   private $languages = array();
   private $types = array();
   private $names = array();

src/applications/diffusion/symbol/DiffusionExternalSymbolsSource.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class DiffusionExternalSymbolsSource {
+abstract class DiffusionExternalSymbolsSource extends Phobject {
 
   /**
    * @return list of PhabricatorRepositorySymbol

src/applications/diviner/atom/DivinerAtom.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DivinerAtom {
+final class DivinerAtom extends Phobject {
 
   const TYPE_ARTICLE   = 'article';
   const TYPE_CLASS     = 'class';

src/applications/diviner/atom/DivinerAtomRef.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DivinerAtomRef {
+final class DivinerAtomRef extends Phobject {
 
   private $book;
   private $context;

src/applications/diviner/atomizer/DivinerAtomizer.php

@@ -3,7 +3,7 @@
 /**
  * Generate @{class:DivinerAtom}s from source code.
  */
-abstract class DivinerAtomizer {
+abstract class DivinerAtomizer extends Phobject {
 
   private $book;
   private $fileName;

src/applications/diviner/cache/DivinerDiskCache.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class DivinerDiskCache {
+abstract class DivinerDiskCache extends Phobject {
 
   private $cache;
 

src/applications/diviner/publisher/DivinerPublisher.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class DivinerPublisher {
+abstract class DivinerPublisher extends Phobject {
 
   private $atomCache;
   private $atomGraphHashToNodeHashMap;

src/applications/diviner/renderer/DivinerRenderer.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class DivinerRenderer {
+abstract class DivinerRenderer extends Phobject {
 
   private $publisher;
   private $atomStack = array();

src/applications/doorkeeper/engine/DoorkeeperFeedStoryPublisher.php

@@ -3,7 +3,7 @@
 /**
  * @task config Configuration
  */
-abstract class DoorkeeperFeedStoryPublisher {
+abstract class DoorkeeperFeedStoryPublisher extends Phobject {
 
   private $feedStory;
   private $viewer;

src/applications/drydock/blueprint/DrydockBlueprintImplementation.php

@@ -5,7 +5,7 @@
  * @task resource   Resource Allocation
  * @task log        Logging
  */
-abstract class DrydockBlueprintImplementation {
+abstract class DrydockBlueprintImplementation extends Phobject {
 
   private $activeResource;
   private $activeLease;

src/applications/drydock/constants/DrydockConstants.php

@@ -1,3 +1,3 @@
 <?php
 
-abstract class DrydockConstants {}
+abstract class DrydockConstants extends Phobject {}

src/applications/drydock/interface/DrydockInterface.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class DrydockInterface {
+abstract class DrydockInterface extends Phobject {
 
   private $config;
 

src/applications/drydock/util/DrydockBlueprintScopeGuard.php

@@ -1,6 +1,6 @@
 <?php
 
-final class DrydockBlueprintScopeGuard {
+final class DrydockBlueprintScopeGuard extends Phobject {
 
   public function __construct(DrydockBlueprintImplementation $blueprint) {
     $this->blueprint = $blueprint;

src/applications/fact/engine/PhabricatorFactEngine.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class PhabricatorFactEngine {
+abstract class PhabricatorFactEngine extends Phobject {
 
   final public static function loadAllEngines() {
     $classes = id(new PhutilSymbolLoader())

src/applications/fact/spec/PhabricatorFactSpec.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class PhabricatorFactSpec {
+abstract class PhabricatorFactSpec extends Phobject {
 
   const UNIT_COUNT = 'unit-count';
   const UNIT_EPOCH = 'unit-epoch';

src/applications/feed/PhabricatorFeedStoryPublisher.php

@@ -1,6 +1,6 @@
 <?php
 
-final class PhabricatorFeedStoryPublisher {
+final class PhabricatorFeedStoryPublisher extends Phobject {
 
   private $relatedPHIDs;
   private $storyType;

src/applications/feed/builder/PhabricatorFeedBuilder.php

@@ -1,6 +1,6 @@
 <?php
 
-final class PhabricatorFeedBuilder {
+final class PhabricatorFeedBuilder extends Phobject {
 
   private $stories;
   private $framed;

src/applications/feed/story/PhabricatorFeedStory.php

@@ -9,6 +9,7 @@
  * @task policy   Policy Implementation
  */
 abstract class PhabricatorFeedStory
+  extends Phobject
   implements
     PhabricatorPolicyInterface,
     PhabricatorMarkupInterface {

src/applications/files/PhabricatorImageTransformer.php

@@ -4,7 +4,7 @@
  * @task enormous Detecting Enormous Images
  * @task save     Saving Image Data
  */
-final class PhabricatorImageTransformer {
+final class PhabricatorImageTransformer extends Phobject {
 
   public function executeMemeTransform(
     PhabricatorFile $file,

src/applications/files/engine/PhabricatorFileStorageEngine.php

@@ -14,7 +14,7 @@
  * @task file Managing File Data
  * @task load Loading Storage Engines
  */
-abstract class PhabricatorFileStorageEngine {
+abstract class PhabricatorFileStorageEngine extends Phobject {
 
   /**
    * Construct a new storage engine.

src/applications/files/query/PhabricatorFileBundleLoader.php

@@ -4,7 +4,7 @@
  * Callback provider for loading @{class@arcanist:ArcanistBundle} file data
  * stored in the Files application.
  */
-final class PhabricatorFileBundleLoader {
+final class PhabricatorFileBundleLoader extends Phobject {
 
   private $viewer;
 

src/applications/flag/constants/PhabricatorFlagConstants.php

@@ -1,3 +1,3 @@
 <?php
 
-abstract class PhabricatorFlagConstants {}
+abstract class PhabricatorFlagConstants extends Phobject {}

src/applications/harbormaster/step/HarbormasterBuildStepImplementation.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class HarbormasterBuildStepImplementation {
+abstract class HarbormasterBuildStepImplementation extends Phobject {
 
   public static function getImplementations() {
     return id(new PhutilSymbolLoader())

src/applications/herald/adapter/HeraldAdapter.php

@@ -3,7 +3,7 @@
 /**
  * @task customfield Custom Field Integration
  */
-abstract class HeraldAdapter {
+abstract class HeraldAdapter extends Phobject {
 
   const FIELD_TITLE                  = 'title';
   const FIELD_BODY                   = 'body';

src/applications/herald/config/HeraldRepetitionPolicyConfig.php

@@ -1,6 +1,6 @@
 <?php
 
-final class HeraldRepetitionPolicyConfig {
+final class HeraldRepetitionPolicyConfig extends Phobject {
 
   const FIRST   = 'first';  // only execute the first time (no repeating)
   const EVERY   = 'every';  // repeat every time

src/applications/herald/config/HeraldRuleTypeConfig.php

@@ -1,6 +1,6 @@
 <?php
 
-final class HeraldRuleTypeConfig {
+final class HeraldRuleTypeConfig extends Phobject {
 
   const RULE_TYPE_GLOBAL = 'global';
   const RULE_TYPE_OBJECT = 'object';

src/applications/herald/engine/HeraldEffect.php

@@ -1,6 +1,6 @@
 <?php
 
-final class HeraldEffect {
+final class HeraldEffect extends Phobject {
 
   private $objectPHID;
   private $action;

src/applications/herald/engine/HeraldEngine.php

@@ -1,14 +1,15 @@
 <?php
 
-final class HeraldEngine {
+final class HeraldEngine extends Phobject {
 
   protected $rules = array();
   protected $results = array();
   protected $stack = array();
-  protected $activeRule = null;
+  protected $activeRule;
+  protected $transcript;
 
   protected $fieldCache = array();
-  protected $object = null;
+  protected $object;
   private $dryRun;
 
   public function setDryRun($dry_run) {

src/applications/herald/extension/HeraldCustomAction.php

@@ -1,6 +1,6 @@
 <?php
 
-abstract class HeraldCustomAction {
+abstract class HeraldCustomAction extends Phobject {
 
   abstract public function appliesToAdapter(HeraldAdapter $adapter);
 

src/applications/herald/storage/transcript/HeraldConditionTranscript.php

@@ -1,6 +1,6 @@
 <?php
 
-final class HeraldConditionTranscript {
+final class HeraldConditionTranscript extends Phobject {
 
   protected $ruleID;
   protected $conditionID;

src/applications/herald/storage/transcript/HeraldObjectTranscript.php

@@ -1,6 +1,6 @@
 <?php
 
-final class HeraldObjectTranscript {
+final class HeraldObjectTranscript extends Phobject {
 
   protected $phid;
   protected $type;