Doc, reorder variables to make purpose explicit

This commit is contained in:
Matt Todd 2015-01-15 13:37:50 -08:00
Родитель cb4bd2ce9f
Коммит 909da53b87
1 изменённых файлов: 7 добавлений и 2 удалений

Просмотреть файл

@ -34,9 +34,11 @@ module GitHub
# #
# Returns Array of Net::LDAP::Entry objects. # Returns Array of Net::LDAP::Entry objects.
def perform(group) def perform(group)
found = Hash.new # track groups found
found = Hash.new
# track DNs searched for (so we don't repeat searches)
searched = [] searched = []
entries = []
# if this is a posixGroup, return members immediately (no nesting) # if this is a posixGroup, return members immediately (no nesting)
uids = member_uids(group) uids = member_uids(group)
@ -89,6 +91,9 @@ module GitHub
end end
end end
# entries to return
entries = []
# pull member DNs, discarding dupes and subgroup DNs # pull member DNs, discarding dupes and subgroup DNs
member_dns = found.values.each_with_object([]) do |group, member_dns| member_dns = found.values.each_with_object([]) do |group, member_dns|
entries << group entries << group