use Date::Parse qw(strptime str2time);
use PublicInbox::Hval;
use PublicInbox::GitCatFile;
+use PublicInbox::View;
use constant {
- DATEFMT => '%Y-%m-%dT%H:%M:%SZ',
- MAX_PER_PAGE => 25,
+ DATEFMT => '%Y-%m-%dT%H:%M:%SZ', # atom standard
+ MAX_PER_PAGE => 25, # this needs to be tunable
+ PRE_WRAP => "<pre\nstyle=\"white-space:pre-wrap\">",
};
-# FIXME: workaround https://rt.cpan.org/Public/Bug/Display.html?id=22817
-
# main function
sub generate {
my ($class, $args) = @_;
require XML::Atom::SimpleFeed;
- require PublicInbox::View;
require POSIX;
my $max = $args->{max} || MAX_PER_PAGE;
my ($add) = @_;
add_to_feed($feed_opts, $feed, $add, $git);
});
+ $git = undef; # destroy pipes
+ Email::Address->purge_cache;
$feed->as_string;
}
sub generate_html_index {
my ($class, $args) = @_;
- require Mail::Thread;
+ require PublicInbox::Thread;
my $max = $args->{max} || MAX_PER_PAGE;
my $feed_opts = get_feedopts($args);
$title = PublicInbox::Hval->new_oneline($title)->as_html;
my @messages;
- my $git = PublicInbox::GitCatFile->new($args->{git_dir});
- my $last = each_recent_blob($args, sub {
- my $mime = do_cat_mail($git, $_[0]) or return 0;
- $mime->body_set(''); # save some memory
-
- my $t = eval { str2time($mime->header('Date')) };
- defined($t) or $t = 0;
- $mime->header_set('X-PI-Date', $t);
- push @messages, $mime;
- 1;
+ my $git_dir = $args->{git_dir};
+ my $git = PublicInbox::GitCatFile->new($git_dir);
+ my ($first, $last) = each_recent_blob($args, sub {
+ mime_load_for_sort($git, $_[0], \@messages);
});
+ $git = undef; # destroy pipes.
- my $th = Mail::Thread->new(@messages);
+ my $th = PublicInbox::Thread->new(@messages);
$th->thread;
my $html = "<html><head><title>$title</title>" .
- '<link rel="alternate" title="Atom feed" href="' .
- $feed_opts->{atomurl} . '" type="application/atom+xml"/>' .
- '</head><body><pre>';
-
- # sort by date, most recent at top
- $th->order(sub {
- sort {
- $b->topmost->message->header('X-PI-Date') <=>
- $a->topmost->message->header('X-PI-Date')
- } @_;
- });
- dump_html_line($_, 0, \$html) for $th->rootset;
+ '<link rel="alternate" title="Atom feed"' . "\nhref=\"" .
+ $feed_opts->{atomurl} . "\"\ntype=\"application/atom+xml\"/>" .
+ '</head><body>' . PRE_WRAP;
+
+ # sort child messages in chronological order
+ $th->order(*PublicInbox::Thread::sort_ts);
+
+ # except we sort top-level messages reverse chronologically
+ my $state = [ time, {}, $first, 0 ];
+ for (PublicInbox::Thread::rsort_ts($th->rootset)) {
+ dump_msg($_, 0, \$html, $state)
+ }
+ Email::Address->purge_cache;
- my $footer = nav_footer($args->{cgi}, $last);
- $footer = "<hr /><pre>$footer</pre>" if $footer;
- $html . "</pre>$footer</html>";
+ my $footer = nav_footer($args->{cgi}, $last, $feed_opts, $state);
+ if ($footer) {
+ my $list_footer = $args->{footer};
+ $footer .= "\n" . $list_footer if $list_footer;
+ $footer = "<hr />" . PRE_WRAP . "$footer</pre>";
+ }
+ $html . "</pre>$footer</body></html>";
}
# private subs
sub nav_footer {
- my ($cgi, $last) = @_;
+ my ($cgi, $last, $feed_opts, $state) = @_;
$cgi or return '';
my $old_r = $cgi->param('r');
my $head = ' ';
my $next = ' ';
+ my $first = $state->[2];
+ my $anchor = $state->[3];
if ($last) {
- $next = qq!<a href="?r=$last">next</a>!;
+ $next = qq!<a\nhref="?r=$last">next</a>!;
}
if ($old_r) {
$head = $cgi->path_info;
- $head = qq!<a href="$head">head</a>!;
+ $head = qq!<a\nhref="$head">head</a>!;
}
- "$next $head";
+ my $atom = "<a\nhref=\"$feed_opts->{atomurl}\">atom</a>";
+ my $permalink = "<a\nhref=\"?r=$first\">permalink</a>";
+ "<a\nname=\"s$anchor\">page:</a> $next $head $atom $permalink";
}
sub each_recent_blob {
# we could use git log -z, but, we already know ssoma will not
# leave us with filenames with spaces in them..
my @cmd = ('git', "--git-dir=$args->{git_dir}",
- qw/log --no-notes --no-color --raw -r/);
+ qw/log --no-notes --no-color --raw -r
+ --abbrev=16 --abbrev-commit/);
push @cmd, $range;
my $pid = open(my $log, '-|', @cmd) or
}
} elsif ($line =~ /$delmsg/o) {
$deleted{$1} = 1;
- } elsif ($line =~ /^commit (${hex}{40})/) {
+ } elsif ($line =~ /^commit (${hex}{7,40})/o) {
push @commits, $1;
}
}
if ($last) {
while (my $line = <$log>) {
- if ($line =~ /^commit (${hex}{40})/) {
+ if ($line =~ /^commit (${hex}{7,40})/o) {
push @commits, $1;
last;
}
close $log; # we may EPIPE here
# for pagination
- $commits[-1];
+ ($commits[0], $commits[-1]);
}
# private functions below
defined $mid or return 0;
$mid = PublicInbox::Hval->new_msgid($mid);
my $href = $mid->as_href . '.html';
- my $content = PublicInbox::View->as_feed_entry($mime, $fullurl . $href);
+ my $content = PublicInbox::View->feed_entry($mime, $fullurl . $href);
defined($content) or return 0;
my $subject = mime_header($mime, 'Subject') or return 0;
1;
}
-sub dump_html_line {
- my ($self, $level, $html) = @_;
- if ($self->message) {
- $$html .= (' ' x $level);
- my $mime = $self->message;
- my $subj = $mime->header('Subject');
- my $mid = $mime->header_obj->header_raw('Message-ID');
- $mid = PublicInbox::Hval->new_msgid($mid);
- my $href = 'm/' . $mid->as_href . '.html';
- my $from = mime_header($mime, 'From');
-
- my @from = Email::Address->parse($from);
- $from = $from[0]->name;
- (defined($from) && length($from)) or $from = $from[0]->address;
-
- $from = PublicInbox::Hval->new_oneline($from)->as_html;
- $subj = PublicInbox::Hval->new_oneline($subj)->as_html;
- $$html .= "<a href=\"$href\">$subj</a> $from\n";
+sub dump_msg {
+ my ($self, $level, $html, $state) = @_;
+ my $mime = $self->message;
+ if ($mime) {
+ $$html .= PublicInbox::View->index_entry($mime, $level, $state);
}
- dump_html_line($self->child, $level+1, $html) if $self->child;
- dump_html_line($self->next, $level, $html) if $self->next;
+ dump_msg($self->child, $level+1, $html, $state) if $self->child;
+ dump_msg($self->next, $level, $html, $state) if $self->next;
}
sub do_cat_mail {
my ($git, $path) = @_;
- my $str = $git->cat_file("HEAD:$path");
- Email::MIME->new($$str);
+ my $mime = eval {
+ my $str = $git->cat_file("HEAD:$path");
+ Email::MIME->new($str);
+ };
+ $@ ? undef : $mime;
+}
+
+sub mime_load_for_sort {
+ my ($git, $path, $messages) = @_;
+ my $mime = do_cat_mail($git, $path) or return 0;
+
+ my $t = eval { str2time($mime->header('Date')) };
+ defined($t) or $t = 0;
+ $mime->header_set('X-PI-TS', $t);
+ push @$messages, $mime;
+ 1;
}
1;