Page MenuHomePhabricator

D8837.id20992.diff
No OneTemporary

D8837.id20992.diff

diff --git a/resources/sql/patches/20131004.dxreviewers.php b/resources/sql/patches/20131004.dxreviewers.php
--- a/resources/sql/patches/20131004.dxreviewers.php
+++ b/resources/sql/patches/20131004.dxreviewers.php
@@ -29,6 +29,13 @@
->setActor(PhabricatorUser::getOmnipotentUser());
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/facebook/phabricator/pull/507
+ continue;
+ }
+
$editor->addEdge(
$revision_phid,
PhabricatorEdgeConfig::TYPE_DREV_HAS_REVIEWER,

File Metadata

Mime Type
text/plain
Expires
Mon, Jun 3, 6:33 PM (3 w, 1 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6299791
Default Alt Text
D8837.id20992.diff (743 B)

Event Timeline