whimsical-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From s...@apache.org
Subject [whimsy] branch master updated: Allow member_apps match on availd
Date Wed, 27 May 2020 11:02:15 GMT
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 84289bc  Allow member_apps match on availd
84289bc is described below

commit 84289bce68e07ad50e6a811e6b1e69e3e1f9937f
Author: Sebb <sebb@apache.org>
AuthorDate: Wed May 27 12:01:59 2020 +0100

    Allow member_apps match on availd
---
 lib/whimsy/asf/memapps.rb      | 1 +
 www/secretary/memapp_check.cgi | 1 +
 2 files changed, 2 insertions(+)

diff --git a/lib/whimsy/asf/memapps.rb b/lib/whimsy/asf/memapps.rb
index 0d56f0e..2e0660d 100644
--- a/lib/whimsy/asf/memapps.rb
+++ b/lib/whimsy/asf/memapps.rb
@@ -61,6 +61,7 @@ module ASF
       [
         (person.icla.legal_name rescue nil),
         (person.icla.name rescue nil),
+        person.id, # allow match on avalid
         person.member_name # this is slow
       ].uniq.each do |name|
         next unless name
diff --git a/www/secretary/memapp_check.cgi b/www/secretary/memapp_check.cgi
index 7f108ff..3bd87f6 100755
--- a/www/secretary/memapp_check.cgi
+++ b/www/secretary/memapp_check.cgi
@@ -14,6 +14,7 @@ require 'wunderbar'
 # Dummy class for members that don't have ids but do have membership apps
 class PersonNoId
   attr_reader :member_name
+  attr_reader :id # needed for memapps.rb
   def initialize name
     @member_name = name
   end


Mime
View raw message