]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/Feed.pm
www: use WwwStream for dumping thread and search views
[public-inbox.git] / lib / PublicInbox / Feed.pm
index 045e495fae9ed8acfe133743451f126014dd3323..36802fa1fc79def41fd380907e274185bc587226 100644 (file)
@@ -65,23 +65,24 @@ sub emit_atom {
        my $fh = $cb->([ 200, ['Content-Type' => 'application/atom+xml']]);
        my $max = $ctx->{max} || MAX_PER_PAGE;
        my $x = atom_header($feed_opts);
-       my $git = $ctx->{git} ||= PublicInbox::Git->new($ctx->{git_dir});
+       my $ibx = $ctx->{-inbox};
        each_recent_blob($ctx, sub {
                my ($path, undef, $ts) = @_;
                if (defined $x) {
                        $fh->write($x . feed_updated(undef, $ts));
                        $x = undef;
                }
-               add_to_feed($feed_opts, $fh, $path, $git);
+               my $s = feed_entry($feed_opts, $path, $ibx) or return 0;
+               $fh->write($s);
+               1;
        });
        end_feed($fh);
 }
 
 sub _no_thread {
        my ($cb) = @_;
-       my $fh = $cb->([404, ['Content-Type' => 'text/plain']]);
-       $fh->write("No feed found for thread\n");
-       $fh->close;
+       $cb->([404, ['Content-Type', 'text/plain'],
+               ["No feed found for thread\n"]]);
 }
 
 sub end_feed {
@@ -102,9 +103,10 @@ sub emit_atom_thread {
        $feed_opts->{url} = $html_url;
        $feed_opts->{emit_header} = 1;
 
-       my $git = $ctx->{git} ||= PublicInbox::Git->new($ctx->{git_dir});
+       my $ibx = $ctx->{-inbox};
        foreach my $msg (@{$res->{msgs}}) {
-               add_to_feed($feed_opts, $fh, mid2path($msg->mid), $git);
+               my $s = feed_entry($feed_opts, mid2path($msg->mid), $ibx);
+               $fh->write($s) if defined $s;
        }
        end_feed($fh);
 }
@@ -165,13 +167,14 @@ sub emit_html_index {
 
 sub emit_index_nosrch {
        my ($ctx, $state) = @_;
-       my $git = $ctx->{git} ||= PublicInbox::Git->new($ctx->{git_dir});
+       my $ibx = $ctx->{-inbox};
+       my $fh = $state->{fh};
        my (undef, $last) = each_recent_blob($ctx, sub {
                my ($path, $commit, $ts, $u, $subj) = @_;
                $state->{first} ||= $commit;
 
-               my $mime = do_cat_mail($git, $path) or return 0;
-               PublicInbox::View::index_entry($mime, 0, $state);
+               my $mime = do_cat_mail($ibx, $path) or return 0;
+               $fh->write(PublicInbox::View::index_entry($mime, $state));
                1;
        });
        $last;
@@ -216,8 +219,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 $git = $ctx->{git} ||= PublicInbox::Git->new($ctx->{git_dir});
-       my $log = $git->popen(qw/log --no-notes --no-color --raw -r
+       my $log = $ctx->{-inbox}->git->popen(qw/log
+                               --no-notes --no-color --raw -r
                                --abbrev=16 --abbrev-commit/,
                                "--format=%h%x00%ct%x00%an%x00%s%x00",
                                $range);
@@ -267,31 +270,16 @@ sub get_feedopts {
        my $inbox = $ctx->{inbox};
        my $obj = $ctx->{-inbox};
        my $cgi = $ctx->{cgi};
-       my %rv = ( description => $obj ? $obj->description : 'FIXME' );
-
-       if ($obj) {
-               $rv{address} = $obj->{address};
-               $rv{id_addr} = $obj->{-primary_address};
-       } elsif ($pi_config && defined $inbox && $inbox ne '') {
-               # TODO: remove
-               my $addr = $pi_config->get($inbox, 'address') || "";
-               $rv{address} = $addr;
-               $addr = $addr->[0] if ref($addr);
-               $rv{id_addr} = $addr;
-       }
-       $rv{id_addr} ||= 'public-inbox@example.com';
+       my %rv = ( description => $obj->description );
 
+       $rv{address} = $obj->{address};
+       $rv{id_addr} = $obj->{-primary_address};
        my $url_base;
-       if ($obj) {
-               $url_base = $obj->base_url($cgi); # CGI may be undef
-               if (my $mid = $ctx->{mid}) { # per-thread feed:
-                       $rv{atomurl} = "$url_base$mid/t.atom";
-               } else {
-                       $rv{atomurl} = $url_base."new.atom";
-               }
+       $url_base = $obj->base_url($cgi); # CGI may be undef
+       if (my $mid = $ctx->{mid}) { # per-thread feed:
+               $rv{atomurl} = "$url_base$mid/t.atom";
        } else {
-               $url_base = 'http://example.com/';
-               $rv{atomurl} = $url_base.'new.atom';
+               $rv{atomurl} = $url_base."new.atom";
        }
        $rv{url} ||= $url_base;
        $rv{midurl} = $url_base;
@@ -307,61 +295,57 @@ sub feed_updated {
        '<updated>' . strftime(DATEFMT, @t) . '</updated>';
 }
 
-# returns 0 (skipped) or 1 (added)
-sub add_to_feed {
-       my ($feed_opts, $fh, $add, $git) = @_;
+# returns undef or string
+sub feed_entry {
+       my ($feed_opts, $add, $ibx) = @_;
 
-       my $mime = do_cat_mail($git, $add) or return 0;
+       my $mime = do_cat_mail($ibx, $add) or return;
        my $url = $feed_opts->{url};
        my $midurl = $feed_opts->{midurl};
 
        my $header_obj = $mime->header_obj;
        my $mid = $header_obj->header_raw('Message-ID');
-       defined $mid or return 0;
+       defined $mid or return;
        $mid = PublicInbox::Hval->new_msgid($mid);
        my $href = $midurl.$mid->as_href;
 
-       my $content = qq(<pre\nstyle="white-space:pre-wrap">) .
-               PublicInbox::View::multipart_text_as_html($mime, $href) .
-               '</pre>';
        my $date = $header_obj->header('Date');
        my $updated = feed_updated($date);
 
        my $title = $header_obj->header('Subject');
-       defined $title or return 0;
+       defined $title or return;
        $title = title_tag($title);
 
-       my $from = $header_obj->header('From') or return 0;
+       my $from = $header_obj->header('From') or return;
        my ($email) = PublicInbox::Address::emails($from);
-       my $name = PublicInbox::Address::from_name($from);
+       my $name = join(', ',PublicInbox::Address::names($from));
        $name = ascii_html($name);
        $email = ascii_html($email);
 
+       my $s = '';
        if (delete $feed_opts->{emit_header}) {
-               $fh->write(atom_header($feed_opts, $title) . $updated);
+               $s .= atom_header($feed_opts, $title) . $updated;
        }
-       $fh->write("<entry><author><name>$name</name><email>$email</email>" .
-                  "</author>$title$updated" .
-                  qq{<content\ntype="xhtml">} .
-                  qq{<div\nxmlns="http://www.w3.org/1999/xhtml">});
-       $fh->write($content);
+       $s .= "<entry><author><name>$name</name><email>$email</email>" .
+               "</author>$title$updated" .
+               qq{<content\ntype="xhtml">} .
+               qq{<div\nxmlns="http://www.w3.org/1999/xhtml">} .
+               qq(<pre\nstyle="white-space:pre-wrap">) .
+               PublicInbox::View::multipart_text_as_html($mime, $href) .
+               '</pre>';
 
        $add =~ tr!/!!d;
        my $h = '[a-f0-9]';
        my (@uuid5) = ($add =~ m!\A($h{8})($h{4})($h{4})($h{4})($h{12})!o);
        my $id = 'urn:uuid:' . join('-', @uuid5);
-       $fh->write(qq!</div></content><link\nhref="$href/"/>!.
-                  "<id>$id</id></entry>");
-       1;
+       $s .= qq!</div></content><link\nhref="$href/"/>!.
+               "<id>$id</id></entry>";
 }
 
 sub do_cat_mail {
-       my ($git, $path) = @_;
-       my $mime = eval {
-               my $str = $git->cat_file("HEAD:$path");
-               Email::MIME->new($str);
-       };
-       $@ ? undef : $mime;
+       my ($ibx, $path) = @_;
+       my $mime = eval { $ibx->msg_by_path($path) } or return;
+       Email::MIME->new($mime);
 }
 
 1;