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 88d6516 Reinstate value, avoiding redefine
88d6516 is described below
commit 88d6516e2e39d301f78b89a79b59fd1b74f1e7b0
Author: Sebb <[email protected]>
AuthorDate: Wed Feb 9 15:14:46 2022 +0000
Reinstate value, avoiding redefine
---
www/roster/models/committee.rb | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/www/roster/models/committee.rb b/www/roster/models/committee.rb
index b45c540..57e2494 100644
--- a/www/roster/models/committee.rb
+++ b/www/roster/models/committee.rb
@@ -53,8 +53,8 @@ class Committee
roster = pmc.roster.dup # from committee-info
# ensure PMC members are all processed even they don't belong to the owner
group
- roster.each do |key, _|
- info[:role] = 'PMC member'
+ roster.each do |key, value|
+ value[:role] = 'PMC member'
next if pmc.ownerids.include?(key) # skip the rest (expensive) if person
is in the owner group
person = ASF::Person[key]
next unless person # in case of missing entry (e.g. renamed uid)
