archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Minor, fix an issue in Diffusion/Herald with commits that have no Auditors.

Browse Source
This commit is contained in:
epriestley
2012-07-24 07:37:41 -07:00
parent 6cef6658a3
commit 1e9120cdfb
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/applications/repository/worker/PhabricatorRepositoryCommitHeraldWorker.php
View File

@@ -222,7 +222,7 @@ final class PhabricatorRepositoryCommitHeraldWorker
$matches = null; $matches = null;
if (!preg_match('/^Auditors:\s*(.*)$/im', $message, $matches)) { if (!preg_match('/^Auditors:\s*(.*)$/im', $message, $matches)) {
return; return array();
} }
$phids = DifferentialFieldSpecification::parseCommitMessageObjectList( $phids = DifferentialFieldSpecification::parseCommitMessageObjectList(
@@ -231,7 +231,7 @@ final class PhabricatorRepositoryCommitHeraldWorker
$allow_partial = true); $allow_partial = true);
if (!$phids) { if (!$phids) {
return; return array();
} }
$requests = id(new PhabricatorRepositoryAuditRequest())->loadAllWhere( $requests = id(new PhabricatorRepositoryAuditRequest())->loadAllWhere(