]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/View.pm
No ext_urls
[public-inbox.git] / lib / PublicInbox / View.pm
index 3dbf8bac33f8c9ecc6eb3e19fb5b406f718aaf7a..77b92653b739d6b51262aa3716149c32c8678d42 100644 (file)
@@ -40,10 +40,10 @@ sub msg_page_i {
                                "../${\mid_href($smsg->{mid})}/" : '';
                if (_msg_page_prepare($eml, $ctx)) {
                        $eml->each_part(\&add_text_body, $ctx, 1);
-                       $ctx->zmore('</pre><hr>');
+                       print { $ctx->{zfh} } '</pre><hr>';
                }
                html_footer($ctx, $ctx->{first_hdr}) if !$ctx->{smsg};
-               delete($ctx->{obuf}) // \'';
+               ''; # XXX TODO cleanup
        } else { # called by WwwStream::async_next or getline
                $ctx->{smsg}; # may be undef
        }
@@ -58,7 +58,7 @@ sub no_over_html ($) {
        PublicInbox::WwwStream::init($ctx);
        if (_msg_page_prepare($eml, $ctx)) { # sets {-title_html}
                $eml->each_part(\&add_text_body, $ctx, 1);
-               $ctx->zmore('</pre><hr>');
+               print { $ctx->{zfh} } '</pre><hr>';
        }
        html_footer($ctx, $eml);
        $ctx->html_done;
@@ -80,11 +80,13 @@ sub msg_page {
        # allow user to easily browse the range around this message if
        # they have ->over
        $ctx->{-t_max} = $smsg->{ts};
+       $ctx->{-spfx} = '../' if $ibx->{coderepo};
        PublicInbox::WwwStream::aresponse($ctx, \&msg_page_i);
 }
 
 # /$INBOX/$MESSAGE_ID/#R
 sub msg_reply ($$) {
+       return "";
        my ($ctx, $hdr) = @_;
        my $se_url =
         'https://kernel.org/pub/software/scm/git/docs/git-send-email.html';
@@ -240,14 +242,12 @@ sub eml_entry {
                my $html = ascii_html($irt);
                $rv .= qq(In-Reply-To: &lt;<a\nhref="$href">$html</a>&gt;\n)
        }
-       $rv .= "\n";
+       say { $ctx->zfh } $rv;
 
        # scan through all parts, looking for displayable text
        $ctx->{mhref} = $mhref;
        $ctx->{changed_href} = "#e$id"; # for diffstat "files? changed,"
-       $ctx->{obuf} = \$rv;
        $eml->each_part(\&add_text_body, $ctx, 1); # expensive
-       $ctx->zmore; # TODO: remove once add_text_body is updated
 
        # add the footer
        $rv = "\n<a\nhref=#$id_m\nid=e$id>^</a> ".
@@ -387,8 +387,9 @@ sub pre_thread  { # walk_thread callback
 sub thread_eml_entry {
        my ($ctx, $eml) = @_;
        my ($beg, $end) = thread_adj_level($ctx, $ctx->{level});
-       $ctx->zmore($beg.'<pre>');
-       eml_entry($ctx, $eml) . '</pre>' . $end;
+       print { $ctx->zfh } $beg, '<pre>';
+       print { $ctx->{zfh} } eml_entry($ctx, $eml), '</pre>';
+       $end;
 }
 
 sub next_in_queue ($$) {
@@ -415,15 +416,15 @@ sub stream_thread_i { # PublicInbox::WwwStream::getline callback
                                if (!$ghost_ok) { # first non-ghost
                                        $ctx->{-title_html} =
                                                ascii_html($smsg->{subject});
-                                       $ctx->zmore($ctx->html_top);
+                                       print { $ctx->zfh } $ctx->html_top;
                                }
                                return $smsg;
                        }
                        # buffer the ghost entry and loop
-                       $ctx->zmore(ghost_index_entry($ctx, $lvl, $smsg));
+                       print { $ctx->zfh } ghost_index_entry($ctx, $lvl, $smsg)
                } else { # all done
-                       $ctx->zmore(join('', thread_adj_level($ctx, 0)));
-                       $ctx->zmore(${delete($ctx->{skel})});
+                       print { $ctx->zfh } thread_adj_level($ctx, 0),
+                                               ${delete($ctx->{skel})};
                        return;
                }
        }
@@ -443,6 +444,7 @@ sub thread_html {
        my $ibx = $ctx->{ibx};
        my ($nr, $msgs) = $ibx->over->get_thread($mid);
        return missing_thread($ctx) if $nr == 0;
+       $ctx->{-spfx} = '../../' if $ibx->{coderepo};
 
        # link $INBOX_DIR/description text to "index_topics" view around
        # the newest message in this thread
@@ -492,7 +494,7 @@ sub thread_html_i { # PublicInbox::WwwStream::getline callback
                my $smsg = $ctx->{smsg};
                if (exists $ctx->{-html_tip}) {
                        $ctx->{-title_html} = ascii_html($smsg->{subject});
-                       $ctx->zmore($ctx->html_top);
+                       print { $ctx->zfh } $ctx->html_top;
                }
                return eml_entry($ctx, $eml);
        } else {
@@ -500,7 +502,7 @@ sub thread_html_i { # PublicInbox::WwwStream::getline callback
                        return $smsg if exists($smsg->{blob});
                }
                my $skel = delete($ctx->{skel}) or return; # all done
-               $ctx->zmore($$skel);
+               print { $ctx->zfh } $$skel;
                undef;
        }
 }
@@ -560,13 +562,10 @@ sub add_text_body { # callback for each_part
        my ($part, $depth, $idx) = @$p;
        my $ct = $part->content_type || 'text/plain';
        my $fn = $part->filename;
-       my $rv = $ctx->{obuf} //= \(my $obuf = '');
        my ($s, $err) = msg_part_text($part, $ct);
-       $s // return $$rv .= (attach_link($ctx, $ct, $p, $fn) // '');
-       if ($part->{is_submsg}) {
-               $$rv .= submsg_hdr($ctx, $part);
-               $$rv .= "\n";
-       }
+       my $zfh = $ctx->zfh;
+       $s // return print $zfh (attach_link($ctx, $ct, $p, $fn) // '');
+       say $zfh submsg_hdr($ctx, $part) if $part->{is_submsg};
 
        # makes no difference to browsers, and don't screw up filename
        # link generation in diffs with the extra '%0D'
@@ -589,24 +588,6 @@ sub add_text_body { # callback for each_part
                $ctx->{-anchors} = {} if $s =~ /^diff --git /sm;
                $diff = 1;
                delete $ctx->{-long_path};
-               my $spfx;
-               # absolute URL (Atom feeds)
-               if ($ibx->{coderepo}) {
-                       if (index($upfx, '//') >= 0) {
-                               $spfx = $upfx;
-                               $spfx =~ s!/([^/]*)/\z!/!;
-                       } else {
-                               my $n_slash = $upfx =~ tr!/!/!;
-                               if ($n_slash == 0) {
-                                       $spfx = '../';
-                               } elsif ($n_slash == 1) {
-                                       $spfx = '';
-                               } else { # nslash == 2
-                                       $spfx = '../../';
-                               }
-                       }
-               }
-               $ctx->{-spfx} = $spfx;
        };
 
        # split off quoted and unquoted blocks:
@@ -614,20 +595,19 @@ sub add_text_body { # callback for each_part
        undef $s; # free memory
        if (defined($fn) || ($depth > 0 && !$part->{is_submsg}) || $err) {
                # badly-encoded message with $err? tell the world about it!
-               $$rv .= attach_link($ctx, $ct, $p, $fn, $err);
-               $$rv .= "\n";
+               say $zfh attach_link($ctx, $ct, $p, $fn, $err);
        }
        delete $part->{bdy}; # save memory
        for my $cur (@sections) { # $cur may be huge
                if ($cur =~ /\A>/) {
                        # we use a <span> here to allow users to specify
                        # their own color for quoted text
-                       $ctx->zmore(qq(<span\nclass="q">),
-                                       $l->to_html($cur), '</span>');
+                       print $zfh qq(<span\nclass="q">),
+                                       $l->to_html($cur), '</span>';
                } elsif ($diff) {
                        flush_diff($ctx, \$cur);
                } else { # regular lines, OK
-                       $ctx->zmore($l->to_html($cur));
+                       print $zfh $l->to_html($cur);
                }
                undef $cur; # free memory
        }
@@ -644,7 +624,8 @@ sub _msg_page_prepare {
                        return;
                }
                $ctx->{-html_tip} =
-"<pre>WARNING: multiple messages have this Message-ID\n</pre><pre>";
+qq[<pre>WARNING: multiple messages have this Message-ID (<a
+href="d/">diff</a>)</pre><pre>];
        } else {
                $ctx->{first_hdr} = $eml->header_obj;
                $ctx->{chash} = content_hash($eml) if $ctx->{smsg}; # reused MID
@@ -689,10 +670,10 @@ sub _msg_page_prepare {
                $hbuf .= qq[Message-ID: &lt;$x&gt; (<a href="raw">raw</a>)\n];
        }
        if (!$nr) { # first (and only) message, common case
-               $ctx->zmore($ctx->html_top, $hbuf);
+               print { $ctx->zfh } $ctx->html_top, $hbuf;
        } else {
                delete $ctx->{-title_html};
-               $ctx->zmore($ctx->{-html_tip}, $hbuf);
+               print { $ctx->zfh } $ctx->{-html_tip}, $hbuf;
        }
        $ctx->{-linkify} //= PublicInbox::Linkify->new;
        $hbuf = '';
@@ -703,7 +684,7 @@ sub _msg_page_prepare {
                        $hbuf .= "$h: $_\n" for ($eml->header_raw($h));
                }
                $ctx->{-linkify}->linkify_mids('..', \$hbuf, 1); # escapes HTML
-               $ctx->zmore($hbuf);
+               print { $ctx->{zfh} } $hbuf;
                $hbuf = '';
        }
        my @irt = $eml->header_raw('In-Reply-To');
@@ -721,8 +702,7 @@ sub _msg_page_prepare {
                $hbuf .= 'References: <'.join(">\n\t<", @$refs).">\n" if @$refs;
        }
        $ctx->{-linkify}->linkify_mids('..', \$hbuf); # escapes HTML
-       $ctx->zmore($hbuf .= "\n");
-       ${$ctx->{obuf}} = ''; # TODO remove
+       say { $ctx->{zfh} } $hbuf;
        1;
 }
 
@@ -776,7 +756,7 @@ sub thread_skel ($$$) {
 sub html_footer {
        my ($ctx, $hdr) = @_;
        my $upfx = '../';
-       my ($related, $skel);
+       my (@related, $skel);
        my $foot = '<pre>';
        my $qry = delete $ctx->{-qry};
        if ($qry && $ctx->{ibx}->isrch) {
@@ -791,7 +771,7 @@ sub html_footer {
                $q = wrap('', '', $q);
                my $rows = ($q =~ tr/\n/\n/) + 1;
                $q = ascii_html($q);
-               $related = <<EOM;
+               $related[0] = <<EOM;
 <form id=related
 action=$upfx
 ><pre>find likely ancestor, descendant, or conflicting patches for <a
@@ -804,7 +784,7 @@ EOM
        if ($ctx->{ibx}->over) {
                my $t = ts2str($ctx->{-t_max});
                my $t_fmt = fmt_ts($ctx->{-t_max});
-               my $fallback = $related ? "\t" : "<a id=related>\t</a>";
+               my $fallback = @related ? "\t" : "<a id=related>\t</a>";
                $skel = <<EOF;
 ${fallback}other threads:[<a
 href="$upfx?t=$t">~$t_fmt UTC</a>|<a
@@ -839,10 +819,10 @@ EOF
        } else { # unindexed inboxes w/o over
                $skel = qq( <a\nhref="$upfx">latest</a>);
        }
-       $foot .= qq(<a\nhref="#R">reply</a>);
        # $skel may be big for big threads, don't append it to $foot
-       $skel .= '</pre>' . ($related // '');
-       $ctx->zmore($foot, $skel .= msg_reply($ctx, $hdr)); # flushes obuf
+       print { $ctx->zfh } $foot, qq(<a\nhref="#R">reply</a>),
+                               $skel, '</pre>', @related,
+                               msg_reply($ctx, $hdr);
 }
 
 sub ghost_parent {
@@ -1040,6 +1020,8 @@ sub _skel_ghost {
        1;
 }
 
+# note: we favor Date: here because git-send-email increments it
+# to preserve [PATCH $N/$M] ordering in series (it can't control Received:)
 sub sort_ds {
        @{$_[0]} = sort {
                (eval { $a->topmost->{ds} } || 0) <=>
@@ -1061,9 +1043,10 @@ sub acc_topic { # walk_thread callback
        if ($has_blob) {
                my $subj = subject_normalized($smsg->{subject});
                $subj = '(no subject)' if $subj eq '';
+               my $ts = $smsg->{ts};
                my $ds = $smsg->{ds};
                if ($level == 0) { # new, top-level topic
-                       my $topic = [ $ds, 1, { $subj => $mid }, $subj ];
+                       my $topic = [ $ts, $ds, 1, { $subj => $mid }, $subj ];
                        $ctx->{-cur_topic} = $topic;
                        push @{$ctx->{order}}, $topic;
                        return 1;
@@ -1071,10 +1054,11 @@ sub acc_topic { # walk_thread callback
 
                # continue existing topic
                my $topic = $ctx->{-cur_topic}; # should never be undef
-               $topic->[0] = $ds if $ds > $topic->[0];
-               $topic->[1]++; # bump N+ message counter
-               my $seen = $topic->[2];
-               if (scalar(@$topic) == 3) { # parent was a ghost
+               $topic->[0] = $ts if $ts > $topic->[0];
+               $topic->[1] = $ds if $ds > $topic->[1];
+               $topic->[2]++; # bump N+ message counter
+               my $seen = $topic->[3];
+               if (scalar(@$topic) == 4) { # parent was a ghost
                        push @$topic, $subj;
                } elsif (!defined($seen->{$subj})) {
                        push @$topic, $level, $subj; # @extra messages
@@ -1082,7 +1066,7 @@ sub acc_topic { # walk_thread callback
                $seen->{$subj} = $mid; # latest for subject
        } else { # ghost message
                return 1 if $level != 0; # ignore child ghosts
-               my $topic = $ctx->{-cur_topic} = [ -666, 0, {} ];
+               my $topic = $ctx->{-cur_topic} = [ -666, -666, 0, {} ];
                push @{$ctx->{order}}, $topic;
        }
        1;
@@ -1103,7 +1087,7 @@ sub dump_topics {
        }
        # sort by recency, this allows new posts to "bump" old topics...
        foreach my $topic (sort { $b->[0] <=> $a->[0] } @$order) {
-               my ($ds, $n, $seen, $top_subj, @extra) = @$topic;
+               my ($ts, $ds, $n, $seen, $top_subj, @extra) = @$topic;
                @$topic = ();
                next unless defined $top_subj;  # ghost topic
                my $mid = delete $seen->{$top_subj};
@@ -1247,4 +1231,30 @@ sub ghost_index_entry {
                . '</pre>' . $end;
 }
 
+# /$INBOX/$MSGID/d/ endpoint
+sub diff_msg {
+       my ($ctx) = @_;
+       require PublicInbox::MailDiff;
+       my $ibx = $ctx->{ibx};
+       my $over = $ibx->over or return no_over_html($ctx);
+       my ($id, $prev);
+       my $md = bless { ctx => $ctx }, 'PublicInbox::MailDiff';
+       my $next_arg = $md->{next_arg} = [ $ctx->{mid}, \$id, \$prev ];
+       my $smsg = $md->{smsg} = $over->next_by_mid(@$next_arg) or
+               return; # undef == 404
+       $ctx->{-t_max} = $smsg->{ts};
+       $ctx->{-upfx} = '../../';
+       $ctx->{-apfx} = '//'; # fail on to_attr()
+       $ctx->{-linkify} = PublicInbox::Linkify->new;
+       my $mid = ascii_html($smsg->{mid});
+       $ctx->{-title_html} = "diff for duplicates of &lt;$mid&gt;";
+       PublicInbox::WwwStream::html_init($ctx);
+       print { $ctx->{zfh} } '<pre>diff for duplicates of &lt;<a href="../">',
+                               $mid, "</a>&gt;\n\n";
+       sub {
+               $ctx->attach($_[0]->([200, delete $ctx->{-res_hdr}]));
+               $md->begin_mail_diff;
+       };
+}
+
 1;