(whimsy) branch master updated: Allow for missing name

[sebb]

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 54cf83cf Allow for missing name
54cf83cf is described below

commit 54cf83cfbf1641fb33e8664dbcdbe9e0c2028e58
Author: Sebb <s...@apache.org>
AuthorDate: Thu Feb 13 14:53:39 2025 +0000

    Allow for missing name
---
 www/roster/views/members.html.rb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/www/roster/views/members.html.rb b/www/roster/views/members.html.rb
index 328f996b..8ef8875e 100644
--- a/www/roster/views/members.html.rb
+++ b/www/roster/views/members.html.rb
@@ -79,7 +79,8 @@ _html do
           end
 
           _tbody do
-            members.sort_by {|_id, info| info[:name]}.each do |id, info|
+            # Allow for missing name (should not happen unless LDAP is 
inconsistent)
+            members.sort_by {|_id, info| info[:name] || '_'}.each do |id, info|
               _tr_ do
                 _td! do
                   if ldap.include? ASF::Person.find(id)