diff options
| author | Rémy Coutable <remy@rymai.me> | 2016-05-12 17:36:15 +0300 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2016-05-12 17:36:15 +0300 |
| commit | a9cc14e08b1508156e62c9cf715fb67ebbfd54e4 (patch) | |
| tree | 5ee70c5aaf9e0fc12b2bf07030af7af7ee09412c /app/views/projects/commits | |
| parent | 9393d6b6f2332c0c369f7b05cb45ed639be99bda (diff) | |
| parent | d9574a7b93de7e88c77ed1351650843749da60f0 (diff) | |
Merge branch '3870-commit-sorting-issue' into 'master'
Group commits by date in server timezone
`Time#to_date` just takes the (timezone-less) year, date, and month, and
creates a new date from that. Because the commits in the list are
grouped by date, rather than chunked when the date changes, a commit can
be shown in the wrong order if its CommitDate has a timezone-less date
that's different to other commits around it.
Convert all CommitDates to the server timezone before grouping, as that
will at least produce consistent results. Users can still see a
timestamp on the commit that doesn't match the date it's grouped under,
because the timestamp shown uses the user's local timezone, and the
grouping uses the server's timezone, but that was an issue anyway.
### Before
### After
Closes #3870 and closes #17300.
See merge request !4052
Diffstat (limited to 'app/views/projects/commits')
| -rw-r--r-- | app/views/projects/commits/_commits.html.haml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/views/projects/commits/_commits.html.haml b/app/views/projects/commits/_commits.html.haml index 64e8da9201d..82f39e59284 100644 --- a/app/views/projects/commits/_commits.html.haml +++ b/app/views/projects/commits/_commits.html.haml @@ -3,7 +3,7 @@ - commits, hidden = limited_commits(@commits) -- commits.group_by { |c| c.committed_date.to_date }.sort.reverse.each do |day, commits| +- commits.group_by { |c| c.committed_date.in_time_zone.to_date }.sort.reverse.each do |day, commits| .row.commits-row .col-md-2.hidden-xs.hidden-sm %h5.commits-row-date |