##// END OF EJS Templates
pull-request: verify resolve TODO comment needs to be bound to the same PR as we're calling
marcink -
r2441:9a41d4d3 default
parent child Browse files
Show More
@@ -224,8 +224,14 b' class CommentsModel(BaseModel):'
224
224
225 comment.repo = repo
225 comment.repo = repo
226 comment.author = user
226 comment.author = user
227 comment.resolved_comment = self.__get_commit_comment(
227 resolved_comment = self.__get_commit_comment(
228 validated_kwargs['resolves_comment_id'])
228 validated_kwargs['resolves_comment_id'])
229 # check if the comment actually belongs to this PR
230 if resolved_comment and resolved_comment.pull_request and \
231 resolved_comment.pull_request != pull_request:
232 # comment not bound to this pull request, forbid
233 resolved_comment = None
234 comment.resolved_comment = resolved_comment
229
235
230 pull_request_id = pull_request
236 pull_request_id = pull_request
231
237
General Comments 0
You need to be logged in to leave comments. Login now