Add a PHID to Changesets

Summary:
Ref T13605. Changesets currently have no PHID, which limits their ability to use standard API infrastructure.

Give them a PHID, since there's no reason they don't have one other than their age.

Test Plan:
  - Ran migrations, saw PHIDs populated.
  - Created new changesets, saw PHIDs.

Subscribers: PHID-OPKG-gm6ozazyms6q6i22gyam

Maniphest Tasks: T13605

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

Author: epriestley

resources/sql/autopatches/20210215.changeset.01.phid.sql

@@ -0,0 +1,2 @@
+ALTER TABLE {$NAMESPACE}_differential.differential_changeset
+  ADD phid VARBINARY(64) NOT NULL;

resources/sql/autopatches/20210215.changeset.02.phid-populate.php

@@ -0,0 +1,26 @@
+<?php
+
+$phid_type = DifferentialChangesetPHIDType::TYPECONST;
+
+$changeset_table = new DifferentialChangeset();
+
+$conn = $changeset_table->establishConnection('w');
+$table_name = $changeset_table->getTableName();
+
+$iterator = new LiskRawMigrationIterator($conn, $table_name);
+foreach ($iterator as $changeset_row) {
+  $phid = $changeset_row['phid'];
+
+  if (strlen($phid)) {
+    continue;
+  }
+
+  $phid = PhabricatorPHID::generateNewPHID($phid_type);
+
+  queryfx(
+    $conn,
+    'UPDATE %T SET phid = %s WHERE id = %d',
+    $table_name,
+    $phid,
+    $changeset_row['id']);
+}

src/__phutil_library_map__.php

@@ -472,6 +472,7 @@
     'DifferentialChangesetOneUpMailRenderer' => 'applications/differential/render/DifferentialChangesetOneUpMailRenderer.php',
     'DifferentialChangesetOneUpRenderer' => 'applications/differential/render/DifferentialChangesetOneUpRenderer.php',
     'DifferentialChangesetOneUpTestRenderer' => 'applications/differential/render/DifferentialChangesetOneUpTestRenderer.php',
+    'DifferentialChangesetPHIDType' => 'applications/differential/phid/DifferentialChangesetPHIDType.php',
     'DifferentialChangesetParser' => 'applications/differential/parser/DifferentialChangesetParser.php',
     'DifferentialChangesetParserTestCase' => 'applications/differential/parser/__tests__/DifferentialChangesetParserTestCase.php',
     'DifferentialChangesetQuery' => 'applications/differential/query/DifferentialChangesetQuery.php',
@@ -6541,6 +6542,7 @@
     'DifferentialChangesetOneUpMailRenderer' => 'DifferentialChangesetRenderer',
     'DifferentialChangesetOneUpRenderer' => 'DifferentialChangesetHTMLRenderer',
     'DifferentialChangesetOneUpTestRenderer' => 'DifferentialChangesetTestRenderer',
+    'DifferentialChangesetPHIDType' => 'PhabricatorPHIDType',
     'DifferentialChangesetParser' => 'Phobject',
     'DifferentialChangesetParserTestCase' => 'PhabricatorTestCase',
     'DifferentialChangesetQuery' => 'PhabricatorCursorPagedPolicyAwareQuery',

src/applications/differential/phid/DifferentialChangesetPHIDType.php

@@ -0,0 +1,41 @@
+<?php
+
+final class DifferentialChangesetPHIDType extends PhabricatorPHIDType {
+
+  const TYPECONST = 'DCNG';
+
+  public function getTypeName() {
+    return pht('Differential Changeset');
+  }
+
+  public function newObject() {
+    return new DifferentialChangeset();
+  }
+
+  public function getPHIDTypeApplicationClass() {
+    return 'PhabricatorDifferentialApplication';
+  }
+
+  protected function buildQueryForObjects(
+    PhabricatorObjectQuery $query,
+    array $phids) {
+
+    return id(new DifferentialChangesetQuery())
+      ->withPHIDs($phids);
+  }
+
+  public function loadHandles(
+    PhabricatorHandleQuery $query,
+    array $handles,
+    array $objects) {
+
+    foreach ($handles as $phid => $handle) {
+      $changeset = $objects[$phid];
+
+      $id = $changeset->getID();
+
+      $handle->setName(pht('Changeset %d', $id));
+    }
+  }
+
+}

src/applications/differential/storage/DifferentialChangeset.php

@@ -45,6 +45,7 @@ final class DifferentialChangeset
 
   protected function getConfiguration() {
     return array(
+      self::CONFIG_AUX_PHID => true,
       self::CONFIG_SERIALIZATION => array(
         'metadata'      => self::SERIALIZATION_JSON,
         'oldProperties' => self::SERIALIZATION_JSON,
@@ -75,6 +76,10 @@ protected function getConfiguration() {
     ) + parent::getConfiguration();
   }
 
+  public function getPHIDType() {
+    return DifferentialChangesetPHIDType::TYPECONST;
+  }
+
   public function getAffectedLineCount() {
     return $this->getAddLines() + $this->getDelLines();
   }