Page MenuHomePhabricator

Correct an issue where the commit list could group commits by server time
ClosedPublic

Authored by epriestley on Jun 2 2017, 11:49 AM.
Tags
None
Referenced Files
F19061333: D18067.id43456.diff
Sat, Nov 29, 12:06 PM
F19051912: D18067.id.diff
Fri, Nov 28, 2:24 AM
F19040490: D18067.diff
Wed, Nov 26, 12:36 PM
F18855025: D18067.diff
Nov 1 2025, 2:19 AM
F18795782: D18067.id43456.diff
Oct 17 2025, 2:27 AM
F18789204: D18067.id43453.diff
Oct 15 2025, 8:31 AM
F18735144: D18067.diff
Sep 30 2025, 11:50 PM
F18126979: D18067.id.diff
Aug 13 2025, 3:10 PM
Subscribers
None

Details

Summary

Commits in the list are grouped by the date they occurred in server time. This may not be the date they occurred in client time.

Use client time, not server time, to group commits.

Test Plan
  • Set server timezone to "Asia/Famagusta".
  • Set client timezone to "America/Los_Angeles".
  • Viewed Phabricator repository history.

Here's what it looks like before the change:

Screen Shot 2017-06-02 at 4.45.03 AM.png (1×1 px, 239 KB)

Note that the headers of the first two groups both say "Yesterday".

This is because the first commits in each group occurred on June 1 and June 2, respectively, in Famagusta, but both occurred on June 1 in Los Angeles.

Here's what it looks like after the change:

Screen Shot 2017-06-02 at 4.47.41 AM.png (1×1 px, 239 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

src/applications/diffusion/view/DiffusionHistoryListView.php
26–29

(These variables were not used.)

This revision is now accepted and ready to land.Jun 2 2017, 1:41 PM
This revision was automatically updated to reflect the committed changes.