Update variable names for clarity

PRX · Apr 20, 2023 · de9a00e · de9a00e
1 parent 84c316a
commit de9a00e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/app/helpers/podcast_planner_helper.rb b/app/helpers/podcast_planner_helper.rb
@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 module PodcastPlannerHelper
-  PERIODIC_WEEKS = I18n.t([:every_one, :every_two, :every_three, :every_four], scope: "podcast_planner.helper.period_options")
-  MONTHLY_WEEKS = I18n.t([:first, :second, :third, :fourth, :fifth], scope: "podcast_planner.helper.monthly_options")
+  PERIODIC_WEEKS = I18n.t([:every_one, :every_two, :every_three, :every_four], scope: [:podcast_planner, :helper, :period_options])
+  MONTHLY_WEEKS = I18n.t([:first, :second, :third, :fourth, :fifth], scope: [:podcast_planner, :helper, :monthly_options])
   DATE_CONTROLLER = "date"
   TOGGLE_ACTION = "click->date#toggleSelect"
   HIGHLIGHT_ACTION = "mouseover->date#highlight"

diff --git a/app/views/podcast_planner/_calendar.html.erb b/app/views/podcast_planner/_calendar.html.erb
@@ -1,6 +1,6 @@
 <div class="col">
-  <%= month_calendar(start_date: month) do |date| %>
-    <% if date_is_in_month?(date, month.month) %>
+  <%= month_calendar(start_date: start_date) do |date| %>
+    <% if date_is_in_month?(date, start_date.month) %>
       <%= date.day %>
 
       <%= form.hidden_field :selected_dates, multiple: true, data: {date_target: "preselected"}, value: date.to_datetime, disabled: !is_preselected_date?(date, @planner.dates), draft: is_preselected_date?(date, @draft_dates) %>

diff --git a/app/views/podcast_planner/_form_side.html.erb b/app/views/podcast_planner/_form_side.html.erb
@@ -34,7 +34,7 @@
           <%= turbo_frame_tag "preview", target: "_top" do %>
             <div class="row">
               <% 6.times do |i| %>
-                <%= render "calendar", form: form, month: Date.today.beginning_of_month + i.months, planner: @planner %>
+                <%= render "calendar", form: form, start_date: Date.today.beginning_of_month + i.months, planner: @planner %>
               <% end %>
             </div>
           <% end %>