Land Revision button for hosted git repos
Summary:
ref T182.
Simple approach of clone, patch, push. While waiting for drydock, implement a hackish mutex
setup for the workspace, which should work ok as long as there's only one committer who is
carefull about theses things.
Less obvious note: This is taking the both author and commiter's 'primary email' for the commit -
which might rub some people wrong.
Test Plan:
With a hosted repo, created some diffs and landed them.
Also clicked button for some error cases, got the right error message.
Reviewers: epriestley, #blessed_reviewers
Reviewed By: epriestley
CC: hach-que, Korvin, epriestley, aran
Maniphest Tasks: T182
Differential Revision: https://secure.phabricator.com/D7486
2013-11-05 22:00:12 +01:00
|
|
|
<?php
|
|
|
|
|
|
|
|
abstract class DifferentialLandingStrategy {
|
|
|
|
|
|
|
|
public abstract function processLandRequest(
|
|
|
|
AphrontRequest $request,
|
|
|
|
DifferentialRevision $revision,
|
|
|
|
PhabricatorRepository $repository);
|
|
|
|
|
|
|
|
/**
|
|
|
|
* returns PhabricatorActionView or an array of PhabricatorActionView or null.
|
|
|
|
*/
|
|
|
|
abstract function createMenuItems(
|
|
|
|
PhabricatorUser $viewer,
|
|
|
|
DifferentialRevision $revision,
|
|
|
|
PhabricatorRepository $repository);
|
|
|
|
|
|
|
|
/**
|
|
|
|
* returns PhabricatorActionView which can be attached to the revision view.
|
|
|
|
*/
|
|
|
|
protected function createActionView($revision, $name, $disabled = false) {
|
|
|
|
$strategy = get_class($this);
|
|
|
|
$revision_id = $revision->getId();
|
|
|
|
return id(new PhabricatorActionView())
|
|
|
|
->setRenderAsForm(true)
|
|
|
|
->setName($name)
|
|
|
|
->setHref("/differential/revision/land/{$revision_id}/{$strategy}/")
|
|
|
|
->setDisabled($disabled);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* might break if repository is not Git.
|
|
|
|
*/
|
|
|
|
protected function getGitWorkspace(PhabricatorRepository $repository) {
|
|
|
|
try {
|
|
|
|
return DifferentialGetWorkingCopy::getCleanGitWorkspace($repository);
|
|
|
|
} catch (Exception $e) {
|
2013-11-08 20:37:57 +01:00
|
|
|
throw new PhutilProxyException(
|
|
|
|
'Failed to allocate a workspace',
|
|
|
|
$e);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* might break if repository is not Mercurial.
|
|
|
|
*/
|
|
|
|
protected function getMercurialWorkspace(PhabricatorRepository $repository) {
|
|
|
|
try {
|
|
|
|
return DifferentialGetWorkingCopy::getCleanMercurialWorkspace(
|
|
|
|
$repository);
|
|
|
|
} catch (Exception $e) {
|
|
|
|
throw new PhutilProxyException(
|
Land Revision button for hosted git repos
Summary:
ref T182.
Simple approach of clone, patch, push. While waiting for drydock, implement a hackish mutex
setup for the workspace, which should work ok as long as there's only one committer who is
carefull about theses things.
Less obvious note: This is taking the both author and commiter's 'primary email' for the commit -
which might rub some people wrong.
Test Plan:
With a hosted repo, created some diffs and landed them.
Also clicked button for some error cases, got the right error message.
Reviewers: epriestley, #blessed_reviewers
Reviewed By: epriestley
CC: hach-que, Korvin, epriestley, aran
Maniphest Tasks: T182
Differential Revision: https://secure.phabricator.com/D7486
2013-11-05 22:00:12 +01:00
|
|
|
'Failed to allocate a workspace',
|
|
|
|
$e);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|