revi-archive/phorge-arcanist
1
0
Fork 0
mirror of https://we.phorge.it/source/arcanist.git synced 2024-12-29 17:00:58 +01:00
Code Issues Releases Wiki Activity

(stable) Promote 2017 Week 40

Browse source
This commit is contained in:
epriestley 2017-10-06 14:19:31 -07:00
commit 282b83bcbe
2 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/repository/api/ArcanistGitAPI.php
View file

@ -497,6 +497,10 @@ final class ArcanistGitAPI extends ArcanistRepositoryAPI {
return null;
}
if (!strlen($branch)) {
return null;
}
return $branch;
}
@ -509,7 +513,7 @@ final class ArcanistGitAPI extends ArcanistRepositoryAPI {
// Verify this, and strip it.
$ref = rtrim($stdout);
$branch = $this->getBranchNameFromRef($ref);
if (!$branch) {
if ($branch === null) {
throw new Exception(
pht('Failed to parse %s output!', 'git symbolic-ref'));
}
@ -1015,7 +1019,7 @@ final class ArcanistGitAPI extends ArcanistRepositoryAPI {
list($ref, $hash, $epoch, $tree, $desc, $text) = $fields;
$branch = $this->getBranchNameFromRef($ref);
if ($branch) {
if ($branch !== null) {
$result[] = array(
'current' => ($branch === $current),
'name' => $branch,

2
src/workflow/ArcanistLandWorkflow.php
View file

@ -444,7 +444,7 @@ EOTEXT
$branch = $this->getArgument('branch');
if (empty($branch)) {
$branch = $this->getBranchOrBookmark();
if ($branch) {
if ($branch !== null) {
$this->branchType = $this->getBranchType($branch);
// TODO: This message is misleading when landing a detached head or