From: Eric Wong <e@80x24.org>
Date: Fri, 23 Jun 2017 01:43:21 +0000 (+0000)
Subject: searchidx: fallback to lookup on pre-set article numbers
X-Git-Tag: v1.0.0~47
X-Git-Url: http://www.git.stargrave.org/?p=public-inbox.git;a=commitdiff_plain;h=59b3f7584817735a69b45388fd2d4c40750cf6e4

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.
---

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 {