Added application config options 'github.status.commit' and
'bitbucket.status.commit' for allow/deny to post commit status.
This commit is contained in:
parent
eecf3ea3ce
commit
daac392dd7
|
@ -272,6 +272,9 @@ class InstallCommand extends Command
|
|||
'commit' => false,
|
||||
'pull_request' => false,
|
||||
],
|
||||
'status' => [
|
||||
'commit' => false,
|
||||
],
|
||||
],
|
||||
'github' => [
|
||||
'token' => null,
|
||||
|
@ -279,6 +282,9 @@ class InstallCommand extends Command
|
|||
'commit' => false,
|
||||
'pull_request' => false,
|
||||
],
|
||||
'status' => [
|
||||
'commit' => false,
|
||||
],
|
||||
],
|
||||
'build' => [
|
||||
'remove_builds' => true,
|
||||
|
|
|
@ -64,6 +64,15 @@ class BitbucketBuild extends RemoteGitBuild
|
|||
return false;
|
||||
}
|
||||
|
||||
$allowStatusCommit = (boolean)Config::getInstance()->get(
|
||||
'php-censor.bitbucket.status.commit',
|
||||
false
|
||||
);
|
||||
|
||||
if (!$allowStatusCommit) {
|
||||
return false;
|
||||
}
|
||||
|
||||
switch ($this->getStatus()) {
|
||||
case 0:
|
||||
case 1:
|
||||
|
@ -224,8 +233,16 @@ class BitbucketBuild extends RemoteGitBuild
|
|||
$lineStart = null,
|
||||
$lineEnd = null
|
||||
) {
|
||||
$allowCommentCommit = (boolean)Config::getInstance()->get('php-censor.bitbucket.comments.commit', false);
|
||||
$allowCommentPullRequest = (boolean)Config::getInstance()->get('php-censor.bitbucket.comments.pull_request', false);
|
||||
$allowCommentCommit = (boolean)Config::getInstance()->get(
|
||||
'php-censor.bitbucket.comments.commit',
|
||||
false
|
||||
);
|
||||
|
||||
$allowCommentPullRequest = (boolean)Config::getInstance()->get(
|
||||
'php-censor.bitbucket.comments.pull_request',
|
||||
false
|
||||
);
|
||||
|
||||
//$file = $builder->buildPath.'test.php';
|
||||
if ($allowCommentCommit || $allowCommentPullRequest) {
|
||||
$diffLineNumber = $this->getDiffLineNumber($builder, $file, $lineStart);
|
||||
|
|
|
@ -56,11 +56,19 @@ class GithubBuild extends RemoteGitBuild
|
|||
}
|
||||
|
||||
$token = Config::getInstance()->get('php-censor.github.token');
|
||||
|
||||
if (empty($token) || empty($this->data['id'])) {
|
||||
return false;
|
||||
}
|
||||
|
||||
$allowStatusCommit = (boolean)Config::getInstance()->get(
|
||||
'php-censor.github.status.commit',
|
||||
false
|
||||
);
|
||||
|
||||
if (!$allowStatusCommit) {
|
||||
return false;
|
||||
}
|
||||
|
||||
switch ($this->getStatus()) {
|
||||
case 0:
|
||||
case 1:
|
||||
|
@ -207,8 +215,15 @@ class GithubBuild extends RemoteGitBuild
|
|||
$lineStart = null,
|
||||
$lineEnd = null
|
||||
) {
|
||||
$allowCommentCommit = (boolean)Config::getInstance()->get('php-censor.github.comments.commit', false);
|
||||
$allowCommentPullRequest = (boolean)Config::getInstance()->get('php-censor.github.comments.pull_request', false);
|
||||
$allowCommentCommit = (boolean)Config::getInstance()->get(
|
||||
'php-censor.github.comments.commit',
|
||||
false
|
||||
);
|
||||
|
||||
$allowCommentPullRequest = (boolean)Config::getInstance()->get(
|
||||
'php-censor.github.comments.pull_request',
|
||||
false
|
||||
);
|
||||
|
||||
if ($allowCommentCommit || $allowCommentPullRequest) {
|
||||
$diffLineNumber = $this->getDiffLineNumber($builder, $file, $lineStart);
|
||||
|
@ -220,9 +235,6 @@ class GithubBuild extends RemoteGitBuild
|
|||
$prNumber = $this->getExtra('pull_request_number');
|
||||
$commit = $this->getCommitId();
|
||||
|
||||
$allowCommentCommit = (boolean)Config::getInstance()->get('php-censor.github.comments.commit', false);
|
||||
$allowCommentPullRequest = (boolean)Config::getInstance()->get('php-censor.github.comments.pull_request', false);
|
||||
|
||||
if (!empty($prNumber)) {
|
||||
if ($allowCommentPullRequest) {
|
||||
$helper->createPullRequestComment($repo, $prNumber, $commit, $file, $diffLineNumber, $message);
|
||||
|
|
|
@ -130,7 +130,6 @@ class BuildService
|
|||
|
||||
/** @var Build $build */
|
||||
$build = $this->buildStore->save($build);
|
||||
|
||||
$buildId = $build->getId();
|
||||
|
||||
if (!empty($buildId)) {
|
||||
|
|
Loading…
Reference in a new issue