1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-22 23:02:42 +01:00

Diffusion - fix T3215

Summary: Ref T2784. We need to always return array() here so the foreach doesn't crap out

Test Plan: chad - can you confirm this fixes the error? I have ref-heavy git repos it seems. should work though.

Reviewers: chad, epriestley

Reviewed By: chad

CC: aran, Korvin

Maniphest Tasks: T3215, T2784

Differential Revision: https://secure.phabricator.com/D5961
This commit is contained in:
Bob Trahan 2013-05-17 18:16:46 -07:00
parent 0212d75632
commit 0ade49fbe4

View file

@ -35,7 +35,7 @@ final class ConduitAPI_diffusion_refsquery_Method
// similar to (remote/one, remote/two, remote/three) // similar to (remote/one, remote/two, remote/three)
$refs = trim($stdout, "() \n"); $refs = trim($stdout, "() \n");
if (!$refs) { if (!$refs) {
return null; return array();
} }
$refs = explode(',', $refs); $refs = explode(',', $refs);
$refs = array_map('trim', $refs); $refs = array_map('trim', $refs);