diff --git a/rhodecode/api/views/pull_request_api.py b/rhodecode/api/views/pull_request_api.py --- a/rhodecode/api/views/pull_request_api.py +++ b/rhodecode/api/views/pull_request_api.py @@ -924,7 +924,7 @@ def close_pull_request( # finally close the PR, with proper message comment comment, status = PullRequestModel().close_pull_request_with_comment( - pull_request, apiuser, repo, message=message) + pull_request, apiuser, repo, message=message, auth_user=apiuser) status_lbl = ChangesetStatus.get_status_lbl(status) Session().commit() diff --git a/rhodecode/apps/repository/views/repo_pull_requests.py b/rhodecode/apps/repository/views/repo_pull_requests.py --- a/rhodecode/apps/repository/views/repo_pull_requests.py +++ b/rhodecode/apps/repository/views/repo_pull_requests.py @@ -1195,7 +1195,8 @@ class RepoPullRequestsView(RepoAppView, 'pull request %s', pull_request_id) raise HTTPForbidden() comment, status = PullRequestModel().close_pull_request_with_comment( - pull_request, self._rhodecode_user, self.db_repo, message=text) + pull_request, self._rhodecode_user, self.db_repo, message=text, + auth_user=self._rhodecode_user) Session().flush() events.trigger( events.PullRequestCommentEvent(pull_request, comment)) diff --git a/rhodecode/model/pull_request.py b/rhodecode/model/pull_request.py --- a/rhodecode/model/pull_request.py +++ b/rhodecode/model/pull_request.py @@ -1174,7 +1174,7 @@ class PullRequestModel(BaseModel): 'repo.pull_request.close', {'data': pr_data}, user, pull_request) def close_pull_request_with_comment( - self, pull_request, user, repo, message=None): + self, pull_request, user, repo, message=None, auth_user=None): pull_request_review_status = pull_request.calculated_review_status() @@ -1198,7 +1198,8 @@ class PullRequestModel(BaseModel): pull_request=pull_request.pull_request_id, status_change=status_lbl, status_change_type=status, - closing_pr=True + closing_pr=True, + auth_user=auth_user, ) # calculate old status before we change it