revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-11-26 08:42:40 +01:00
Code Issues Releases Wiki Activity

Fix invalid parameter in phutil_console_wrap()

Browse source 
Summary:
Fixes T8314. Change the phutil_console_wrap() call to only have a single
string parameter.

Test Plan:
Ran `arc diff` added text into the title, summary and test plan fields,
but did not specify any reviewers. When prompted to continue, clicked 'No' and
saw the '(Message saved to commit message.)' string.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley

Maniphest Tasks: T8314

Differential Revision: https://secure.phabricator.com/D13015
This commit is contained in:
Nevogd 2015-05-26 06:07:02 -07:00 committed by epriestley
parent 64d03ff68b
commit 407af00ef6
1 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/workflow/ArcanistDiffWorkflow.php
View file

@ -1724,11 +1724,7 @@ EOTEXT
} }
} catch (Exception $ex) { } catch (Exception $ex) {
if ($wrote) { if ($wrote) {
echo phutil_console_wrap( echo phutil_console_wrap(pht('(Message saved to %s.)', $where)."\n");
"%s\n",
pht(
'(Message saved to %s.)',
$where));
} }
throw $ex; throw $ex;
} }