events: properly refresh comment object to load it's relationship.

This prevents from issues of reading associated statuses of comment during
a partial commit (using Session().flush)

- Fixes  #5412

rhodecode/apps/repository/views/repo_pull_requests.py

@@ -1159,6 +1159,10 b' class RepoPullRequestsView(RepoAppView, '
                     )
 
                 Session().flush()
+                # this is somehow required to get access to some relationship
+                # loaded on comment
+                Session().refresh(comment)
+
                 events.trigger(
                     events.PullRequestCommentEvent(pull_request, comment))
 

rhodecode/model/changeset_status.py

@@ -18,16 +18,14 b''
 # RhodeCode Enterprise Edition, including its added features, Support services,
 # and proprietary license terms, please see https://rhodecode.com/licenses/
 
-"""
-Changeset status conttroller
-"""
 
 import itertools
 import logging
 from collections import defaultdict
 
 from rhodecode.model import BaseModel
-from rhodecode.model.db import ChangesetStatus, ChangesetComment, PullRequest
+from rhodecode.model.db import (
+    ChangesetStatus, ChangesetComment, PullRequest, Session)
 from rhodecode.lib.exceptions import StatusChangeOnClosedPullRequestError
 from rhodecode.lib.markup_renderer import (
     DEFAULT_COMMENTS_RENDERER, RstTemplateRenderer)
@@ -188,7 +186,7 b' class ChangesetStatusModel(BaseModel):'
         if cur_statuses:
             for st in cur_statuses:
                 st.version += 1
-                self.sa.add(st)
+                Session().add(st)
 
         def _create_status(user, repo, status, comment, revision, pull_request):
             new_status = ChangesetStatus()
@@ -215,7 +213,7 b' class ChangesetStatusModel(BaseModel):'
             new_status = _create_status(
                 user=user, repo=repo, status=status, comment=comment,
                 revision=revision, pull_request=pull_request)
-            self.sa.add(new_status)
+            Session().add(new_status)
             return new_status
         elif pull_request:
             # pull request can have more than one revision associated to it
@@ -227,7 +225,7 b' class ChangesetStatusModel(BaseModel):'
                     user=user, repo=repo, status=status, comment=comment,
                     revision=rev, pull_request=pull_request)
                 new_statuses.append(new_status)
-                self.sa.add(new_status)
+                Session().add(new_status)
             return new_statuses
 
     def reviewers_statuses(self, pull_request):

rhodecode/model/comment.py

@@ -432,7 +432,8 b' class CommentsModel(BaseModel):'
                     pull_request_id=pull_request.pull_request_id,
                     _anchor='comment-%s' % comment.comment_id)
             else:
-                return request.route_url('pullrequest_show',
+                return request.route_url(
+                    'pullrequest_show',
                     repo_name=safe_str(pull_request.target_repo.repo_name),
                     pull_request_id=pull_request.pull_request_id,
                     _anchor='comment-%s' % comment.comment_id)