Use classes to define standard field types and implement an "int" type

Summary:
Currently, `ManiphestAuxiliaryFieldDefaultSpecification` uses about a dozen giant `switch` statements to implement stadard field types (int, string, date, bool, select, user, remarkup, etc). This is:

  - pretty gross;
  - not extensible; and
  - doesn't really let us share that much code.

I got about halfway through porting a similar implementation into StandardField but I wasn't thrilled with it. Subclass StandardField instead to implement custom field types.

Test Plan: Added an "int" custom field, verified it had integer semantics and indexed into the integer index.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

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

src/__phutil_library_map__.php

@@ -1635,8 +1635,10 @@ phutil_register_library_map(array(
     'PhabricatorSlugTestCase' => 'infrastructure/util/__tests__/PhabricatorSlugTestCase.php',
     'PhabricatorSortTableExample' => 'applications/uiexample/examples/PhabricatorSortTableExample.php',
     'PhabricatorSourceCodeView' => 'view/layout/PhabricatorSourceCodeView.php',
-    'PhabricatorStandardCustomField' => 'infrastructure/customfield/field/PhabricatorStandardCustomField.php',
+    'PhabricatorStandardCustomField' => 'infrastructure/customfield/standard/PhabricatorStandardCustomField.php',
+    'PhabricatorStandardCustomFieldInt' => 'infrastructure/customfield/standard/PhabricatorStandardCustomFieldInt.php',
     'PhabricatorStandardCustomFieldInterface' => 'infrastructure/customfield/interface/PhabricatorStandardCustomFieldInterface.php',
+    'PhabricatorStandardCustomFieldText' => 'infrastructure/customfield/standard/PhabricatorStandardCustomFieldText.php',
     'PhabricatorStandardPageView' => 'view/page/PhabricatorStandardPageView.php',
     'PhabricatorStatusController' => 'applications/system/PhabricatorStatusController.php',
     'PhabricatorStorageFixtureScopeGuard' => 'infrastructure/testing/fixture/PhabricatorStorageFixtureScopeGuard.php',
@@ -3783,6 +3785,8 @@ phutil_register_library_map(array(
     'PhabricatorSortTableExample' => 'PhabricatorUIExample',
     'PhabricatorSourceCodeView' => 'AphrontView',
     'PhabricatorStandardCustomField' => 'PhabricatorCustomField',
+    'PhabricatorStandardCustomFieldInt' => 'PhabricatorStandardCustomField',
+    'PhabricatorStandardCustomFieldText' => 'PhabricatorStandardCustomField',
     'PhabricatorStandardPageView' => 'PhabricatorBarePageView',
     'PhabricatorStatusController' => 'PhabricatorController',
     'PhabricatorStorageManagementDatabasesWorkflow' => 'PhabricatorStorageManagementWorkflow',

src/infrastructure/customfield/standard/PhabricatorStandardCustomField.php

@@ -1,27 +1,41 @@
 <?php
 
-final class PhabricatorStandardCustomField
+abstract class PhabricatorStandardCustomField
   extends PhabricatorCustomField {
 
   private $fieldKey;
   private $fieldName;
-  private $fieldType;
   private $fieldValue;
   private $fieldDescription;
   private $fieldConfig;
   private $applicationField;
 
+  abstract public function getFieldType();
+
   public static function buildStandardFields(
     PhabricatorCustomField $template,
     array $config) {
 
+    $types = id(new PhutilSymbolLoader())
+      ->setAncestorClass(__CLASS__)
+      ->loadObjects();
+    $types = mpull($types, null, 'getFieldType');
+
     $fields = array();
     foreach ($config as $key => $value) {
+      $type = idx($value, 'type', 'text');
+      if (empty($types[$type])) {
+        // TODO: We should have better typechecking somewhere, and then make
+        // this more serious.
+        continue;
+      }
+
       $namespace = $template->getStandardCustomFieldNamespace();
       $full_key = "std:{$namespace}:{$key}";
 
       $template = clone $template;
-      $standard = id(new PhabricatorStandardCustomField($full_key))
+      $standard = id(clone $types[$type])
+        ->setFieldKey($full_key)
         ->setFieldConfig($value)
         ->setApplicationField($template);
 
@@ -32,10 +46,6 @@ final class PhabricatorStandardCustomField
     return $fields;
   }
 
-  public function __construct($key) {
-    $this->fieldKey = $key;
-  }
-
   public function setApplicationField(
     PhabricatorStandardCustomFieldInterface $application_field) {
     $this->applicationField = $application_field;
@@ -51,15 +61,6 @@ final class PhabricatorStandardCustomField
     return $this;
   }
 
-  public function setFieldType($type) {
-    $this->fieldType = $type;
-    return $this;
-  }
-
-  public function getFieldType() {
-    return $this->fieldType;
-  }
-
   public function getFieldValue() {
     return $this->fieldValue;
   }
@@ -75,19 +76,17 @@ final class PhabricatorStandardCustomField
   }
 
   public function setFieldConfig(array $config) {
-    $this->setFieldType('text');
-
     foreach ($config as $key => $value) {
       switch ($key) {
         case 'name':
           $this->setFieldName($value);
           break;
-        case 'type':
-          $this->setFieldType($value);
-          break;
         case 'description':
           $this->setFieldDescription($value);
           break;
+        case 'type':
+          // We set this earlier on.
+          break;
       }
     }
     $this->fieldConfig = $config;
@@ -103,6 +102,11 @@ final class PhabricatorStandardCustomField
 /* -(  PhabricatorCustomField  )--------------------------------------------- */
 
 
+  public function setFieldKey($field_key) {
+    $this->fieldKey = $field_key;
+    return $this;
+  }
+
   public function getFieldKey() {
     return $this->fieldKey;
   }
@@ -136,20 +140,19 @@ final class PhabricatorStandardCustomField
   }
 
   public function readValueFromRequest(AphrontRequest $request) {
-    $this->setFieldValue($request->getStr($this->getFieldKey()));
+    $value = $request->getStr($this->getFieldKey());
+    if (!strlen($value)) {
+      $value = null;
+    }
+    $this->setFieldValue($value);
   }
 
   public function renderEditControl() {
-    $type = $this->getFieldType();
-    switch ($type) {
-      case 'text':
-      default:
     return id(new AphrontFormTextControl())
       ->setName($this->getFieldKey())
       ->setValue($this->getFieldValue())
       ->setLabel($this->getFieldName());
   }
-  }
 
   public function newStorageObject() {
     return $this->getApplicationField()->newStorageObject();
@@ -176,42 +179,20 @@ final class PhabricatorStandardCustomField
   }
 
   public function buildFieldIndexes() {
-    $type = $this->getFieldType();
+    return array();
-    switch ($type) {
-      case 'text':
-      default:
-        return array(
-          $this->newStringIndex($this->getFieldValue()),
-        );
-    }
   }
 
   public function readApplicationSearchValueFromRequest(
     PhabricatorApplicationSearchEngine $engine,
     AphrontRequest $request) {
-    $type = $this->getFieldType();
+    return;
-    switch ($type) {
-      case 'text':
-      default:
-        return $request->getStr('std:'.$this->getFieldIndex());
-    }
   }
 
   public function applyApplicationSearchConstraintToQuery(
     PhabricatorApplicationSearchEngine $engine,
     PhabricatorCursorPagedPolicyAwareQuery $query,
     $value) {
-    $type = $this->getFieldType();
+    return;
-    switch ($type) {
-      case 'text':
-      default:
-        if (strlen($value)) {
-          $query->withApplicationSearchContainsConstraint(
-            $this->newStringIndex(null),
-            $value);
-        }
-        break;
-    }
   }
 
   public function appendToApplicationSearchForm(
@@ -219,19 +200,7 @@ final class PhabricatorStandardCustomField
     AphrontFormView $form,
     $value,
     array $handles) {
-
+    return;
-    $type = $this->getFieldType();
-    switch ($type) {
-      case 'text':
-      default:
-        $form->appendChild(
-          id(new AphrontFormTextControl())
-            ->setLabel($this->getFieldName())
-            ->setName('std:'.$this->getFieldIndex())
-            ->setValue($value));
-        break;
-    }
-
   }
 
 }

src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldInt.php

@@ -0,0 +1,71 @@
+<?php
+
+final class PhabricatorStandardCustomFieldInt
+  extends PhabricatorStandardCustomField {
+
+  public function getFieldType() {
+    return 'int';
+  }
+
+  public function buildFieldIndexes() {
+    $indexes = array();
+
+    $value = $this->getFieldValue();
+    if (strlen($value)) {
+      $indexes[] = $this->newNumericIndex((int)$value);
+    }
+
+    return $indexes;
+  }
+
+  public function getValueForStorage() {
+    $value = $this->getFieldValue();
+    if (strlen($value)) {
+      return (int)$value;
+    } else {
+      return null;
+    }
+  }
+
+  public function setValueFromStorage($value) {
+    if (strlen($value)) {
+      $value = (int)$value;
+    } else {
+      $value = null;
+    }
+    return $this->setFieldValue($value);
+  }
+
+  public function readApplicationSearchValueFromRequest(
+    PhabricatorApplicationSearchEngine $engine,
+    AphrontRequest $request) {
+
+    return $request->getStr($this->getFieldKey());
+  }
+
+  public function applyApplicationSearchConstraintToQuery(
+    PhabricatorApplicationSearchEngine $engine,
+    PhabricatorCursorPagedPolicyAwareQuery $query,
+    $value) {
+
+    if (strlen($value)) {
+      $query->withApplicationSearchContainsConstraint(
+        $this->newNumericIndex(null),
+        $value);
+    }
+  }
+
+  public function appendToApplicationSearchForm(
+    PhabricatorApplicationSearchEngine $engine,
+    AphrontFormView $form,
+    $value,
+    array $handles) {
+
+    $form->appendChild(
+      id(new AphrontFormTextControl())
+        ->setLabel($this->getFieldName())
+        ->setName($this->getFieldKey())
+        ->setValue($value));
+  }
+
+}

src/infrastructure/customfield/standard/PhabricatorStandardCustomFieldText.php

@@ -0,0 +1,53 @@
+<?php
+
+final class PhabricatorStandardCustomFieldText
+  extends PhabricatorStandardCustomField {
+
+  public function getFieldType() {
+    return 'text';
+  }
+
+  public function buildFieldIndexes() {
+    $indexes = array();
+
+    $value = $this->getFieldValue();
+    if (strlen($value)) {
+      $indexes[] = $this->newStringIndex($value);
+    }
+
+    return $indexes;
+  }
+
+  public function readApplicationSearchValueFromRequest(
+    PhabricatorApplicationSearchEngine $engine,
+    AphrontRequest $request) {
+
+    return $request->getStr($this->getFieldKey());
+  }
+
+  public function applyApplicationSearchConstraintToQuery(
+    PhabricatorApplicationSearchEngine $engine,
+    PhabricatorCursorPagedPolicyAwareQuery $query,
+    $value) {
+
+    if (strlen($value)) {
+      $query->withApplicationSearchContainsConstraint(
+        $this->newStringIndex(null),
+        $value);
+    }
+  }
+
+  public function appendToApplicationSearchForm(
+    PhabricatorApplicationSearchEngine $engine,
+    AphrontFormView $form,
+    $value,
+    array $handles) {
+
+    $form->appendChild(
+      id(new AphrontFormTextControl())
+        ->setLabel($this->getFieldName())
+        ->setName($this->getFieldKey())
+        ->setValue($value));
+  }
+
+}