Add locals declarations in several views and partial calls

app/controllers/subscriptions_controller.rb

@@ -12,6 +12,8 @@ class SubscriptionsController < ApplicationController
   def new
     @subscription = @subscribable.subscriptions.new(user: current_user)
     authorize @subscription
+
+    render :new, locals: { subscription: @subscription }
   end
 
   # POST /subscribable/:subscribable_id/subscriptions
@@ -35,7 +37,7 @@ def create
       render :create, locals: { subscription: @subscription, subscribable: @subscribable, protocol: protocol }
     else
       if @subscription.subscribable.is_a?(Event)
-        render :new, status: :unprocessable_entity
+        render :new, locals: { subscription: @subscription }, status: :unprocessable_entity
       else
         flash.now[:danger] = @subscription.errors.full_messages.to_sentence
         render turbo_stream: turbo_stream.replace("flash", partial: "layouts/flash")

app/views/event_groups/finish_line.html.erb

@@ -18,7 +18,7 @@
             <li class="breadcrumb-item active">Finish Line</li>
           </ul>
         </div>
-        <%= render "events/time_and_course_info" %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/event_groups/follow.html.erb

@@ -15,7 +15,7 @@
             <li class="breadcrumb-item">Follow</li>
           </ul>
         </div>
-        <%= render 'events/time_and_course_info' %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/event_groups/raw_times.html.erb

@@ -16,7 +16,7 @@
             <li class="breadcrumb-item active">List</li>
           </ul>
         </div>
-        <%= render "events/time_and_course_info" %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/event_groups/roster.html.erb

@@ -20,7 +20,7 @@
             <li class="breadcrumb-item active">Roster</li>
           </ul>
         </div>
-        <%= render "events/time_and_course_info" %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/event_groups/split_raw_times.html.erb

@@ -16,7 +16,7 @@
             <li class="breadcrumb-item active">Splits</li>
           </ul>
         </div>
-        <%= render 'events/time_and_course_info' %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/event_groups/stats.html.erb

@@ -17,7 +17,7 @@
             <li class="breadcrumb-item active">Stats</li>
           </ul>
         </div>
-        <%= render 'events/time_and_course_info' %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/event_groups/traffic.html.erb

@@ -16,7 +16,7 @@
             <li class="breadcrumb-item active"><%= @presenter.split_name %></li>
           </ul>
         </div>
-        <%= render 'events/time_and_course_info' %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/event_groups/webhooks.html.erb

@@ -15,11 +15,11 @@
             <li class="breadcrumb-item">Webhooks</li>
           </ul>
         </div>
-        <%= render "events/time_and_course_info" %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->
-    <%= render "events/view_buttons", view_object: @presenter %>
+    <%= render partial: "events/view_buttons", locals: { view_object: @presenter } %>
   </div>
 </header>
 
@@ -50,7 +50,7 @@
           <div class="col">
             <h4>
               <span class="fw-bold"><%= event.name %></span>
-              <%= render "subscriptions/subscriptions_count", subscribable: event, current_user: current_user %>
+              <%= render partial: "subscriptions/subscriptions_count", locals: { subscribable: event, current_user: current_user } %>
             </h4>
           </div>
           <div class="col-4 col-md-2 text-end d-grid">

app/views/events/_time_and_course_info.html.erb

@@ -1,7 +1,9 @@
+<%# locals: (presenter:) %>
+
 <div class="ost-subtitle">
-  <p><%= l(@presenter.scheduled_start_time_local, format: :full_day_time_and_zone) %>
-    <% if @presenter.respond_to?(:course) %>
-      on <%= link_to @presenter.course_name, organization_course_path(@presenter.organization, @presenter.course) %>
+  <p><%= l(presenter.scheduled_start_time_local, format: :full_day_time_and_zone) %>
+    <% if presenter.respond_to?(:course) %>
+      on <%= link_to presenter.course_name, organization_course_path(presenter.organization, presenter.course) %>
     <% end %>
   </p>
 </div>

app/views/events/_view_buttons.html.erb

@@ -1,3 +1,5 @@
+<%# locals: (view_object:) %>
+
 <ul class="nav nav-tabs nav-tabs-ost d-print-none">
   <%= results_dropdown_menu(view_object) %>
   <%= raw_times_dropdown_menu(view_object) if current_user&.authorized_to_edit?(view_object.event_group) %>

app/views/events/finish_history.html.erb

@@ -18,7 +18,7 @@
             <li class="breadcrumb-item active">Finish history</li>
           </ul>
         </div>
-        <%= render "time_and_course_info" %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/events/podium.html.erb

@@ -18,7 +18,7 @@
             <li class="breadcrumb-item active">Podium</li>
           </ul>
         </div>
-        <%= render "time_and_course_info" %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/events/spread.html.erb

@@ -18,7 +18,7 @@
             <li class="breadcrumb-item active">Full results</li>
           </ul>
         </div>
-        <%= render "time_and_course_info" %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
       <aside class="col-auto">
         <%= link_to_beacon_button(@presenter) %>

app/views/events/summary.html.erb

@@ -18,7 +18,7 @@
             <li class="breadcrumb-item active">Results summary</li>
           </ul>
         </div>
-        <%= render "time_and_course_info" %>
+        <%= render partial: "events/time_and_course_info", locals: { presenter: @presenter } %>
       </div>
     </div>
     <!-- Navigation -->

app/views/subscriptions/_subscriptions_count.html.erb

@@ -1,3 +1,5 @@
 <%# locals: (subscribable:, current_user:) %>
 
-<span id="<%= dom_id(subscribable, :subscriptions_count) %>" class="h5 text-muted"><%= "#{pluralize_with_delimiter(subscribable.subscriptions.for_user(current_user).count, 'subscription')}" %></span>
+<span id="<%= dom_id(subscribable, :subscriptions_count) %>" class="h5 text-muted">
+  <%= "#{pluralize_with_delimiter(subscribable.subscriptions.for_user(current_user).count, 'subscription')}" %>
+</span>

app/views/subscriptions/new.html.erb

@@ -5,5 +5,5 @@
 <% end %>
 
 <article class="ost-article container">
-  <%= render "new_modal", subscription: @subscription %>
+  <%= render partial: "new_modal", locals: { subscription: subscription } %>
 </article>