[whimsy] branch master updated: Check both groups for membership

[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 99a97769 Check both groups for membership
99a97769 is described below

commit 99a97769fd4ccff179219c6f8c145e41cb9f3bf3
Author: Sebb <s...@apache.org>
AuthorDate: Tue Aug 29 11:45:59 2023 +0100

    Check both groups for membership
---
 www/secretary/ldap-check.cgi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/www/secretary/ldap-check.cgi b/www/secretary/ldap-check.cgi
index 90b7bcad..a4166614 100755
--- a/www/secretary/ldap-check.cgi
+++ b/www/secretary/ldap-check.cgi
@@ -200,7 +200,8 @@ _html do
       _th 'Login'
       _th 'Projects (if any)'
     end
-    people.select {|p| p.inactive? and cmtrol.include? 
p.name}.sort_by(&:name).each do |p|
+    people.select {|p| p.inactive? and
+      (cmtrol.include?(p.name) or 
cmtgrp.include?(p.name))}.sort_by(&:name).each do |p|
       _tr do
         _td do
           _a p.name, href: '/roster/committer/' + p.name