You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by Sam Ruby <ru...@apache.org> on 2016/03/04 22:12:09 UTC
[whimsy.git] [2/2] Commit cf91355: make variables necessary to produce email visible
Commit cf9135539b52eec4a3acef0b86dc4372dd28953d:
make variables necessary to produce email visible
Branch: refs/heads/master
Author: Sam Ruby <ru...@intertwingly.net>
Committer: Sam Ruby <ru...@intertwingly.net>
Pusher: rubys <ru...@apache.org>
------------------------------------------------------------
www/roster/views/actions/committee.json.rb | ++++ ----
------------------------------------------------------------
8 changes: 4 additions, 4 deletions.
------------------------------------------------------------
diff --git a/www/roster/views/actions/committee.json.rb b/www/roster/views/actions/committee.json.rb
index 72d4321..87c1275 100644
--- a/www/roster/views/actions/committee.json.rb
+++ b/www/roster/views/actions/committee.json.rb
@@ -1,10 +1,10 @@
if env.password
+ person = ASF::Person.find(@id)
+ pmc = ASF::Committee.find(@pmc) if @targets.include? 'pmc'
+ group = ASF::Group.find(@pmc) if @targets.include? 'commit'
+
# update LDAP
ASF::LDAP.bind(env.user, env.password) do
- person = ASF::Person.find(@id)
- pmc = ASF::Committee.find(@pmc) if @targets.include? 'pmc'
- group = ASF::Group.find(@pmc) if @targets.include? 'commit'
-
if @action == 'add'
pmc.add(person) if pmc
group.add(person) if group