mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-23 15:22:41 +01:00
7cab903943
Summary: Modernize Differential edges to subclass `PhabricatorEdgeType`. Largely based on D11045. Test Plan: From previous experience, these changes are fairly trivial and safe. I poked around a little to make sure things looked reasonably okay. Reviewers: epriestley, #blessed_reviewers Reviewed By: epriestley, #blessed_reviewers Subscribers: Korvin, Krenair, epriestley Differential Revision: https://secure.phabricator.com/D11074
51 lines
1.3 KiB
PHP
51 lines
1.3 KiB
PHP
<?php
|
|
|
|
$table = new DifferentialRevision();
|
|
$conn_w = $table->establishConnection('w');
|
|
|
|
// NOTE: We migrate by revision because the relationship table doesn't have
|
|
// an "id" column.
|
|
|
|
foreach (new LiskMigrationIterator($table) as $revision) {
|
|
$revision_id = $revision->getID();
|
|
$revision_phid = $revision->getPHID();
|
|
|
|
echo "Migrating reviewers for D{$revision_id}...\n";
|
|
|
|
$reviewer_phids = queryfx_all(
|
|
$conn_w,
|
|
'SELECT objectPHID FROM %T WHERE revisionID = %d
|
|
AND relation = %s ORDER BY sequence',
|
|
'differential_relationship',
|
|
$revision_id,
|
|
'revw');
|
|
$reviewer_phids = ipull($reviewer_phids, 'objectPHID');
|
|
|
|
if (!$reviewer_phids) {
|
|
continue;
|
|
}
|
|
|
|
$editor = new PhabricatorEdgeEditor();
|
|
foreach ($reviewer_phids as $dst) {
|
|
if (phid_get_type($dst) == PhabricatorPHIDConstants::PHID_TYPE_UNKNOWN) {
|
|
// At least one old install ran into some issues here. Skip the row if we
|
|
// can't figure out what the destination PHID is. See here:
|
|
// https://github.com/phacility/phabricator/pull/507
|
|
continue;
|
|
}
|
|
|
|
$editor->addEdge(
|
|
$revision_phid,
|
|
DifferentialRevisionHasReviewerEdgeType::EDGECONST,
|
|
$dst,
|
|
array(
|
|
'data' => array(
|
|
'status' => DifferentialReviewerStatus::STATUS_ADDED,
|
|
),
|
|
));
|
|
}
|
|
|
|
$editor->save();
|
|
}
|
|
|
|
echo "Done.\n";
|