@@ -1,29 +1,29 @@ <table> - <% @group.people_names_and_ids.each do |person| %> - <% if person and @logged_in.sees? person %> - <tr> - <td> - <a href="<%= url_for person %>"> - <%= render :partial => 'people/thumbnail', :locals => {:person => person} %> - </a> - </td> - <td class="person"> - <%= link_to h(person.name), person %> - <% if @group.admin?(person, true) %><br />(admin)<% end %> - <% if @logged_in.can_edit?(@group) and (person.email.to_s.empty? or !@group.get_options_for(person).get_email?) %> - <em><br />(no mail)</em> - <% end %> - - <% if person.birthday_soon? %><span class="small"> - <br/> - <%= image_tag('birthday.png', :alt => 'Birthday', :class => 'icon') %> - <%= person.birthday.strftime('%m/%d') %></span> - <% end %> - - - </span> - </td> - </tr> + <% @group.people.each do |person| %> + <% if person and @logged_in.can_see?(person) %> + <tr> + <td> + <a href="<%= url_for person %>"> + <%= render :partial => 'people/thumbnail', :locals => {:person => person} %> + </a> + </td> + <td class="person"> + <%= link_to h(person.name), person %> + <% if @group.admin?(person, true) %> + <br/>(admin) + <% end %> + <% if @logged_in.can_edit?(@group) and (person.email.to_s.empty? or !@group.get_options_for(person).get_email?) %> + <br/><em>(no mail)</em> + <% end %> + <% if person.birthday_soon? %> + <br/> + <span class="small"> + <%= image_tag('birthday.png', :alt => 'Birthday', :class => 'icon') %> + <%= person.birthday.strftime('%m/%d') %> + </span> + <% end %> + </td> + </tr> <% end %> <% end %> </table> app/views/groups/_people.erb c30a91d1b0872668341bd9a8edc4c3a781760bc8 Tim Morgan tim@timmorgan.org http://github.com/seven1m/onebody/commit/42ae62d5b7c64246d7bc495cf4b87e2811f3b099 42ae62d5b7c64246d7bc495cf4b87e2811f3b099 2008-10-15T15:09:42-07:00 2008-10-15T06:14:00-07:00 Don't use people_names_and_ids in group people partial (fix bug with missing people for non-admins). e62dc1c731de68c88aa70e2020871d7c2bf1a2d2 Tim Morgan tim@timmorgan.org