]> Sergey Matveev's repositories - public-inbox.git/commitdiff
Merge branch 'thread-view-skel'
authorEric Wong <e@80x24.org>
Thu, 30 Jun 2016 08:57:49 +0000 (08:57 +0000)
committerEric Wong <e@80x24.org>
Thu, 30 Jun 2016 08:57:49 +0000 (08:57 +0000)
* thread-view-skel:
  view: show thread size when linking to summary
  view: default to flat/hybrid thread display
  view: fix up some HTML injection via Message-ID vectors
  www: reinstate old thread view as an option
  view: show more nearby messages in flat thread view
  view: tweak thread/index header slightly
  feed: add $INBOX/new.html endpoint
  view: merge $state hash with existing $ctx
  view: show thread context in the thread-aware flat view
  www: use WwwStream for dumping thread and search views
  www: implement hybrid flat+thread conversation view

TODO
lib/PublicInbox/Feed.pm
lib/PublicInbox/SearchView.pm
lib/PublicInbox/View.pm
lib/PublicInbox/WWW.pm
lib/PublicInbox/WwwStream.pm

diff --git a/TODO b/TODO
index f29f2f0f29d6da22673eb169508796ae0c9f5c6d..3b6401f62062a2b88ff5b9c5b2b138b6ad3a5c01 100644 (file)
--- a/TODO
+++ b/TODO
@@ -4,8 +4,6 @@ TODO items for public-inbox
 
 * mailmap support (same as git) for remapping expired email addresses
 
-* WWW: Hybrid flat view + thread skeleton (requires Xapian)
-
 * POP3 server, since some webmail providers support external POP3:
   https://public-inbox.org/meta/20160411034104.GA7817@dcvr.yhbt.net/
 
index 8e2330619a35ac032f951a9fd91be683caee653a..c16c417a099e9e6b88bdc94e5f9897e37293380a 100644 (file)
@@ -34,6 +34,33 @@ sub generate_html_index {
        sub { emit_html_index($_[0], $ctx) };
 }
 
+sub new_html {
+       my ($ctx) = @_;
+       my @paths;
+       my (undef, $last) = each_recent_blob($ctx, sub {
+               my ($path, $commit, $ts, $u, $subj) = @_;
+               $ctx->{first} ||= $commit;
+               push @paths, $path;
+       });
+       if (!@paths) {
+               return [404, ['Content-Type', 'text/plain'],
+                       ["No messages, yet\n"] ];
+       }
+       $ctx->{-html_tip} = '<pre>';
+       $ctx->{-upfx} = '';
+       my $res = PublicInbox::WwwStream->new($ctx, sub {
+               while (my $path = shift @paths) {
+                       my $m = do_cat_mail($ctx->{-inbox}, $path) or next;
+                       my $more = scalar @paths;
+                       my $s = PublicInbox::View::index_entry($m, $ctx, $more);
+                       $s .= '</pre>' unless $more;
+                       return $s;
+               }
+               undef;
+       });
+       [ 200, ['Content-Type', 'text/html; charset=UTF-8'], $res ]
+}
+
 # private subs
 
 sub title_tag {
@@ -138,9 +165,12 @@ sub emit_html_index {
        my $fh = $res->([200,['Content-Type'=>'text/html; charset=UTF-8']]);
 
        my $max = $ctx->{max} || MAX_PER_PAGE;
+       $ctx->{-upfx} = '';
 
        my ($footer, $param, $last);
-       my $state = { ctx => $ctx, seen => {}, anchor_idx => 0, fh => $fh };
+       $ctx->{seen} = {};
+       $ctx->{anchor_idx} = 0;
+       $ctx->{fh} = $fh;
        my $srch = $ctx->{srch};
        $fh->write(_html_index_top($feed_opts, $srch));
 
@@ -148,14 +178,13 @@ sub emit_html_index {
        # which we must continue supporting:
        my $qp = $ctx->{qp};
        if ($qp && !$qp->{r} && $srch) {
-               $state->{srch} = $srch;
-               $last = PublicInbox::View::emit_index_topics($state);
+               $last = PublicInbox::View::emit_index_topics($ctx);
                $param = 'o';
        } else {
-               $last = emit_index_nosrch($ctx, $state);
+               $last = emit_index_nosrch($ctx);
                $param = 'r';
        }
-       $footer = nav_footer($ctx, $last, $feed_opts, $state, $param);
+       $footer = nav_footer($ctx, $last, $feed_opts, $param);
        if ($footer) {
                my $list_footer = $ctx->{footer};
                $footer .= "\n\n" . $list_footer if $list_footer;
@@ -166,27 +195,28 @@ sub emit_html_index {
 }
 
 sub emit_index_nosrch {
-       my ($ctx, $state) = @_;
+       my ($ctx) = @_;
        my $ibx = $ctx->{-inbox};
+       my $fh = $ctx->{fh};
        my (undef, $last) = each_recent_blob($ctx, sub {
                my ($path, $commit, $ts, $u, $subj) = @_;
-               $state->{first} ||= $commit;
+               $ctx->{first} ||= $commit;
 
                my $mime = do_cat_mail($ibx, $path) or return 0;
-               PublicInbox::View::index_entry($mime, 0, $state);
+               $fh->write(PublicInbox::View::index_entry($mime, $ctx, 1));
                1;
        });
        $last;
 }
 
 sub nav_footer {
-       my ($ctx, $last, $feed_opts, $state, $param) = @_;
+       my ($ctx, $last, $feed_opts, $param) = @_;
        my $qp = $ctx->{qp} or return '';
        my $old_r = $qp->{$param};
        my $head = '    ';
        my $next = '    ';
-       my $first = $state->{first};
-       my $anchor = $state->{anchor_idx};
+       my $first = $ctx->{first};
+       my $anchor = $ctx->{anchor_idx};
 
        if ($last) {
                $next = qq!<a\nhref="?$param=$last"\nrel=next>next</a>!;
index ae875bf736d2bb3be8f28ec63d19373f856243fe..4af6cadbaf0e901b0a571912f5d53d4d1623d8c2 100644 (file)
@@ -8,12 +8,14 @@ use warnings;
 use PublicInbox::SearchMsg;
 use PublicInbox::Hval qw/ascii_html/;
 use PublicInbox::View;
-use PublicInbox::MID qw(mid2path mid_mime);
+use PublicInbox::MID qw(mid2path mid_mime mid_clean);
 use Email::MIME;
 require PublicInbox::Git;
 require PublicInbox::Thread;
 our $LIM = 50;
 
+sub noop {}
+
 sub sres_top_html {
        my ($ctx) = @_;
        my $q = PublicInbox::SearchQuery->new($ctx->{qp});
@@ -27,44 +29,46 @@ sub sres_top_html {
                relevance => $q->{r},
        };
        my ($mset, $total);
-
        eval {
-               $mset = $ctx->{srch}->query($q->{q}, $opts);
+               $mset = $ctx->{srch}->query($q->{'q'}, $opts);
                $total = $mset->get_matches_estimated;
        };
        my $err = $@;
-       my $res = html_start($q, $ctx) . '<pre>';
+       ctx_prepare($q, $ctx);
+       my $cb;
        if ($err) {
                $code = 400;
-               $res .= err_txt($ctx, $err) . "</pre><hr /><pre>" . foot($ctx);
+               $ctx->{-html_tip} = '<pre>'.err_txt($ctx, $err).'</pre><hr />';
+               $cb = *noop;
        } elsif ($total == 0) {
                $code = 404;
-               $res .= "\n\n[No results found]</pre><hr /><pre>".foot($ctx);
+               $ctx->{-html_tip} = "<pre>\n[No results found]</pre><hr />";
+               $cb = *noop;
        } else {
                my $x = $q->{x};
                return sub { adump($_[0], $mset, $q, $ctx) } if ($x eq 'A');
 
-               $res .= search_nav_top($mset, $q) . "\n\n";
+               $ctx->{-html_tip} = search_nav_top($mset, $q) . "\n\n";
                if ($x eq 't') {
-                       return sub { tdump($_[0], $res, $mset, $q, $ctx) };
+                       $cb = mset_thread($ctx, $mset, $q);
+               } else {
+                       $cb = mset_summary($ctx, $mset, $q);
                }
-               dump_mset(\$res, $mset);
-               $res .= '</pre>' . search_nav_bot($mset, $q) .
-                       "\n\n" . foot($ctx);
        }
 
-       $res .= "</pre></body></html>";
-       [$code, ['Content-Type'=>'text/html; charset=UTF-8'], [$res]];
+       [ $code, ['Content-Type', 'text/html; charset=UTF-8'],
+               PublicInbox::WwwStream->new($ctx, $cb) ];
 }
 
 # display non-threaded search results similar to what users expect from
 # regular WWW search engines:
-sub dump_mset {
-       my ($res, $mset) = @_;
+sub mset_summary {
+       my ($ctx, $mset, $q) = @_;
 
        my $total = $mset->get_matches_estimated;
        my $pad = length("$total");
        my $pfx = ' ' x $pad;
+       my $res = \($ctx->{-html_tip});
        foreach my $m ($mset->items) {
                my $rank = sprintf("%${pad}d", $m->get_rank + 1);
                my $pct = $m->get_percent;
@@ -77,6 +81,8 @@ sub dump_mset {
                        $s . "</a></b>\n";
                $$res .= "$pfx  - by $f @ $ts UTC [$pct%]\n\n";
        }
+       $$res .= search_nav_bot($mset, $q);
+       *noop;
 }
 
 sub err_txt {
@@ -85,14 +91,14 @@ sub err_txt {
        $u = PublicInbox::Hval::prurl($ctx->{cgi}->{env}, $u);
        $err =~ s/^\s*Exception:\s*//; # bad word to show users :P
        $err = ascii_html($err);
-       "\n\nBad query: <b>$err</b>\n" .
+       "\nBad query: <b>$err</b>\n" .
                qq{See <a\nhref="$u">$u</a> for Xapian query syntax};
 }
 
 sub search_nav_top {
        my ($mset, $q) = @_;
 
-       my $rv = "Search results ordered by [";
+       my $rv = "<pre>Search results ordered by [";
        if ($q->{r}) {
                my $d = $q->qs_html(r => 0);
                $rv .= qq{<a\nhref="?$d">date</a>|<b>relevance</b>};
@@ -122,7 +128,7 @@ sub search_nav_bot {
        my $o = $q->{o};
        my $end = $o + $nr;
        my $beg = $o + 1;
-       my $rv = "<hr /><pre>Results $beg-$end of $total";
+       my $rv = "</pre><hr /><pre>Results $beg-$end of $total";
        my $n = $o + $LIM;
 
        if ($n < $total) {
@@ -135,13 +141,11 @@ sub search_nav_bot {
                my $qs = $q->qs_html(o => ($p > 0 ? $p : 0));
                $rv .= qq{<a\nhref="?$qs"\nrel=prev>prev</a>};
        }
-       $rv;
+       $rv .= '</pre>';
 }
 
-sub tdump {
-       my ($cb, $res, $mset, $q, $ctx) = @_;
-       my $fh = $cb->([200, ['Content-Type'=>'text/html; charset=UTF-8']]);
-       $fh->write($res .= '</pre>');
+sub mset_thread {
+       my ($ctx, $mset, $q) = @_;
        my %pct;
        my @m = map {
                my $i = $_;
@@ -163,68 +167,51 @@ sub tdump {
        } else { # order by time (default for threaded view)
                $th->order(*PublicInbox::View::sort_ts);
        }
-       my $state = {
-               ctx => $ctx,
-               anchor_idx => 0,
-               pct => \%pct,
-               cur_level => 0,
-               -inbox => $ctx->{-inbox},
-               fh => $fh,
-       };
-       $ctx->{searchview} = 1;
-       PublicInbox::View::walk_thread($th, $state, *tdump_ent);
-       PublicInbox::View::thread_adj_level($state, 0);
-
-       $fh->write(search_nav_bot($mset, $q). "\n\n" .
-                       foot($ctx). '</pre></body></html>');
-
-       $fh->close;
-}
-
-sub tdump_ent {
-       my ($state, $level, $node) = @_;
-       my $mime = $node->message;
-
-       if ($mime) {
-               # lazy load the full message from mini_mime:
-               my $mid = mid_mime($mime);
-               $mime = eval { $state->{-inbox}->msg_by_mid($mid) } and
+       my $skel = search_nav_bot($mset, $q). "<pre>";
+       my $inbox = $ctx->{-inbox};
+       $ctx->{-upfx} = '';
+       $ctx->{anchor_idx} = 1;
+       $ctx->{cur_level} = 0;
+       $ctx->{dst} = \$skel;
+       $ctx->{mapping} = {};
+       $ctx->{pct} = \%pct;
+       $ctx->{prev_attr} = '';
+       $ctx->{prev_level} = 0;
+       $ctx->{seen} = {};
+
+       PublicInbox::View::walk_thread($th, $ctx,
+               *PublicInbox::View::pre_thread);
+
+       my $msgs = \@m;
+       my $mime;
+       sub {
+               return unless $msgs;
+               while ($mime = shift @$msgs) {
+                       my $mid = mid_clean(mid_mime($mime));
+                       $mime = $inbox->msg_by_mid($mid) and last;
+               }
+               if ($mime) {
                        $mime = Email::MIME->new($mime);
-       }
-       if ($mime) {
-               my $end = PublicInbox::View::thread_adj_level($state, $level);
-               PublicInbox::View::index_entry($mime, $level, $state);
-               $state->{fh}->write($end) if $end;
-       } else {
-               my $mid = $node->messageid;
-               PublicInbox::View::ghost_flush($state, '', $mid, $level);
-       }
-}
-
-sub foot {
-       my ($ctx) = @_;
-       my $foot = $ctx->{footer} || '';
-       qq{Back to <a\nhref=".">index</a>.\n$foot};
+                       return PublicInbox::View::index_entry($mime, $ctx);
+               }
+               $msgs = undef;
+               $skel .= "\n</pre>";
+       };
 }
 
-sub html_start {
+sub ctx_prepare {
        my ($q, $ctx) = @_;
        my $qh = ascii_html($q->{'q'});
-       my $A = $q->qs_html(x => 'A', r => undef);
-       my $res = '<html><head>' . PublicInbox::Hval::STYLE .
-               "<title>$qh - search results</title>" .
-               qq{<link\nrel=alternate\ntitle="Atom feed"\n} .
-               qq!href="?$A"\ntype="application/atom+xml"/></head>! .
-               qq{<body><form\naction="">} .
-               qq{<input\nname=q\nvalue="$qh"\ntype=text />};
-
-       $res .= qq{<input\ntype=hidden\nname=r />} if $q->{r};
+       $ctx->{-q_value_html} = $qh;
+       $ctx->{-atom} = '?'.$q->qs_html(x => 'A', r => undef);
+       $ctx->{-title_html} = "$qh - search results";
+       my $extra = '';
+       $extra .= qq{<input\ntype=hidden\nname=r />} if $q->{r};
        if (my $x = $q->{x}) {
                $x = ascii_html($x);
-               $res .= qq{<input\ntype=hidden\nname=x\nvalue="$x" />};
+               $extra .= qq{<input\ntype=hidden\nname=x\nvalue="$x" />};
        }
-
-       $res .= qq{<input\ntype=submit\nvalue=search /></form>};
+       $ctx->{-extra_form_html} = $extra;
 }
 
 sub adump {
index 0d980f13ada05acc566885e7b34ca6c0d67cbddc..db2bd20f3a8e9b8920bb6042ffbeb4c51a97d122 100644 (file)
@@ -90,151 +90,269 @@ sub _hdr_names ($$) {
        ascii_html(join(', ', PublicInbox::Address::names($val)));
 }
 
+sub nr_to_s ($$$) {
+       my ($nr, $singular, $plural) = @_;
+       return "0 $plural" if $nr == 0;
+       $nr == 1 ? "$nr $singular" : "$nr $plural";
+}
+
 # this is already inside a <pre>
 sub index_entry {
-       my ($mime, $level, $state) = @_;
-       my $midx = $state->{anchor_idx}++;
-       my $ctx = $state->{ctx};
+       my ($mime, $ctx, $more) = @_;
        my $srch = $ctx->{srch};
        my $hdr = $mime->header_obj;
        my $subj = $hdr->header('Subject');
 
        my $mid_raw = mid_clean(mid_mime($mime));
-       my $id = anchor_for($mid_raw);
-       my $seen = $state->{seen};
-       $seen->{$id} = "#$id"; # save the anchor for children, later
-
+       my $id = id_compress($mid_raw, 1);
+       my $id_m = 'm'.$id;
        my $mid = PublicInbox::Hval->new_msgid($mid_raw);
 
-       my $root_anchor = $state->{root_anchor} || '';
-       my $path = $root_anchor ? '../../' : '';
-       my $href = $mid->as_href;
+       my $root_anchor = $ctx->{root_anchor} || '';
        my $irt = in_reply_to($hdr);
-       my $parent_anchor = $seen->{anchor_for($irt)} if defined $irt;
-
-       $subj = ascii_html($subj);
-       $subj = "<a\nhref=\"${path}$href/\">$subj</a>";
-       $subj = "<u\nid=u>$subj</u>" if $root_anchor eq $id;
-
-       my $ts = _msg_date($hdr);
-       my $rv = "<pre\nid=s$midx>";
-       $rv .= "<b\nid=$id>$subj</b>\n";
-       my $txt = "${path}$href/raw";
-       my $fh = $state->{fh};
-       my $from = _hdr_names($hdr, 'From');
-       $rv .= "- $from @ $ts UTC (<a\nhref=\"$txt\">raw</a>)\n";
+
+       my $rv = '<b>'.ascii_html($subj).'</b>';
+       $rv = "<u\nid=u>$rv</u>" if $root_anchor eq $id_m;
+       $rv .= "\n";
+       $rv .= _th_index_lite($mid_raw, $irt, $id, $ctx);
        my @tocc;
        foreach my $f (qw(To Cc)) {
                my $dst = _hdr_names($hdr, $f);
                push @tocc, "$f: $dst" if $dst ne '';
        }
+       $rv .= "From: "._hdr_names($hdr, 'From').' @ '._msg_date($hdr)." UTC";
+       my $upfx = $ctx->{-upfx};
+       my $mhref = $upfx . $mid->as_href . '/';
+       $rv .= qq{ (<a\nhref="$mhref/">permalink</a> / };
+       $rv .= qq{<a\nhref="$mhref/raw">raw</a>)\n};
        $rv .= '  '.join('; +', @tocc) . "\n" if @tocc;
-       $fh->write($rv .= "\n");
 
-       my $mhref = "${path}$href/";
+       my $mapping = $ctx->{mapping};
+       if (!$mapping && $irt) {
+               my $mirt = PublicInbox::Hval->msgid($irt);
+               my $href = $upfx . $mirt->as_href . '/';
+               my $html = $mirt->as_html;
+               $rv .= qq(In-Reply-To: &lt;<a\nhref="$href/">$html</a>&gt;\n)
+       }
+       $rv .= "\n";
 
        # scan through all parts, looking for displayable text
-       msg_iter($mime, sub { index_walk($fh, $mhref, $_[0]) });
-       $rv = "\n" . html_footer($hdr, 0, $ctx, "$path$href/#R");
-
-       if (defined $irt) {
-               unless (defined $parent_anchor) {
-                       my $v = PublicInbox::Hval->new_msgid($irt, 1);
-                       $v = $v->as_href;
-                       $parent_anchor = "${path}$v/";
-               }
-               $rv .= " <a\nhref=\"$parent_anchor\">parent</a>";
-       }
-       if (my $pct = $state->{pct}) { # used by SearchView.pm
+       msg_iter($mime, sub { $rv .= add_text_body($mhref, $_[0]) });
+
+       # add the footer
+       $rv .= "\n<a\nhref=#$id_m\nid=e$id>^</a> ".
+               "<a\nhref=\"$mhref\">permalink</a>" .
+               " / <a\nhref=\"${mhref}raw\">raw</a>" .
+               " / <a\nhref=\"${mhref}#R\">reply</a>";
+       if (my $pct = $ctx->{pct}) { # used by SearchView.pm
                $rv .= " [relevance $pct->{$mid_raw}%]";
-       } elsif ($srch) {
+       } elsif ($mapping) {
                my $threaded = 'threaded';
                my $flat = 'flat';
                my $end = '';
                if ($ctx->{flat}) {
                        $flat = "<b>$flat</b>";
-                       $end = "\n"; # for lynx
                } else {
                        $threaded = "<b>$threaded</b>";
                }
-               $rv .= " [<a\nhref=\"${path}$href/t/#u\">$threaded</a>";
-               $rv .= "|<a\nhref=\"${path}$href/T/#u\">$flat</a>]$end";
+               $rv .= " / [<a\nhref=\"${mhref}T/#u\">$flat</a>";
+               $rv .= "|<a\nhref=\"${mhref}t/#u\">$threaded</a>]";
+               $rv .= " / <a\nhref=#r$id>$ctx->{s_nr}</a>";
        }
-       $fh->write($rv .= '</pre>');
+
+       $rv .= $more ? "\n\n" : "\n";
 }
 
-sub thread_html {
-       my ($ctx, $foot, $srch) = @_;
-       # $_[0] in sub is the Plack callback
-       sub { emit_thread_html($_[0], $ctx, $foot, $srch) }
+sub pad_link ($$;$) {
+       my ($mid, $level, $s) = @_;
+       $s ||= '...';
+       my $id = id_compress($mid, 1);
+       (' 'x19).indent_for($level).th_pfx($level)."<a\nhref=#r$id>($s)</a>\n";
+}
+
+sub _th_index_lite {
+       my ($mid_raw, $irt, $id, $ctx) = @_;
+       my $rv = '';
+       my $mapping = $ctx->{mapping} or return $rv;
+       my $pad = '  ';
+       # map = [children, attr, node, idx, level]
+       my $map = $mapping->{$mid_raw};
+       my $nr_c = scalar @{$map->[0]};
+       my $nr_s = 0;
+       my $level = $map->[4];
+       my $idx = $map->[3];
+       if (defined $irt) {
+               my $irt_map = $mapping->{$irt};
+               my $siblings = $irt_map->[0];
+               $nr_s = scalar(@$siblings) - 1;
+               $rv .= $pad . $irt_map->[1];
+               if ($idx > 0) {
+                       my $prev = $siblings->[$idx - 1];
+                       my $pmid = $prev->messageid;
+                       if ($idx > 2) {
+                               my $s = ($idx - 1). ' preceding siblings ...';
+                               $rv .= pad_link($pmid, $level, $s);
+                       } elsif ($idx == 2) {
+                               my $ppmid = $siblings->[0]->messageid;
+                               $rv .= $pad . $mapping->{$ppmid}->[1];
+                       }
+                       $rv .= $pad . $mapping->{$pmid}->[1];
+               }
+       }
+       my $s_s = nr_to_s($nr_s, 'sibling', 'siblings');
+       my $s_c = nr_to_s($nr_c, 'reply', 'replies');
+       my $this = $map->[1];
+       $this =~ s!\n\z!</b>\n!s;
+       $this =~ s!<a\nhref.*</a> !!s; # no point in duplicating subject
+       $rv .= "<b>@ $this";
+       my $node = $map->[2];
+       if (my $child = $node->child) {
+               my $cmid = $child->messageid;
+               $rv .= $pad . $mapping->{$cmid}->[1];
+               if ($nr_c > 2) {
+                       my $s = ($nr_c - 1). ' more replies';
+                       $rv .= pad_link($cmid, $level + 1, $s);
+               } elsif (my $cn = $child->next) {
+                       $rv .= $pad . $mapping->{$cn->messageid}->[1];
+               }
+       }
+       if (my $next = $node->next) {
+               my $nmid = $next->messageid;
+               $rv .= $pad . $mapping->{$nmid}->[1];
+               my $nnext = $nr_s - $idx;
+               if ($nnext > 2) {
+                       my $s = ($nnext - 1).' subsequent siblings';
+                       $rv .= pad_link($nmid, $level, $s);
+               } elsif (my $nn = $next->next) {
+                       $rv .= $pad . $mapping->{$nn->messageid}->[1];
+               }
+       }
+       $rv .= "<a\nhref=#e$id\nid=m$id>_</a> ";
+       $rv .= "<a\nhref=#r$id>$s_s, $s_c; $ctx->{s_nr}</a>\n";
 }
 
 sub walk_thread {
-       my ($th, $state, $cb) = @_;
+       my ($th, $ctx, $cb) = @_;
        my @q = map { (0, $_) } $th->rootset;
        while (@q) {
                my $level = shift @q;
                my $node = shift @q or next;
-               $cb->($state, $level, $node);
+               $cb->($ctx, $level, $node);
                unshift @q, $level+1, $node->child, $level, $node->next;
        }
 }
 
-# only private functions below.
-
-sub emit_thread_html {
-       my ($res, $ctx, $foot, $srch) = @_;
-       my $mid = $ctx->{mid};
-       my $flat = $ctx->{flat};
-       my $msgs = load_results($srch->get_thread($mid, { asc => $flat }));
-       my $nr = scalar @$msgs;
-       return missing_thread($res, $ctx) if $nr == 0;
-       my $seen = {};
-       my $state = {
-               res => $res,
-               ctx => $ctx,
-               seen => $seen,
-               root_anchor => anchor_for($mid),
-               anchor_idx => 0,
-               cur_level => 0,
-       };
-
-       require PublicInbox::Git;
-       $ctx->{git} ||= PublicInbox::Git->new($ctx->{git_dir});
-       if ($flat) {
-               pre_anchor_entry($seen, $_) for (@$msgs);
-               __thread_entry($state, $_, 0) for (@$msgs);
-       } else {
-               walk_thread(thread_results($msgs), $state, *thread_entry);
-               if (my $max = $state->{cur_level}) {
-                       $state->{fh}->write(
-                               ('</ul></li>' x ($max - 1)) . '</ul>');
-               }
+sub pre_thread  {
+       my ($ctx, $level, $node) = @_;
+       my $mapping = $ctx->{mapping};
+       my $idx = -1;
+       if (my $parent = $node->parent) {
+               my $m = $mapping->{$parent->messageid}->[0];
+               $idx = scalar @$m;
+               push @$m, $node;
        }
+       $mapping->{$node->messageid} = [ [], '', $node, $idx, $level ];
+       skel_dump($ctx, $level, $node);
+}
 
-       # there could be a race due to a message being deleted in git
-       # but still being in the Xapian index:
-       my $fh = delete $state->{fh} or return missing_thread($res, $ctx);
-
-       my $final_anchor = $state->{anchor_idx};
-       my $next = "<a\nid=s$final_anchor>";
-       $next .= $final_anchor == 1 ? 'only message in' : 'end of';
-       $next .= " thread</a>, back to <a\nhref=\"../../\">index</a>";
-       $next .= "\ndownload thread: ";
-       $next .= "<a\nhref=\"../t.mbox.gz\">mbox.gz</a>";
-       $next .= " / follow: <a\nhref=\"../t.atom\">Atom feed</a>";
-       $fh->write('<hr /><pre>' . $next . "\n\n".
-                       $foot .  '</pre></body></html>');
-       $fh->close;
+sub thread_index_entry {
+       my ($ctx, $level, $mime) = @_;
+       my ($beg, $end) = thread_adj_level($ctx, $level);
+       $beg . '<pre>' . index_entry($mime, $ctx, 0) . '</pre>' . $end;
 }
 
-sub index_walk {
-       my ($fh, $upfx, $p) = @_;
-       my $s = add_text_body($upfx, $p);
+sub stream_thread ($$) {
+       my ($th, $ctx) = @_;
+       my $inbox = $ctx->{-inbox};
+       my $mime;
+       my @q = map { (0, $_) } $th->rootset;
+       my $level;
+       while (@q) {
+               $level = shift @q;
+               my $node = shift @q or next;
+               unshift @q, $level+1, $node->child, $level, $node->next;
+               $mime = $inbox->msg_by_mid($node->messageid) and last;
+       }
+       return missing_thread($ctx) unless $mime;
 
-       return if $s eq '';
+       $mime = Email::MIME->new($mime);
+       $ctx->{-title_html} = ascii_html($mime->header('Subject'));
+       $ctx->{-html_tip} = thread_index_entry($ctx, $level, $mime);
+       my $body = PublicInbox::WwwStream->new($ctx, sub {
+               return unless $ctx;
+               while (@q) {
+                       $level = shift @q;
+                       my $node = shift @q or next;
+                       unshift @q, $level+1, $node->child, $level, $node->next;
+                       my $mid = $node->messageid;
+                       if ($mime = $inbox->msg_by_mid($mid)) {
+                               $mime = Email::MIME->new($mime);
+                               return thread_index_entry($ctx, $level, $mime);
+                       } else {
+                               return ghost_index_entry($ctx, $level, $mid);
+                       }
+               }
+               my $ret = join('', thread_adj_level($ctx, 0));
+               $ret .= ${$ctx->{dst}}; # skel
+               $ctx = undef;
+               $ret;
+       });
+       [ 200, ['Content-Type', 'text/html; charset=UTF-8'], $body ];
+}
 
-       $fh->write($s);
+sub thread_html {
+       my ($ctx) = @_;
+       my $mid = $ctx->{mid};
+       my $sres = $ctx->{srch}->get_thread($mid, { asc => 1 });
+       my $msgs = load_results($sres);
+       my $nr = $sres->{total};
+       return missing_thread($ctx) if $nr == 0;
+       my $skel = '<hr /><pre>';
+       $skel .= $nr == 1 ? 'only message in thread' : 'end of thread';
+       $skel .= ", back to <a\nhref=\"../../\">index</a>";
+       $skel .= "\n<a\nid=t>$nr+ messages in thread:</a> (download: ";
+       $skel .= "<a\nhref=\"../t.mbox.gz\">mbox.gz</a>";
+       $skel .= " / follow: <a\nhref=\"../t.atom\">Atom feed</a>)\n";
+       $ctx->{-upfx} = '../../';
+       $ctx->{cur_level} = 0;
+       $ctx->{dst} = \$skel;
+       $ctx->{prev_attr} = '';
+       $ctx->{prev_level} = 0;
+       $ctx->{root_anchor} = anchor_for($mid);
+       $ctx->{seen} = {};
+       $ctx->{mapping} = {};
+       $ctx->{s_nr} = "$nr+ messages in thread";
+
+       my $th = thread_results($msgs);
+       walk_thread($th, $ctx, *pre_thread);
+       $skel .= '</pre>';
+       return stream_thread($th, $ctx) unless $ctx->{flat};
+
+       # flat display: lazy load the full message from mini_mime:
+       my $inbox = $ctx->{-inbox};
+       my $mime;
+       while ($mime = shift @$msgs) {
+               $mime = $inbox->msg_by_mid(mid_clean(mid_mime($mime))) and last;
+       }
+       return missing_thread($ctx) unless $mime;
+       $mime = Email::MIME->new($mime);
+       $ctx->{-title_html} = ascii_html($mime->header('Subject'));
+       $ctx->{-html_tip} = '<pre>'.index_entry($mime, $ctx, scalar @$msgs);
+       $mime = undef;
+       my $body = PublicInbox::WwwStream->new($ctx, sub {
+               return unless $msgs;
+               while ($mime = shift @$msgs) {
+                       $mid = mid_clean(mid_mime($mime));
+                       $mime = $inbox->msg_by_mid($mid) and last;
+               }
+               if ($mime) {
+                       $mime = Email::MIME->new($mime);
+                       return index_entry($mime, $ctx, scalar @$msgs);
+               }
+               $msgs = undef;
+               '</pre>'.$skel;
+       });
+       [ 200, ['Content-Type', 'text/html; charset=UTF-8'], $body ];
 }
 
 sub multipart_text_as_html {
@@ -384,7 +502,7 @@ sub thread_skel {
        my $mid = mid_clean($hdr->header_raw('Message-ID'));
        my $sres = $srch->get_thread($mid);
        my $nr = $sres->{total};
-       my $expand = qq(<a\nhref="${tpfx}t/#u">expand</a> ) .
+       my $expand = qq(<a\nhref="${tpfx}T/#u">expand</a> ) .
                        qq(/ <a\nhref="${tpfx}t.mbox.gz">mbox.gz</a> ) .
                        qq(/ <a\nhref="${tpfx}t.atom">Atom feed</a>);
 
@@ -405,17 +523,12 @@ sub thread_skel {
        $$dst .= qq! / <a\nhref="#b">[top]</a>)\n!;
 
        my $subj = $srch->subject_path($hdr->header('Subject'));
-       my $state = {
-               seen => { $subj => 1 },
-               srch => $srch,
-               cur => $mid,
-               prev_attr => '',
-               prev_level => 0,
-               upfx => "$tpfx../",
-               dst => $dst,
-       };
-       walk_thread(thread_results(load_results($sres)), $state, *skel_dump);
-       $ctx->{next_msg} = $state->{next_msg};
+       $ctx->{seen} = { $subj => 1 };
+       $ctx->{cur} = $mid;
+       $ctx->{prev_attr} = '';
+       $ctx->{prev_level} = 0;
+       $ctx->{dst} = $dst;
+       walk_thread(thread_results(load_results($sres)), $ctx, *skel_dump);
        $ctx->{parent_msg} = $parent;
 }
 
@@ -543,31 +656,7 @@ sub linkify_ref_nosrch {
 
 sub anchor_for {
        my ($msgid) = @_;
-       my $id = $msgid;
-       if ($id !~ /\A[a-f0-9]{40}\z/) {
-               $id = id_compress(mid_clean($id), 1);
-       }
-       'm' . $id;
-}
-
-sub thread_html_head {
-       my ($hdr, $state) = @_;
-       my $res = delete $state->{res} or die "BUG: no Plack callback in {res}";
-       my $fh = $res->([200, ['Content-Type'=> 'text/html; charset=UTF-8']]);
-       $state->{fh} = $fh;
-
-       my $s = ascii_html($hdr->header('Subject'));
-       $fh->write("<html><head><title>$s</title>".
-               qq{<link\nrel=alternate\ntitle="Atom feed"\n} .
-               qq!href="../t.atom"\ntype="application/atom+xml"/>! .
-               PublicInbox::Hval::STYLE .
-               "</head><body>");
-}
-
-sub pre_anchor_entry {
-       my ($seen, $mime) = @_;
-       my $id = anchor_for(mid_mime($mime));
-       $seen->{$id} = "#$id"; # save the anchor for children, later
+       'm' . id_compress($msgid, 1);
 }
 
 sub ghost_parent {
@@ -581,84 +670,11 @@ sub ghost_parent {
        qq{[parent not found: &lt;<a\nhref="$upfx$href/">$html</a>&gt;]};
 }
 
-sub thread_adj_level {
-       my ($state, $level) = @_;
-
-       my $max = $state->{cur_level};
-       if ($level <= 0) {
-               return '' if $max == 0; # flat output
-
-               # reset existing lists
-               my $x = $max > 1 ? ('</ul></li>' x ($max - 1)) : '';
-               $state->{fh}->write($x . '</ul>');
-               $state->{cur_level} = 0;
-               return '';
-       }
-       if ($level == $max) { # continue existing list
-               $state->{fh}->write('<li>');
-       } elsif ($level < $max) {
-               my $x = $max > 1 ? ('</ul></li>' x ($max - $level)) : '';
-               $state->{fh}->write($x .= '<li>');
-               $state->{cur_level} = $level;
-       } else { # ($level > $max) # start a new level
-               $state->{cur_level} = $level;
-               $state->{fh}->write(($max ? '<li>' : '') . '<ul><li>');
-       }
-       '</li>';
-}
-
-sub ghost_flush {
-       my ($state, $upfx, $mid, $level) = @_;
-       my $end = '<pre>'. ghost_parent($upfx, $mid) . '</pre>';
-       $state->{fh}->write($end .= thread_adj_level($state, $level));
-}
-
-sub __thread_entry {
-       my ($state, $mime, $level) = @_;
-
-       # lazy load the full message from mini_mime:
-       $mime = eval {
-               my $mid = mid_clean(mid_mime($mime));
-               $state->{ctx}->{-inbox}->msg_by_mid($mid);
-       } or return;
-       $mime = Email::MIME->new($mime);
-
-       thread_html_head($mime, $state) if $state->{anchor_idx} == 0;
-       if (my $ghost = delete $state->{ghost}) {
-               # n.b. ghost messages may only be parents, not children
-               foreach my $g (@$ghost) {
-                       ghost_flush($state, '../../', @$g);
-               }
-       }
-       my $end = thread_adj_level($state, $level);
-       index_entry($mime, $level, $state);
-       $state->{fh}->write($end) if $end;
-
-       1;
-}
-
 sub indent_for {
        my ($level) = @_;
        INDENT x ($level - 1);
 }
 
-sub __ghost_prepare {
-       my ($state, $node, $level) = @_;
-       my $ghost = $state->{ghost} ||= [];
-       push @$ghost, [ $node->messageid, $level ];
-}
-
-sub thread_entry {
-       my ($state, $level, $node) = @_;
-       if (my $mime = $node->message) {
-               unless (__thread_entry($state, $mime, $level)) {
-                       __ghost_prepare($state, $node, $level);
-               }
-       } else {
-               __ghost_prepare($state, $node, $level);
-       }
-}
-
 sub load_results {
        my ($sres) = @_;
 
@@ -681,10 +697,9 @@ sub thread_results {
 }
 
 sub missing_thread {
-       my ($res, $ctx) = @_;
+       my ($ctx) = @_;
        require PublicInbox::ExtMsg;
-
-       $res->(PublicInbox::ExtMsg::ext_msg($ctx))
+       PublicInbox::ExtMsg::ext_msg($ctx);
 }
 
 sub _msg_date {
@@ -696,73 +711,94 @@ sub _msg_date {
 sub fmt_ts { POSIX::strftime('%Y-%m-%d %k:%M', gmtime($_[0])) }
 
 sub _skel_header {
-       my ($state, $hdr, $level) = @_;
+       my ($ctx, $hdr, $level) = @_;
 
-       my $dst = $state->{dst};
-       my $cur = $state->{cur};
+       my $dst = $ctx->{dst};
+       my $cur = $ctx->{cur};
        my $mid = mid_clean($hdr->header_raw('Message-ID'));
        my $f = ascii_html($hdr->header('X-PI-From'));
-       my $d = _msg_date($hdr);
-       my $pfx = "$d " . indent_for($level) . th_pfx($level);
+       my $d = _msg_date($hdr) . ' ' . indent_for($level) . th_pfx($level);
        my $attr = $f;
-       $state->{first_level} ||= $level;
+       $ctx->{first_level} ||= $level;
 
-       if ($attr ne $state->{prev_attr} || $state->{prev_level} > $level) {
-               $state->{prev_attr} = $attr;
+       if ($attr ne $ctx->{prev_attr} || $ctx->{prev_level} > $level) {
+               $ctx->{prev_attr} = $attr;
        } else {
                $attr = '';
        }
-       $state->{prev_level} = $level;
+       $ctx->{prev_level} = $level;
 
        if ($cur) {
                if ($cur eq $mid) {
-                       delete $state->{cur};
-                       $$dst .= "$pfx<b><a\nid=r\nhref=\"#t\">".
+                       delete $ctx->{cur};
+                       $$dst .= $d;
+                       $$dst .= "<b><a\nid=r\nhref=\"#t\">".
                                 "$attr [this message]</a></b>\n";
-
                        return;
                }
        } else {
-               $state->{next_msg} ||= $mid;
+               $ctx->{next_msg} ||= $mid;
        }
 
        # Subject is never undef, this mail was loaded from
        # our Xapian which would've resulted in '' if it were
        # really missing (and Filter rejects empty subjects)
        my $s = $hdr->header('Subject');
-       my $h = $state->{srch}->subject_path($s);
-       if ($state->{seen}->{$h}) {
+       my $h = $ctx->{srch}->subject_path($s);
+       if ($ctx->{seen}->{$h}) {
                $s = undef;
        } else {
-               $state->{seen}->{$h} = 1;
+               $ctx->{seen}->{$h} = 1;
                $s = PublicInbox::Hval->new($s);
                $s = $s->as_html;
        }
        my $m = PublicInbox::Hval->new_msgid($mid);
-       $m = $state->{upfx} . $m->as_href . '/';
-       $$dst .= "$pfx<a\nhref=\"$m\">";
-       $$dst .= defined($s) ? "$s</a> $f\n" : "$f</a>\n";
+       my $id = '';
+       my $mapping = $ctx->{mapping};
+       my $end = defined($s) ? "$s</a> $f\n" : "$f</a>\n";
+       if ($mapping) {
+               my $map = $mapping->{$mid};
+               $id = id_compress($mid, 1);
+               $m = '#m'.$id;
+               $map->[1] = "$d<a\nhref=\"$m\">$end";
+               $id = "\nid=r".$id;
+       } else {
+               $m = $ctx->{-upfx}.$m->as_href.'/';
+       }
+       $$dst .=  $d . "<a\nhref=\"$m\"$id>" . $end;
 }
 
 sub skel_dump {
-       my ($state, $level, $node) = @_;
+       my ($ctx, $level, $node) = @_;
        if (my $mime = $node->message) {
-               my $hdr = $mime->header_obj;
-               my $mid = mid_clean($hdr->header_raw('Message-ID'));
-               _skel_header($state, $hdr, $level);
+               _skel_header($ctx, $mime->header_obj, $level);
        } else {
                my $mid = $node->messageid;
-               my $dst = $state->{dst};
+               my $dst = $ctx->{dst};
+               my $mapping = $ctx->{mapping};
+               my $map = $mapping->{$mid} if $mapping;
                if ($mid eq 'subject dummy') {
-                       $$dst .= "\t[no common parent]\n";
+                       my $ncp = "\t[no common parent]\n";
+                       $map->[1] = $ncp if $map;
+                       $$dst .= $ncp;
+                       return;
+               }
+               my $d = $ctx->{pct} ? '    [irrelevant] ' # search result
+                                   : '     [not found] ';
+               $d .= indent_for($level) . th_pfx($level);
+               my $upfx = $ctx->{-upfx};
+               my $m = PublicInbox::Hval->new_msgid($mid);
+               my $href = $upfx . $m->as_href . '/';
+               my $html = $m->as_html;
+
+               if ($map) {
+                       my $id = id_compress($mid, 1);
+                       $map->[1] = $d . qq{&lt;<a\nhref=#r$id>$html</a>&gt;\n};
+                       $d .= qq{&lt;<a\nhref="$href"\nid=r$id>$html</a>&gt;\n};
                } else {
-                       $$dst .= '     [not found] ';
-                       $$dst .= indent_for($level) . th_pfx($level);
-                       $mid = PublicInbox::Hval->new_msgid($mid);
-                       my $href = $state->{upfx} . $mid->as_href . '/';
-                       my $html = $mid->as_html;
-                       $$dst .= qq{&lt;<a\nhref="$href">$html</a>&gt;\n};
+                       $d .= qq{&lt;<a\nhref="$href">$html</a>&gt;\n};
                }
+               $$dst .= $d;
        }
 }
 
@@ -782,8 +818,8 @@ sub _tryload_ghost ($$) {
 # accumulate recent topics if search is supported
 # returns 1 if done, undef if not
 sub add_topic {
-       my ($state, $level, $node) = @_;
-       my $srch = $state->{srch};
+       my ($ctx, $level, $node) = @_;
+       my $srch = $ctx->{srch};
        my $mid = $node->messageid;
        my $x = $node->message || _tryload_ghost($srch, $mid);
        my ($subj, $ts);
@@ -796,21 +832,21 @@ sub add_topic {
                $ts = -666;
                $subj = "<$mid>";
        }
-       if (++$state->{subjs}->{$subj} == 1) {
-               push @{$state->{order}}, [ $level, $subj ];
+       if (++$ctx->{subjs}->{$subj} == 1) {
+               push @{$ctx->{order}}, [ $level, $subj ];
        }
-       my $exist = $state->{latest}->{$subj};
+       my $exist = $ctx->{latest}->{$subj};
        if (!$exist || $exist->[1] < $ts) {
-               $state->{latest}->{$subj} = [ $mid, $ts ];
+               $ctx->{latest}->{$subj} = [ $mid, $ts ];
        }
 }
 
 sub emit_topics {
-       my ($state) = @_;
-       my $order = $state->{order};
-       my $subjs = $state->{subjs};
-       my $latest = $state->{latest};
-       my $fh = $state->{fh};
+       my ($ctx) = @_;
+       my $order = $ctx->{order};
+       my $subjs = $ctx->{subjs};
+       my $latest = $ctx->{latest};
+       my $fh = $ctx->{fh};
        return $fh->write("\n[No topics in range]</pre>") unless scalar @$order;
        my $pfx;
        my $prev = 0;
@@ -837,7 +873,7 @@ sub emit_topics {
                }
 
                $subj = PublicInbox::Hval->new($subj)->as_html;
-               $cur->[1] .= "<a\nhref=\"$mid/t/#u\"><b>$subj</b></a>\n";
+               $cur->[1] .= "<a\nhref=\"$mid/T/#u\"><b>$subj</b></a>\n";
                $ts = fmt_ts($ts);
                my $attr = " $ts UTC";
 
@@ -859,23 +895,53 @@ sub emit_topics {
 }
 
 sub emit_index_topics {
-       my ($state) = @_;
-       my ($off) = (($state->{ctx}->{cgi}->param('o') || '0') =~ /(\d+)/);
-       $state->{order} = [];
-       $state->{subjs} = {};
-       $state->{latest} = {};
+       my ($ctx) = @_;
+       my ($off) = (($ctx->{cgi}->param('o') || '0') =~ /(\d+)/);
+       $ctx->{order} = [];
+       $ctx->{subjs} = {};
+       $ctx->{latest} = {};
        my $max = 25;
        my %opts = ( offset => $off, limit => $max * 4 );
-       while (scalar @{$state->{order}} < $max) {
-               my $sres = $state->{srch}->query('', \%opts);
+       while (scalar @{$ctx->{order}} < $max) {
+               my $sres = $ctx->{srch}->query('', \%opts);
                my $nr = scalar @{$sres->{msgs}} or last;
                $sres = load_results($sres);
-               walk_thread(thread_results($sres), $state, *add_topic);
+               walk_thread(thread_results($sres), $ctx, *add_topic);
                $opts{offset} += $nr;
        }
 
-       emit_topics($state);
+       emit_topics($ctx);
        $opts{offset};
 }
 
+sub thread_adj_level {
+       my ($ctx, $level) = @_;
+
+       my $max = $ctx->{cur_level};
+       if ($level <= 0) {
+               return ('', '') if $max == 0; # flat output
+
+               # reset existing lists
+               my $beg = $max > 1 ? ('</ul></li>' x ($max - 1)) : '';
+               $ctx->{cur_level} = 0;
+               ("$beg</ul>", '');
+       } elsif ($level == $max) { # continue existing list
+               qw(<li> </li>);
+       } elsif ($level < $max) {
+               my $beg = $max > 1 ? ('</ul></li>' x ($max - $level)) : '';
+               $ctx->{cur_level} = $level;
+               ("$beg<li>", '</li>');
+       } else { # ($level > $max) # start a new level
+               $ctx->{cur_level} = $level;
+               my $beg = ($max ? '<li>' : '') . '<ul><li>';
+               ($beg, '</li>');
+       }
+}
+
+sub ghost_index_entry {
+       my ($ctx, $level, $mid) = @_;
+       my ($beg, $end) = thread_adj_level($ctx,  $level);
+       $beg . '<pre>'. ghost_parent($ctx->{-upfx}, $mid) . '</pre>' . $end;
+}
+
 1;
index d6b07bf7ff03be20db2abb3893a54c8c41fcb71f..cbd3142d1c62c256766ac0fe842486f1c164364a 100644 (file)
@@ -75,7 +75,8 @@ sub call {
                invalid_inbox($self, $ctx, $1) || get_index($ctx);
        } elsif ($path_info =~ m!$INBOX_RE/(?:atom\.xml|new\.atom)\z!o) {
                invalid_inbox($self, $ctx, $1) || get_atom($ctx);
-
+       } elsif ($path_info =~ m!$INBOX_RE/new\.html\z!o) {
+               invalid_inbox($self, $ctx, $1) || get_new($ctx);
        } elsif ($path_info =~ m!$INBOX_RE/
                                ($PublicInbox::GitHTTPBackend::ANY)\z!ox) {
                my $path = $2;
@@ -190,6 +191,13 @@ sub get_atom {
        PublicInbox::Feed::generate($ctx);
 }
 
+# /$INBOX/new.html                     -> HTML only
+sub get_new {
+       my ($ctx) = @_;
+       require PublicInbox::Feed;
+       PublicInbox::Feed::new_html($ctx);
+}
+
 # /$INBOX/?r=$GIT_COMMIT                 -> HTML only
 sub get_index {
        my ($ctx) = @_;
@@ -235,11 +243,10 @@ sub get_mid_html {
 # /$INBOX/$MESSAGE_ID/t/
 sub get_thread {
        my ($ctx, $flat) = @_;
-       my $srch = searcher($ctx) or return need_search($ctx);
-       require PublicInbox::View;
-       my $foot = footer($ctx);
+       searcher($ctx) or return need_search($ctx);
        $ctx->{flat} = $flat;
-       PublicInbox::View::thread_html($ctx, $foot, $srch);
+       require PublicInbox::View;
+       PublicInbox::View::thread_html($ctx);
 }
 
 sub ctx_get {
@@ -411,11 +418,11 @@ sub msg_page {
        my $ret;
        $ret = invalid_inbox_mid($self, $ctx, $inbox, $mid) and return $ret;
        '' eq $e and return get_mid_html($ctx);
+       'T/' eq $e and return get_thread($ctx, 1);
        't/' eq $e and return get_thread($ctx);
        't.atom' eq $e and return get_thread_atom($ctx);
        't.mbox' eq $e and return get_thread_mbox($ctx);
        't.mbox.gz' eq $e and return get_thread_mbox($ctx, '.gz');
-       'T/' eq $e and return get_thread($ctx, 1);
        'raw' eq $e and return get_mid_txt($ctx);
 
        # legacy, but no redirect for compatibility:
index 34f32c0bdcdb088f056baa12e383622339321c0d..d2bf318bed49a6027e68f7a696a91de1ff29de61 100644 (file)
@@ -22,10 +22,20 @@ sub _html_top ($) {
        my $title = $ctx->{-title_html} || $desc;
        my $upfx = $ctx->{-upfx} || '';
        my $atom = $ctx->{-atom} || $upfx.'new.atom';
+       my $tip = $ctx->{-html_tip} || '';
        my $top = "<b>$desc</b> (<a\nhref=\"$atom\">Atom feed</a>)";
        if ($obj->search) {
+               my $q_val = $ctx->{-q_value_html};
+               if (defined $q_val && $q_val ne '') {
+                       $q_val = qq(\nvalue="$q_val" );
+               } else {
+                       $q_val = '';
+               }
+               # XXX gross, for SearchView.pm
+               my $extra = $ctx->{-extra_form_html} || '';
                $top = qq{<form\naction="$upfx"><pre>$top} .
-                         qq{ <input\nname=q\ntype=text />} .
+                         qq{ <input\nname=q\ntype=text$q_val/>} .
+                         $extra .
                          qq{<input\ntype=submit\nvalue=search />} .
                          q{</pre></form>}
        } else {
@@ -35,7 +45,7 @@ sub _html_top ($) {
                "<link\nrel=alternate\ntitle=\"Atom feed\"\n".
                "href=\"$atom\"\ntype=\"application/atom+xml\"/>" .
                PublicInbox::Hval::STYLE .
-               "</head><body>$top";
+               "</head><body>". $top . $tip;
 }
 
 sub _html_end {