From 0b7f760a41947e7ee07f1d56045edee0d5e9f9fe Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 16 Jan 2013 09:00:20 -0800 Subject: [PATCH] Base class of `differential.field-selector` should be DifferentialFieldSelector, not DifferentialDefaultFieldSelector Summary: Missed this -- the "Default" flavor extends from the actual abstract base. Test Plan: Loaded setup issues, no longer saw an issue raised for my local extension class. Reviewers: codeblock Reviewed By: codeblock CC: aran Maniphest Tasks: T2255 Differential Revision: https://secure.phabricator.com/D4463 --- .../config/PhabricatorDifferentialConfigOptions.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/applications/differential/config/PhabricatorDifferentialConfigOptions.php b/src/applications/differential/config/PhabricatorDifferentialConfigOptions.php index e2356622cb..4bf32bb1b7 100644 --- a/src/applications/differential/config/PhabricatorDifferentialConfigOptions.php +++ b/src/applications/differential/config/PhabricatorDifferentialConfigOptions.php @@ -50,7 +50,7 @@ final class PhabricatorDifferentialConfigOptions 'differential.field-selector', 'class', 'DifferentialDefaultFieldSelector') - ->setBaseClass('DifferentialDefaultFieldSelector') + ->setBaseClass('DifferentialFieldSelector') ->setDescription(pht('Field selector class')), $this->newOption('differential.show-host-field', 'bool', false) ->setBoolOptions(