mirror of
https://github.com/gitbucket/gitbucket.git
synced 2025-11-09 23:15:49 +01:00
(refs #224)Add delete branch button to pull request from same repository.
This commit is contained in:
@@ -103,6 +103,21 @@ trait PullRequestsControllerBase extends ControllerBase {
|
|||||||
} getOrElse NotFound
|
} getOrElse NotFound
|
||||||
})
|
})
|
||||||
|
|
||||||
|
get("/:owner/:repository/pull/:id/delete/:branchName")(collaboratorsOnly { repository =>
|
||||||
|
params("id").toIntOpt.map { issueId =>
|
||||||
|
val branchName = params("branchName")
|
||||||
|
val userName = context.loginAccount.get.userName
|
||||||
|
if(repository.repository.defaultBranch != branchName){
|
||||||
|
using(Git.open(getRepositoryDir(repository.owner, repository.name))){ git =>
|
||||||
|
git.branchDelete().setBranchNames(branchName).call()
|
||||||
|
recordDeleteBranchActivity(repository.owner, repository.name, userName, branchName)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
createComment(repository.owner, repository.name, userName, issueId, "Delete branch", "delete")
|
||||||
|
redirect(s"/${repository.owner}/${repository.name}/pull/${issueId}")
|
||||||
|
} getOrElse NotFound
|
||||||
|
})
|
||||||
|
|
||||||
post("/:owner/:repository/pull/:id/merge", mergeForm)(collaboratorsOnly { (form, repository) =>
|
post("/:owner/:repository/pull/:id/merge", mergeForm)(collaboratorsOnly { (form, repository) =>
|
||||||
params("id").toIntOpt.flatMap { issueId =>
|
params("id").toIntOpt.flatMap { issueId =>
|
||||||
val owner = repository.owner
|
val owner = repository.owner
|
||||||
|
|||||||
@@ -6,7 +6,7 @@
|
|||||||
@import context._
|
@import context._
|
||||||
@import view.helpers._
|
@import view.helpers._
|
||||||
@comments.map { comment =>
|
@comments.map { comment =>
|
||||||
@if(comment.action != "close" && comment.action != "reopen"){
|
@if(comment.action != "close" && comment.action != "reopen" && comment.action != "delete"){
|
||||||
<div class="issue-avatar-image">@avatar(comment.commentedUserName, 48)</div>
|
<div class="issue-avatar-image">@avatar(comment.commentedUserName, 48)</div>
|
||||||
<div class="box issue-comment-box" id="comment-@comment.commentId">
|
<div class="box issue-comment-box" id="comment-@comment.commentId">
|
||||||
<div class="box-header-small">
|
<div class="box-header-small">
|
||||||
@@ -63,6 +63,13 @@
|
|||||||
@user(comment.commentedUserName, styleClass="username strong") reopened the issue @datetime(comment.registeredDate)
|
@user(comment.commentedUserName, styleClass="username strong") reopened the issue @datetime(comment.registeredDate)
|
||||||
</div>
|
</div>
|
||||||
}
|
}
|
||||||
|
@if(comment.action == "delete"){
|
||||||
|
<div class="small issue-comment-action">
|
||||||
|
<span class="label">Deleted</span>
|
||||||
|
@avatar(comment.commentedUserName, 20)
|
||||||
|
@user(comment.commentedUserName, styleClass="username strong") deleted the <span class="label label-info monospace">@pullreq.map(_.requestBranch)</span> branch @datetime(comment.registeredDate)
|
||||||
|
</div>
|
||||||
|
}
|
||||||
}
|
}
|
||||||
<script>
|
<script>
|
||||||
$(function(){
|
$(function(){
|
||||||
|
|||||||
@@ -35,6 +35,18 @@
|
|||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
}
|
}
|
||||||
|
@if(hasWritePermission && issue.closed && pullreq.userName == pullreq.requestUserName &&
|
||||||
|
pullreq.repositoryName == pullreq.requestRepositoryName && repository.branchList.contains(pullreq.requestBranch)){
|
||||||
|
<div class="box issue-comment-box" style="background-color: #d0eeff;">
|
||||||
|
<div class="box-content"class="issue-content" style="border: 1px solid #87a8c9; padding: 10px;">
|
||||||
|
<a href="@url(repository)/pull/@issue.issueId/delete/@pullreq.requestBranch" class="btn btn-info pull-right delete-branch" data-name="@pullreq.requestBranch">Delete branch</a>
|
||||||
|
<div>
|
||||||
|
<span class="strong">Pull request successfully merged and closed</span>
|
||||||
|
</div>
|
||||||
|
<span class="small muted">You're all set-the <span class="label label-info monospace">@pullreq.requestBranch</span> branch can be safely deleted.</span>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
}
|
||||||
@issues.html.commentform(issue, hasWritePermission, repository)
|
@issues.html.commentform(issue, hasWritePermission, repository)
|
||||||
</div>
|
</div>
|
||||||
<div class="span2">
|
<div class="span2">
|
||||||
@@ -62,8 +74,12 @@ $(function(){
|
|||||||
|
|
||||||
@if(hasWritePermission){
|
@if(hasWritePermission){
|
||||||
$('.check-conflict').show();
|
$('.check-conflict').show();
|
||||||
$.get('@url(repository)/pull/@issue.issueId/mergeguide',
|
$.get('@url(repository)/pull/@issue.issueId/mergeguide', function(data){ $('.check-conflict').html(data); });
|
||||||
function(data){ $('.check-conflict').html(data); });
|
|
||||||
|
$('.delete-branch').click(function(e){
|
||||||
|
var branchName = $(e.target).data('name');
|
||||||
|
return confirm('Are you sure you want to remove the ' + branchName + ' branch?');
|
||||||
|
});
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
</script>
|
</script>
|
||||||
Reference in New Issue
Block a user