delta/gitlab/gitlab-ce.git/app/models/merge_request_diff.rb, branch fix/serialized-commit-path

Absorb gitlab_git

2017-01-05T00:09:28+00:00

Replace references to MergeRequestDiff#commits with st_commits

2016-12-01T11:17:30+00:00

when we care only about the number of commits

We do not have to instantiate all objects in this case.

Hide empty merge request diffs when comparing versions

2016-11-21T12:26:26+00:00

Also keep commits from source_project around, feedback:

2016-10-24T16:08:30+00:00

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6658#note_17190236

Refactor straight compare diff code

2016-10-12T13:32:57+00:00

Signed-off-by: Dmitriy Zaporozhets

Improve tests for merge request diff model

2016-10-12T13:32:57+00:00

Signed-off-by: Dmitriy Zaporozhets

Use straight diff approach when compare merge request versions

2016-10-12T13:32:57+00:00

Signed-off-by: Dmitriy Zaporozhets

Fix Error 500 when viewing old merge requests with bad diff data

2016-10-10T20:05:09+00:00

Customers running old versions of GitLab may have MergeRequestDiffs with
the text ["--broken diff"] due to text generated by gitlab_git 1.0.3.
To avoid the Error 500, verify that each element is a type that gitlab_git
will accept before attempting to create a DiffCollection.

Closes #20776

So that st_commits could never be nil

2016-09-22T09:28:36+00:00

For empty merge_request_diff st_commits would be nil

2016-09-22T09:01:01+00:00

Closes #22438