]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/SearchView.pm
searchview: increase limit for displaying search results
[public-inbox.git] / lib / PublicInbox / SearchView.pm
index 0d54c3df5c35c9ec485eab8ed7fb17cbb2ac108f..f1c4b6a052181e28c28127403a07d3732972151b 100644 (file)
@@ -8,11 +8,12 @@ use warnings;
 use PublicInbox::SearchMsg;
 use PublicInbox::Hval qw/ascii_html/;
 use PublicInbox::View;
+use PublicInbox::WwwAtomStream;
 use PublicInbox::MID qw(mid2path mid_mime mid_clean mid_escape);
-use Email::MIME;
+use PublicInbox::MIME;
 require PublicInbox::Git;
 require PublicInbox::SearchThread;
-our $LIM = 50;
+our $LIM = 200;
 
 sub noop {}
 
@@ -46,7 +47,7 @@ sub sres_top_html {
                $cb = *noop;
        } else {
                my $x = $q->{x};
-               return sub { adump($_[0], $mset, $q, $ctx) } if ($x eq 'A');
+               return adump($_[0], $mset, $q, $ctx) if $x eq 'A';
 
                $ctx->{-html_tip} = search_nav_top($mset, $q) . "\n\n";
                if ($x eq 't') {
@@ -58,6 +59,17 @@ sub sres_top_html {
        PublicInbox::WwwStream->response($ctx, $code, $cb);
 }
 
+# allow undef for individual doc loads...
+sub load_doc_retry {
+       my ($srch, $mitem) = @_;
+
+       eval {
+               $srch->retry_reopen(sub {
+                       PublicInbox::SearchMsg->load_doc($mitem->get_document)
+               });
+       }
+}
+
 # display non-threaded search results similar to what users expect from
 # regular WWW search engines:
 sub mset_summary {
@@ -67,10 +79,18 @@ sub mset_summary {
        my $pad = length("$total");
        my $pfx = ' ' x $pad;
        my $res = \($ctx->{-html_tip});
+       my $srch = $ctx->{srch};
        foreach my $m ($mset->items) {
                my $rank = sprintf("%${pad}d", $m->get_rank + 1);
                my $pct = $m->get_percent;
-               my $smsg = PublicInbox::SearchMsg->load_doc($m->get_document);
+               my $smsg = load_doc_retry($srch, $m);
+               unless ($smsg) {
+                       eval {
+                               $m = "$m ".$m->get_docid . " expired\n";
+                               $ctx->{env}->{'psgi.errors'}->print($m);
+                       };
+                       next;
+               }
                my $s = ascii_html($smsg->subject);
                my $f = ascii_html($smsg->from_name);
                my $ts = PublicInbox::View::fmt_ts($smsg->ts);
@@ -125,15 +145,21 @@ sub search_nav_bot {
        my $o = $q->{o};
        my $end = $o + $nr;
        my $beg = $o + 1;
-       my $rv = "</pre><hr><pre>Results $beg-$end of $total";
+       my $rv = '</pre><hr><pre>';
+       if ($beg <= $end) {
+               $rv .= "Results $beg-$end of $total";
+               $rv .= ' (estimated)' if $end != $total;
+       } else {
+               $rv .= "No more results, only $total";
+       }
        my $n = $o + $LIM;
 
        if ($n < $total) {
                my $qs = $q->qs_html(o => $n);
-               $rv .= qq{, <a\nhref="?$qs"\nrel=next>next</a>}
+               $rv .= qq{  <a\nhref="?$qs"\nrel=next>next</a>}
        }
        if ($o > 0) {
-               $rv .= $n < $total ? '/' : ',      ';
+               $rv .= $n < $total ? '/' : '       ';
                my $p = $o - $LIM;
                my $qs = $q->qs_html(o => ($p > 0 ? $p : 0));
                $rv .= qq{<a\nhref="?$qs"\nrel=prev>prev</a>};
@@ -141,29 +167,27 @@ sub search_nav_bot {
        $rv .= '</pre>';
 }
 
+sub sort_relevance {
+       my ($pct) = @_;
+       sub {
+               [ sort { (eval { $pct->{$b->topmost->{id}} } || 0)
+                               <=>
+                       (eval { $pct->{$a->topmost->{id}} } || 0)
+       } @{$_[0]} ] };
+}
+
 sub mset_thread {
        my ($ctx, $mset, $q) = @_;
        my %pct;
-       my @m = map {
+       my $msgs = $ctx->{srch}->retry_reopen(sub { [ map {
                my $i = $_;
-               my $m = PublicInbox::SearchMsg->load_doc($i->get_document);
-               $pct{$m->mid} = $i->get_percent;
-               $m = $m->mini_mime;
-               $m;
-       } ($mset->items);
+               my $smsg = PublicInbox::SearchMsg->load_doc($i->get_document);
+               $pct{$smsg->mid} = $i->get_percent;
+               $smsg;
+       } ($mset->items) ]});
 
-       my $th = PublicInbox::SearchThread->new(\@m);
-       $th->thread;
-       if ($q->{r}) { # order by relevance
-               $th->order(sub {
-                       [ sort { (eval { $pct{$b->topmost->messageid} } || 0)
-                                       <=>
-                               (eval { $pct{$a->topmost->messageid} } || 0)
-                       } @{$_[0]} ];
-               });
-       } else { # order by time (default for threaded view)
-               $th->order(*PublicInbox::View::sort_ts);
-       }
+       my $rootset = PublicInbox::SearchThread::thread($msgs,
+               $q->{r} ? sort_relevance(\%pct) : *PublicInbox::View::sort_ts);
        my $skel = search_nav_bot($mset, $q). "<pre>";
        my $inbox = $ctx->{-inbox};
        $ctx->{-upfx} = '';
@@ -175,21 +199,19 @@ sub mset_thread {
        $ctx->{prev_attr} = '';
        $ctx->{prev_level} = 0;
        $ctx->{seen} = {};
-       $ctx->{s_nr} = scalar(@m).'+ results';
+       $ctx->{s_nr} = scalar(@$msgs).'+ results';
 
-       PublicInbox::View::walk_thread($th, $ctx,
+       PublicInbox::View::walk_thread($rootset, $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;
+                       $mime = $inbox->msg_by_smsg($mime) and last;
                }
                if ($mime) {
-                       $mime = Email::MIME->new($mime);
+                       $mime = PublicInbox::MIME->new($mime);
                        return PublicInbox::View::index_entry($mime, $ctx,
                                scalar @$msgs);
                }
@@ -215,23 +237,18 @@ sub ctx_prepare {
 
 sub adump {
        my ($cb, $mset, $q, $ctx) = @_;
-       my $fh = $cb->([ 200, ['Content-Type' => 'application/atom+xml']]);
        my $ibx = $ctx->{-inbox};
-       my $feed_opts = PublicInbox::Feed::get_feedopts($ctx);
-       my $x = ascii_html($q->{'q'});
-       $x = qq{$x - search results};
-       $feed_opts->{atomurl} = $feed_opts->{url} . '?'. $q->qs_html;
-       $feed_opts->{url} .= '?'. $q->qs_html(x => undef);
-       $x = PublicInbox::Feed::atom_header($feed_opts, $x);
-       $fh->write($x. PublicInbox::Feed::feed_updated());
-
-       for ($mset->items) {
-               $x = PublicInbox::SearchMsg->load_doc($_->get_document)->mid;
-               $x = mid2path($x);
-               my $s = PublicInbox::Feed::feed_entry($feed_opts, $x, $ibx);
-               $fh->write($s) if defined $s;
-       }
-       PublicInbox::Feed::end_feed($fh);
+       my @items = $mset->items;
+       $ctx->{search_query} = $q;
+       my $srch = $ctx->{srch};
+       PublicInbox::WwwAtomStream->response($ctx, 200, sub {
+               while (my $x = shift @items) {
+                       $x = load_doc_retry($srch, $x);
+                       $x = $ibx->msg_by_smsg($x) and
+                                       return PublicInbox::MIME->new($x);
+               }
+               return undef;
+       });
 }
 
 package PublicInbox::SearchQuery;