From dfe5ae08aabcf00aa2f5fce50f6d32835ab2ce51 Mon Sep 17 00:00:00 2001 From: vrana Date: Thu, 4 Oct 2012 16:50:12 -0700 Subject: [PATCH] Handle empty argv in daemon log view Summary: This could be empty probably for old logs Test Plan: Ran the controller. Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D3633 --- .../daemon/controller/PhabricatorDaemonLogViewController.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/applications/daemon/controller/PhabricatorDaemonLogViewController.php b/src/applications/daemon/controller/PhabricatorDaemonLogViewController.php index 26f751937f..16622c2cbb 100644 --- a/src/applications/daemon/controller/PhabricatorDaemonLogViewController.php +++ b/src/applications/daemon/controller/PhabricatorDaemonLogViewController.php @@ -41,7 +41,9 @@ final class PhabricatorDaemonLogViewController $content = array(); $argv = $log->getArgv(); - $argv = implode("\n", $argv); + if ($argv) { + $argv = implode("\n", $argv); + } $form = id(new AphrontFormView()) ->setUser($user)