Refactor event host partial to render a mixed array of hosts instead of directly iterating through the event_host records.

Author: rsmithlal

app/helpers/better_together/events_helper.rb

@@ -3,5 +3,12 @@
 module BetterTogether
   # View helpers for events
   module EventsHelper
+    # Return hosts for an event that the current user is authorized to view.
+    # Keeps view markup small and centralizes the policy logic for testing.
+    def visible_event_hosts(event)
+      return [] unless event.respond_to?(:event_hosts)
+
+      event.event_hosts.map { |eh| eh.host if policy(eh.host).show? }.compact
+    end
   end
 end

app/views/better_together/events/_event_host.html.erb

@@ -0,0 +1,12 @@
+<%# locals: (event:, visible_hosts: visible_event_hosts(event)) %>
+
+<% if visible_hosts.any? %>
+<%# visible_hosts is now a mixed array of host instances instead of an EventHostCollection %>
+	<div id="<%= dom_id(event, :event_hosts) %>">
+		<h3><%= t('better_together.events.hosted_by') %></h3>
+
+		<div class="mt-2 row row-cols-1 row-cols-md-2 row-cols-lg-3 g-4">
+			<%= render visible_hosts %>
+		</div>
+	</div>
+<% end %>

app/views/better_together/events/_event_hosts.html.erb

@@ -88,9 +88,7 @@
               <%= @resource.description.presence || 'No description available.' %>
             </p>
 
-						<div>
-							<%= render partial: 'better_together/events/event_host', locals: {hosts: @event.event_hosts} %>
-						</div>
+            <%= render 'better_together/events/event_hosts', event: @event %>
           </div>
         </section>
       </div>