diff options
author | Eric Wong <e@80x24.org> | 2021-09-25 22:16:45 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-09-26 00:06:26 +0000 |
commit | 69225ecae508b9bb83960ac51e38d7d5eade2a6a (patch) | |
tree | d50e0ef5ad5322b42f63d03ba0b4576908ae35cb /lib/PublicInbox/AdminEdit.pm | |
parent | 71f6861703fde6f6687b002cd15c38cb7ef4a028 (diff) | |
download | public-inbox-69225ecae508b9bb83960ac51e38d7d5eade2a6a.tar.gz |
`undef' entries still take up a slot in the hash table, and cause the `exists' check to false-positive in ->cleanup_shards. This should fully fix the (innocuous) messages introduced in commit 63d7b8ce (daemons: revamp periodic cleanup task, 2021-09-23)
Diffstat (limited to 'lib/PublicInbox/AdminEdit.pm')
-rw-r--r-- | lib/PublicInbox/AdminEdit.pm | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/PublicInbox/AdminEdit.pm b/lib/PublicInbox/AdminEdit.pm index 2f6707d8..c8c3d3e8 100644 --- a/lib/PublicInbox/AdminEdit.pm +++ b/lib/PublicInbox/AdminEdit.pm @@ -27,8 +27,9 @@ sub check_editable ($) { # Make sure it's purged in that case: $ibx->over or die "no over.sqlite3 in $ibx->{inboxdir}\n"; - # $ibx->{search} is populated by $ibx->over call - my $xdir_ro = $ibx->{search}->xdir(1); + require PublicInbox::Search; + my $xdir_ro = PublicInbox::Search->new($ibx)->xdir(1); + my $nshard = 0; foreach my $shard (<$xdir_ro/*>) { if (-d $shard && $shard =~ m!/[0-9]+\z!) { |