[PATCH] lei/store: do not put NULL into over.num column

[Eric Wong]

Simplify oid2docid and filter out undefined docids in ->add_eml,
instead.  This avoids SQLite "datatype mismatch" errors in
OverIdx->add_over

Fixes: d1052f03ea85d4af ("lei/store: cull redundant docids based on blob OID")
---
 lib/PublicInbox/LeiStore.pm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/PublicInbox/LeiStore.pm b/lib/PublicInbox/LeiStore.pm
index 4ba1e647..e26b622d 100644
--- a/lib/PublicInbox/LeiStore.pm
+++ b/lib/PublicInbox/LeiStore.pm
@@ -269,7 +269,7 @@ W: $oid indexed as multiple docids: $docid @cull, culling 
to fixup old bugs
 EOF
                remove_docids($self, @cull);
        }
-       wantarray ? ($docid) : $docid;
+       $docid;
 }
 
 sub add_eml {
@@ -292,8 +292,8 @@ sub add_eml {
                if (scalar keys %$xoids) {
                        my %docids = map { $_ => 1 } @$vivify_xvmd;
                        for my $oid (keys %$xoids) {
-                               my @id = oid2docid($self, $oid);
-                               @docids{@id} = @id;
+                               my $docid = oid2docid($self, $oid);
+                               $docids{$docid} = $docid if defined($docid);
                        }
                        @$vivify_xvmd = sort { $a <=> $b } keys(%docids);
                }
--
unsubscribe: one-click, see List-Unsubscribe header
archive: https://public-inbox.org/meta/