Code

gitweb: Unify the way long timestamp is displayed
authorJakub Narebski <jnareb@gmail.com>
Thu, 28 Apr 2011 19:04:08 +0000 (21:04 +0200)
committerJunio C Hamano <gitster@pobox.com>
Tue, 24 May 2011 18:22:45 +0000 (11:22 -0700)
format_timestamp_html loses its "-localtime => 1" option, and now
always print the local time (in author/comitter/tagger local
timezone), with "atnight" warning if needed.

This means that both 'summary' and 'log' views now display localtime.
In the case of 'log' view this can be thought as an improvement, as
now one can easily see which commits in a series are made "atnight"
and should be examined closer.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
gitweb/gitweb.perl

index 7329db289c102abbf8ecaf9eb50fa513b720fa19..67bcfe894ed5ae5d25f00fb03e045ab3ae433503 100755 (executable)
@@ -3939,11 +3939,9 @@ sub git_print_section {
 }
 
 sub format_timestamp_html {
-       my ($date, %opts) = @_;
+       my $date = shift;
        my $strtime = $date->{'rfc2822'};
 
-       return $strtime unless $opts{'-localtime'};
-
        my $localtime_format = '(%02d:%02d %s)';
        if ($date->{'hour_local'} < 6) {
                $localtime_format = '(<span class="atnight">%02d:%02d</span> %s)';
@@ -3965,7 +3963,7 @@ sub git_print_authorship {
        my %ad = parse_date($co->{'author_epoch'}, $co->{'author_tz'});
        print "<$tag class=\"author_date\">" .
              format_search_author($author, "author", esc_html($author)) .
-             " [".format_timestamp_html(\%ad, %opts)."]".
+             " [".format_timestamp_html(\%ad)."]".
              git_get_avatar($co->{'author_email'}, -pad_before => 1) .
              "</$tag>\n";
 }
@@ -3992,7 +3990,7 @@ sub git_print_authorship_rows {
                      "</td></tr>\n" .
                      "<tr>" .
                      "<td></td><td>" .
-                     format_timestamp_html(\%wd, -localtime=>1) .
+                     format_timestamp_html(\%wd) .
                      "</td>" .
                      "</tr>\n";
        }