[whimsy] branch master updated: Protect against crash

[se...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new f249451  Protect against crash
f249451 is described below

commit f2494515c05f5ed0fd5efad3282463adbba410ce
Author: Sebb <se...@apache.org>
AuthorDate: Wed Apr 7 00:09:11 2021 +0100

    Protect against crash
---
 www/roster/models/committee.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/roster/models/committee.rb b/www/roster/models/committee.rb
index 824e387..34d24a8 100644
--- a/www/roster/models/committee.rb
+++ b/www/roster/models/committee.rb
@@ -58,7 +58,7 @@ class Committee
       info[:role] = 'PMC member'
       next if pmc.ownerids.include?(key) # skip the rest (expensive) if person is in the owner group
       person = ASF::Person[key]
-      if analysePrivateSubs
+      if analysePrivateSubs && person # in case of missing entry (e.g. renamed uid)
         # Analyse the subscriptions, matching against canonicalised personal emails
         allMail = person.all_mail.map{|m| ASF::Mail.to_canonical(m.downcase)}
         # pSubs is already downcased