From 59b3f7584817735a69b45388fd2d4c40750cf6e4 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Fri, 23 Jun 2017 01:43:21 +0000 Subject: [PATCH] searchidx: fallback to lookup on pre-set article numbers Yet another hiccup from reusing pre-set article numbers on various ruby-lang.org mailing lists. This was causing messages to not appear to NNTP readers which use XOVER. --- lib/PublicInbox/SearchIdx.pm | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/PublicInbox/SearchIdx.pm b/lib/PublicInbox/SearchIdx.pm index 69b7a6f6..0824db03 100644 --- a/lib/PublicInbox/SearchIdx.pm +++ b/lib/PublicInbox/SearchIdx.pm @@ -478,7 +478,12 @@ sub unindex_blob { sub index_mm { my ($self, $mime) = @_; - $self->{mm}->mid_insert(mid_clean(mid_mime($mime))); + my $mid = mid_clean(mid_mime($mime)); + my $mm = $self->{mm}; + my $num = $mm->mid_insert($mid); + + # fallback to num_for since filters like RubyLang set the number + defined $num ? $num : $mm->num_for($mid); } sub unindex_mm { -- 2.44.0