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 @@ -130,7 +130,8 @@ class PullRequestModel(BaseModel): } def __get_pull_request(self, pull_request): - return self._get_instance(PullRequest, pull_request) + return self._get_instance(( + PullRequest, PullRequestVersion), pull_request) def _check_perms(self, perms, pull_request, user, api=False): if not api: @@ -705,7 +706,7 @@ class PullRequestModel(BaseModel): version.title = pull_request.title version.description = pull_request.description version.status = pull_request.status - version.created_on = pull_request.created_on + version.created_on = datetime.datetime.now() version.updated_on = pull_request.updated_on version.user_id = pull_request.user_id version.source_repo = pull_request.source_repo diff --git a/rhodecode/tests/models/test_pullrequest.py b/rhodecode/tests/models/test_pullrequest.py --- a/rhodecode/tests/models/test_pullrequest.py +++ b/rhodecode/tests/models/test_pullrequest.py @@ -764,7 +764,10 @@ def test_create_version_from_snapshot_up assert version.title == pr_util.create_parameters['title'] assert version.description == pr_util.create_parameters['description'] assert version.status == PullRequest.STATUS_CLOSED - assert version.created_on == created_on + + # versions get updated created_on + assert version.created_on != created_on + assert version.updated_on == updated_on assert version.user_id == pull_request.user_id assert version.revisions == pr_util.create_parameters['revisions']