Reuse code: only set up :members in one method
diff --git a/lib/whimsy/asf/ldap.rb b/lib/whimsy/asf/ldap.rb
index 49ce8ee..d537bcc 100644
--- a/lib/whimsy/asf/ldap.rb
+++ b/lib/whimsy/asf/ldap.rb
@@ -1167,11 +1167,7 @@
# list of committers on this project. Stored in LDAP as a <tt>member</tt>
# attribute.
def members
- members = weakref(:members) do
- ASF.search_one(base, "cn=#{name}", 'member').flatten
- end
-
- members.map {|uid| Person.find uid[/uid=(.*?),/,1]}
+ memberids.map {|id| Person.find id}
end
# list of member ids in the project
