From fc92ce8845ac5f09939722537624fa48441f7c0b Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Fri, 3 Apr 2020 05:06:59 +0000 Subject: [PATCH] view: handle the topic-free case properly There may be no topics for a given timestamp range, so don't attempt to treat `undef' as an arrayref. --- lib/PublicInbox/View.pm | 2 +- t/psgi_v2.t | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index 89174296..c42654b6 100644 --- a/lib/PublicInbox/View.pm +++ b/lib/PublicInbox/View.pm @@ -1056,7 +1056,7 @@ sub acc_topic { # walk_thread callback sub dump_topics { my ($ctx) = @_; my $order = delete $ctx->{order}; # [ ds, subj1, subj2, subj3, ... ] - if (!@$order) { + unless ($order) { $ctx->{-html_tip} = '
[No topics in range]
'; return 404; } diff --git a/t/psgi_v2.t b/t/psgi_v2.t index 2c9387a4..c4f80869 100644 --- a/t/psgi_v2.t +++ b/t/psgi_v2.t @@ -250,6 +250,8 @@ test_psgi(sub { $www->call(@_) }, sub { is($old->content, 'old', 'got expected old content'); is($new->content, 'new', 'got expected new content'); } + $res = $cb->(GET('/v2test/?t=1970'.'01'.'01'.'000000')); + is($res->code, 404, '404 for out-of-range t= param'); }); done_testing(); -- 2.44.0