revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2025-02-02 09:58:23 +01:00
Code Issues Releases Wiki Activity

Provide user-friendly error in arc diff --update

Browse source 
Summary:
If you forget to provide an argument for --update and have another argument
following it (e.g. HEAD^), we should provide a nice error message instead
of passing that argument through to a conduit call and then printing the
conduit error.

Test Plan: ran 'arc diff --update HEAD^' and got a nice error message

Reviewers: epriestley, jungejason, vrana

Reviewed By: epriestley

CC: aran, Koolvin

Differential Revision: https://secure.phabricator.com/D2468
This commit is contained in:
Nick Harper 2012-05-11 18:40:02 -07:00
parent a31fc544c8
commit 032b9b30b0
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/workflow/diff/ArcanistDiffWorkflow.php
View file

@ -1278,7 +1278,12 @@ EOTEXT
return $this->getCommitMessageFromUser();
}
} else if ($is_update) {
return $this->getCommitMessageFromRevision($is_update);
$revision_id = $this->normalizeRevisionID($is_update);
if (!is_numeric($revision_id)) {
throw new ArcanistUsageException(
'Parameter to --update must be a Differential Revision number');
}
return $this->getCommitMessageFromRevision($revision_id);
} else {
// This is --raw without enough info to create a revision, so force just
// a diff.
@ -1484,7 +1489,7 @@ EOTEXT
* @task message
*/
private function getCommitMessageFromRevision($revision_id) {
$id = $this->normalizeRevisionID($revision_id);
$id = $revision_id;
$revision = $this->getConduit()->callMethodSynchronous(
'differential.query',