]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Feed.pm
line-wrap generated HTML source around attrs for readability
[public-inbox.git] / lib / PublicInbox / Feed.pm
index 87b5b2a937469929aebbd190ccda866185423aa6..339d9c2031834e8bacf72b1c28f76c347726933a 100644 (file)
@@ -7,23 +7,21 @@ use Email::Address;
 use Email::MIME;
 use Date::Parse qw(strptime str2time);
 use PublicInbox::Hval;
-eval { require Git }; # this is GPLv2+, so we are OK to use it
+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;
 
-       local $ENV{GIT_DIR} = $args->{git_dir};
        my $feed_opts = get_feedopts($args);
        my $addr = $feed_opts->{address};
        $addr = $addr->[0] if ref($addr);
@@ -39,30 +37,30 @@ sub generate {
                updated => POSIX::strftime(DATEFMT, gmtime),
        );
 
-       my $git = try_git_pm($args->{git_dir});
+       my $git = PublicInbox::GitCatFile->new($args->{git_dir});
        each_recent_blob($args, sub {
                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;
-       local $ENV{GIT_DIR} = $args->{git_dir};
        my $feed_opts = get_feedopts($args);
 
        my $title = $feed_opts->{description} || '';
        $title = PublicInbox::Hval->new_oneline($title)->as_html;
 
        my @messages;
-       my $git = try_git_pm($args->{git_dir});
-       my $last = each_recent_blob($args, sub {
+       my $git = PublicInbox::GitCatFile->new($args->{git_dir});
+       my ($first, $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;
@@ -70,45 +68,59 @@ sub generate_html_index {
                push @messages, $mime;
                1;
        });
+       $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>';
+               '<link rel="alternate" title="Atom feed"' . "\nhref=\"" .
+               $feed_opts->{atomurl} . "\"\ntype=\"application/atom+xml\"/>" .
+               '</head><body>' . PRE_WRAP;
 
-       # sort by date, most recent at top
+       # sort child messages in chronological order
        $th->order(sub {
                sort {
-                       $b->topmost->message->header('X-PI-Date') <=>
-                       $a->topmost->message->header('X-PI-Date')
+                       $a->topmost->message->header('X-PI-Date') <=>
+                       $b->topmost->message->header('X-PI-Date')
                } @_;
        });
-       dump_html_line($_, 0, \$html) for $th->rootset;
 
-       my $footer = nav_footer($args->{cgi}, $last);
-       $footer = "<hr /><pre>$footer</pre>" if $footer;
+       my %seen;
+       # except we sort top-level messages reverse chronologically
+       for (sort { (eval { $b->message->header('X-PI-Date') } || 0) <=>
+                   (eval { $a->message->header('X-PI-Date') } || 0)
+                 } $th->rootset) {
+               dump_msg($_, 0, \$html, time, \%seen);
+       }
+
+       Email::Address->purge_cache;
+
+       my $footer = nav_footer($args->{cgi}, $first, $last, $feed_opts);
+       my $list_footer = $args->{footer};
+       $footer .= "\n" . $list_footer if ($footer && $list_footer);
+       $footer = "<hr />" . PRE_WRAP . "$footer</pre>" if $footer;
        $html . "</pre>$footer</html>";
 }
 
 # private subs
 
 sub nav_footer {
-       my ($cgi, $last) = @_;
+       my ($cgi, $first, $last, $feed_opts) = @_;
        $cgi or return '';
        my $old_r = $cgi->param('r');
        my $head = '    ';
        my $next = '    ';
 
        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>";
+       "$next $head $atom $permalink";
 }
 
 sub each_recent_blob {
@@ -130,7 +142,8 @@ sub each_recent_blob {
        # 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/;
+       my @cmd = ('git', "--git-dir=$args->{git_dir}",
+                       qw/log --no-notes --no-color --raw -r/);
        push @cmd, $range;
 
        my $pid = open(my $log, '-|', @cmd) or
@@ -168,7 +181,7 @@ sub each_recent_blob {
 
        close $log; # we may EPIPE here
        # for pagination
-       $commits[-1];
+       ($commits[0], $commits[-1]);
 }
 
 # private functions below
@@ -241,7 +254,7 @@ sub add_to_feed {
        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;
@@ -271,59 +284,24 @@ sub add_to_feed {
        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, $now, $seen) = @_;
+       my $mime = $self->message;
+       if ($mime) {
+               $$html .=
+                   PublicInbox::View->index_entry($mime, $now, $level, $seen);
        }
-       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, $now, $seen) if $self->child;
+       dump_msg($self->next, $level, $html, $now, $seen) if $self->next;
 }
 
-sub try_git_pm {
-       my ($dir) = @_;
-       eval { Git->repository(Directory => $dir) };
-};
-
 sub do_cat_mail {
        my ($git, $path) = @_;
-       my $str;
-       if ($git) {
-               open my $fh, '>', \$str or
-                               die "failed to setup string handle: $!\n";
-               binmode $fh;
-               my $err = '';
-               my $bytes;
-               {
-                       local $SIG{__WARN__} = sub { $err .= $_[0] };
-                       $bytes = $git->cat_blob("HEAD:$path", $fh);
-               }
-               close $fh or die "failed to close string handle: $!\n";
-
-               if ($bytes < 0 && $err &&
-                               $err !~ /doesn't exist in the repository/) {
-                       warn $err;
-               }
-               return if $bytes <= 0;
-       } else {
-               $str = `git cat-file blob HEAD:$path`;
-               return if $? != 0 || length($str) == 0;
-       }
-       Email::MIME->new($str);
+       my $mime = eval {
+               my $str = $git->cat_file("HEAD:$path");
+               Email::MIME->new($str);
+       };
+       $@ ? undef : $mime;
 }
 
 1;