You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whimsical.apache.org by ru...@apache.org on 2017/09/08 19:49:34 UTC

[whimsy] branch master updated: fix podling removals

This is an automated email from the ASF dual-hosted git repository.

rubys 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 c978462  fix podling removals
c978462 is described below

commit c978462d19f86711baf5f2df0596a6776db62696
Author: Sam Ruby <ru...@intertwingly.net>
AuthorDate: Fri Sep 8 15:49:25 2017 -0400

    fix podling removals
---
 www/roster/views/actions/ppmc.json.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/www/roster/views/actions/ppmc.json.rb b/www/roster/views/actions/ppmc.json.rb
index 4ab37a4..614f91f 100644
--- a/www/roster/views/actions/ppmc.json.rb
+++ b/www/roster/views/actions/ppmc.json.rb
@@ -43,11 +43,11 @@ if env.password
           else
             removals = people & icommit
             podlings = ASF::Podling.current.map(&:id)
-            removals.select do |person| 
+            removals.select! do |person| 
               not incubator.owners.include? person and
               (person.projects.map(&:id) & podlings).empty?
             end
-            incubator.remove_committers(removals) unless removals.empty?
+            incubator.remove_members(removals) unless removals.empty?
           end
         end
       end

-- 
To stop receiving notification emails like this one, please contact
['"commits@whimsical.apache.org" <co...@whimsical.apache.org>'].