about summary refs log tree commit homepage
path: root/lib
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2017-06-15 23:07:58 +0000
committerEric Wong <e@80x24.org>2017-06-15 23:13:00 +0000
commit1f15ff775a967794ff3e409722b72e5684f2c0a2 (patch)
treef7482bdfe7cd439e3b4d39024145c337a16dc270 /lib
parent833c1c89b9b482452c2863c623f3e390545d50f0 (diff)
downloadpublic-inbox-1f15ff775a967794ff3e409722b72e5684f2c0a2.tar.gz
This fixes a bug introduced in
commit 7eeadcb62729b0efbcb53cd9b7b181897c92cf9a
("search: remove unnecessary abstractions and functionality")
Diffstat (limited to 'lib')
-rw-r--r--lib/PublicInbox/SearchIdx.pm8
1 files changed, 6 insertions, 2 deletions
diff --git a/lib/PublicInbox/SearchIdx.pm b/lib/PublicInbox/SearchIdx.pm
index 9ba9437d..69b7a6f6 100644
--- a/lib/PublicInbox/SearchIdx.pm
+++ b/lib/PublicInbox/SearchIdx.pm
@@ -366,8 +366,12 @@ sub remove_message {
         $mid = mid_clean($mid);
 
         eval {
-                $doc_id = $self->find_unique_doc_id('mid', $mid);
-                $db->delete_document($doc_id) if defined $doc_id;
+                $doc_id = $self->find_unique_doc_id('Q' . $mid);
+                if (defined $doc_id) {
+                        $db->delete_document($doc_id);
+                } else {
+                        warn "cannot remove non-existent <$mid>\n";
+                }
         };
 
         if ($@) {