[whimsy] branch master updated: WHIMSY-75 Non-members listed give "Internal Server Error"

[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  7623f80   WHIMSY-75 Non-members listed give "Internal Server Error"
7623f80 is described below

commit 7623f80277d12aec755c0c36ed9eec15e87c6069
Author: Sebb <se...@apache.org>
AuthorDate: Mon Mar 6 23:06:36 2017 +0000

    WHIMSY-75 Non-members listed give "Internal Server Error"
---
 www/roster/models/committer.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/www/roster/models/committer.rb b/www/roster/models/committer.rb
index 7a48803..6daed41 100644
--- a/www/roster/models/committer.rb
+++ b/www/roster/models/committer.rb
@@ -94,7 +94,8 @@ class Committer
           [
             person.icla.legal_name, 
             person.icla.name,
-            member[:info].split("\n").first.strip
+            # allow for member in LDAP to not be in members.txt (e.g. infra staff)
+            (member[:info] or "?\n").split("\n").first.strip
           ].uniq.each do |name|
             next unless name
             memapp = name.downcase.gsub(/\s/, '-').untaint

-- 
To stop receiving notification emails like this one, please contact
['"commits@whimsical.apache.org" <co...@whimsical.apache.org>'].