require PublicInbox::View;
require POSIX;
my $max = $args->{max} || MAX_PER_PAGE;
- my $top = $args->{top}; # bool
local $ENV{GIT_DIR} = $args->{git_dir};
my $feed_opts = get_feedopts($args);
my $git = try_git_pm($args->{git_dir});
each_recent_blob($args, sub {
my ($add) = @_;
- add_to_feed($feed_opts, $feed, $add, $top, $git);
+ add_to_feed($feed_opts, $feed, $add, $git);
});
$feed->as_string;
}
require Mail::Thread;
my $max = $args->{max} || MAX_PER_PAGE;
- my $top = $args->{top}; # bool
local $ENV{GIT_DIR} = $args->{git_dir};
my $feed_opts = get_feedopts($args);
my @messages;
my $git = try_git_pm($args->{git_dir});
- my ($first, $last) = each_recent_blob($args, sub {
- my $mime = do_cat_mail($git, $_[0])
- or return 0;
- if ($top && ($mime->header('In-Reply-To') ||
- $mime->header('References'))) {
- return 0;
- }
+ 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')) };
my $th = Mail::Thread->new(@messages);
$th->thread;
- my @out = (
- "<html><head><title>$title</title>" .
- '<link rel=alternate title=Atom.feed href="' .
+ my $html = "<html><head><title>$title</title>" .
+ '<link rel="alternate" title="Atom feed" href="' .
$feed_opts->{atomurl} . '" type="application/atom+xml"/>' .
- '</head><body><pre>');
- push @out, $feed_opts->{midurl};
+ '</head><body><pre>';
# sort by date, most recent at top
$th->order(sub {
$a->topmost->message->header('X-PI-Date')
} @_;
});
- dump_html_line($_, 0, \@out) for $th->rootset;
+ dump_html_line($_, 0, \$html) for $th->rootset;
- my $footer = nav_footer($args->{cgi}, $first, $last);
+ my $footer = nav_footer($args->{cgi}, $last);
$footer = "<hr /><pre>$footer</pre>" if $footer;
- $out[0] . "</pre>$footer</html>";
+ $html . "</pre>$footer</html>";
}
# private subs
sub nav_footer {
- my ($cgi, $first, $last) = @_;
+ my ($cgi, $last) = @_;
$cgi or return '';
my $old_r = $cgi->param('r');
- my $prev = ' ';
+ my $head = ' ';
my $next = ' ';
- my %opts = (-path => 1, -query => 1, -relative => 1);
if ($last) {
- $cgi->param('r', $last);
- $next = $cgi->url(%opts);
- $next = qq!<a href="$next">next</a>!;
+ $next = qq!<a href="?r=$last">next</a>!;
}
- if ($first && $old_r) {
- $cgi->param('r', "$first..");
- $prev = $cgi->url(%opts);
- $prev = qq!<a href="$prev">prev</a>!;
+ if ($old_r) {
+ $head = $cgi->path_info;
+ $head = qq!<a href="$head">head</a>!;
}
- "$prev $next";
+ "$next $head";
}
sub each_recent_blob {
my $cgi = $args->{cgi};
# revision ranges may be specified
- my $reverse;
my $range = 'HEAD';
my $r = $cgi->param('r') if $cgi;
- if ($r) {
- if ($r =~ /\A(?:$refhex\.\.)?$refhex\z/o) {
- $range = $r;
- } elsif ($r =~ /\A(?:$refhex\.\.)\z/o) {
- $reverse = 1;
- $range = $r;
- }
+ if ($r && ($r =~ /\A(?:$refhex\.\.)?$refhex\z/o)) {
+ $range = $r;
}
# get recent messages
# we could use git log -z, but, we already know ssoma will not
# leave us with filenames with spaces in them..
my @cmd = qw/git log --no-notes --no-color --raw -r/;
- push @cmd, '--reverse' if $reverse;
push @cmd, $range;
- my $first;
my $pid = open(my $log, '-|', @cmd) or
die('open `'.join(' ', @cmd) . " pipe failed: $!\n");
last;
}
}
- } elsif ($reverse) {
- unshift @commits, undef
} else {
push @commits, undef;
}
close $log; # we may EPIPE here
# for pagination
- $reverse ? ($commits[-1],$commits[0]) : ($commits[0],$commits[-1]);
+ $commits[-1];
}
# private functions below
}
my $url_base;
if ($cgi) {
- my $cgi_url = $cgi->url(-path=>1, -relative=>1);
- my $base = $cgi->url(-base);
- $url_base = $cgi_url;
+ my $path_info = $cgi->path_info;
+ my $base;
+ if (ref($cgi) eq 'CGI') {
+ $base = $cgi->url(-base);
+ } else {
+ $base = "${$cgi->base}";
+ $base =~ s!/\z!!;
+ }
+ $url_base = $path_info;
if ($url_base =~ s!/(?:|index\.html)?\z!!) {
$rv{atomurl} = "$base$url_base/atom.xml";
} else {
$url_base =~ s!/atom\.xml\z!!;
- $rv{atomurl} = $base . $cgi_url;
+ $rv{atomurl} = $base . $path_info;
$url_base = $base . $url_base; # XXX is this needed?
}
} else {
# returns 0 (skipped) or 1 (added)
sub add_to_feed {
- my ($feed_opts, $feed, $add, $top, $git) = @_;
+ my ($feed_opts, $feed, $add, $git) = @_;
my $mime = do_cat_mail($git, $add) or return 0;
- if ($top && $mime->header('In-Reply-To')) {
- return 0;
- }
-
my $midurl = $feed_opts->{midurl} || 'http://example.com/m/';
my $fullurl = $feed_opts->{fullurl} || 'http://example.com/f/';
my $mid = $mime->header_obj->header_raw('Message-ID');
defined $mid or return 0;
$mid = PublicInbox::Hval->new_msgid($mid);
- my $href = $mid->as_href;
- my $content = PublicInbox::View->as_feed_entry($mime,
- "$fullurl$href.html");
+ my $href = $mid->as_href . '.html';
+ my $content = PublicInbox::View->as_feed_entry($mime, $fullurl . $href);
defined($content) or return 0;
my $subject = mime_header($mime, 'Subject') or return 0;
author => { name => $name, email => $email },
title => $subject,
updated => $date,
- content => { type => "html", content => $content },
+ content => { type => 'xhtml', content => $content },
link => $midurl . $href,
id => 'urn:uuid:' . join('-', @uuid5),
);
}
sub dump_html_line {
- my ($self, $level, $args) = @_; # args => [ $html, $midurl ]
+ my ($self, $level, $html) = @_;
if ($self->message) {
- $args->[0] .= (' ' x $level);
+ $$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 $url = $args->[1] . $mid->as_href;
+ my $href = 'm/' . $mid->as_href . '.html';
my $from = mime_header($mime, 'From');
my @from = Email::Address->parse($from);
$from = PublicInbox::Hval->new_oneline($from)->as_html;
$subj = PublicInbox::Hval->new_oneline($subj)->as_html;
- $args->[0] .= "<a href=\"$url.html\">$subj</a> $from\n";
+ $$html .= "<a href=\"$href\">$subj</a> $from\n";
}
- dump_html_line($self->child, $level+1, $args) if $self->child;
- dump_html_line($self->next, $level, $args) if $self->next;
+ dump_html_line($self->child, $level+1, $html) if $self->child;
+ dump_html_line($self->next, $level, $html) if $self->next;
}
sub try_git_pm {