From: Eric Wong Date: Mon, 14 Sep 2015 00:04:30 +0000 (+0000) Subject: searchview: do not link Atom feed by relevance X-Git-Tag: v1.0.0~881 X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=501730e6782d3f8839378145f56a53d205838931;p=public-inbox.git searchview: do not link Atom feed by relevance Atom feeds only make sense when sorted by time, not when our search indexing rules change and affect relevance. So do not include the relevance option when linking to Atom feeds. However, we shall still honor the 'r' query parameter in case somebody wants to manually include that in the URL for testing/experimental purposes. We simply will not advertise it. --- diff --git a/lib/PublicInbox/SearchView.pm b/lib/PublicInbox/SearchView.pm index b65351aa..6bc66ceb 100644 --- a/lib/PublicInbox/SearchView.pm +++ b/lib/PublicInbox/SearchView.pm @@ -108,7 +108,7 @@ sub search_nav_top { $rv .= qq{summary|}; $rv .= qq{threaded}; } - my $A = $q->qs_html(x => 'A'); + my $A = $q->qs_html(x => 'A', r => undef); $rv .= qq{|Atom}; $rv .= ']'; } @@ -216,7 +216,7 @@ sub html_start { my $query = PublicInbox::Hval->new_oneline($q->{q}); my $qh = $query->as_html; - my $A = $q->qs_html(x => 'A'); + my $A = $q->qs_html(x => 'A', r => undef); my $res = "$qh - search results" . qq{! .