You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by se...@apache.org on 2022/02/09 15:14:53 UTC
[whimsy] branch master updated: Reinstate value, avoiding redefine
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 <se...@apache.org>
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)