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/05 00:47:14 UTC

[whimsy.git] [1/1] Commit 6722ecb: refetch even if things go wrong

Commit 6722ecbe541954975f88982171bd8b52981ffbd5:
    refetch even if things go wrong


Branch: refs/heads/master
Author: Sam Ruby <ru...@intertwingly.net>
Committer: Sam Ruby <ru...@intertwingly.net>
Pusher: rubys <ru...@apache.org>

------------------------------------------------------------
lib/whimsy/asf/ldap.rb                                       | ++++ 
------------------------------------------------------------
4 changes: 4 additions, 0 deletions.
------------------------------------------------------------


diff --git a/lib/whimsy/asf/ldap.rb b/lib/whimsy/asf/ldap.rb
index 4ac1e09..7e89b61 100644
--- a/lib/whimsy/asf/ldap.rb
+++ b/lib/whimsy/asf/ldap.rb
@@ -465,6 +465,7 @@ def remove(people)
       people = Array(people).map(&:id)
       mod = ::LDAP::Mod.new(::LDAP::LDAP_MOD_DELETE, 'memberUid', people)
       ASF.ldap.modify(self.dn, [mod])
+    ensure
       @members = nil
     end
 
@@ -472,6 +473,7 @@ def add(people)
       people = Array(people).map(&:id)
       mod = ::LDAP::Mod.new(::LDAP::LDAP_MOD_ADD, 'memberUid', people)
       ASF.ldap.modify(self.dn, [mod])
+    ensure
       @members = nil
     end
   end
@@ -517,6 +519,7 @@ def remove(people)
       people = Array(people).map(&:dn)
       mod = ::LDAP::Mod.new(::LDAP::LDAP_MOD_DELETE, 'member', people)
       ASF.ldap.modify(self.dn, [mod])
+    ensure
       @members = nil
     end
 
@@ -524,6 +527,7 @@ def add(people)
       people = Array(people).map(&:dn)
       mod = ::LDAP::Mod.new(::LDAP::LDAP_MOD_ADD, 'member', people)
       ASF.ldap.modify(self.dn, [mod])
+    ensure
       @members = nil
     end
   end