diff options
author | Eric Wong <e@yhbt.net> | 2020-04-18 03:38:48 +0000 |
---|---|---|
committer | Eric Wong <e@yhbt.net> | 2020-04-19 08:51:21 +0000 |
commit | 95e5b6c343c45bb303db67fe7eb77e395c031e8d (patch) | |
tree | 1684f4a57a7c21df8530ea438a8ed9404e38ddd4 /lib | |
parent | b15ca9a77bff088a3f5f0b8955de8b6a60565b04 (diff) | |
download | public-inbox-95e5b6c343c45bb303db67fe7eb77e395c031e8d.tar.gz |
It's probably common to have inboxes initially setup without these files properly configured, so don't memoize at that stage.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/PublicInbox/Inbox.pm | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/lib/PublicInbox/Inbox.pm b/lib/PublicInbox/Inbox.pm index 95ffd039..e49f85fc 100644 --- a/lib/PublicInbox/Inbox.pm +++ b/lib/PublicInbox/Inbox.pm @@ -219,19 +219,22 @@ sub try_cat { sub description { my ($self) = @_; - $self->{description} //= do { + ($self->{description} //= do { my $desc = try_cat("$self->{inboxdir}/description"); local $/ = "\n"; chomp $desc; $desc =~ s/\s+/ /smg; - $desc eq '' ? '($INBOX_DIR/description missing)' : $desc; - }; + $desc eq '' ? undef : $desc; + }) // '($INBOX_DIR/description missing)'; } sub cloneurl { my ($self) = @_; - $self->{cloneurl} //= - [ split(/\s+/s, try_cat("$self->{inboxdir}/cloneurl")) ]; + ($self->{cloneurl} //= do { + my $s = try_cat("$self->{inboxdir}/cloneurl"); + my @urls = split(/\s+/s, $s); + scalar(@urls) ? \@urls : undef + }) // []; } sub base_url { |