diff --git a/app/components/budgets/phases_component.html.erb b/app/components/budgets/phases_component.html.erb
index 6f2d0596a..fa0db2ffa 100644
--- a/app/components/budgets/phases_component.html.erb
+++ b/app/components/budgets/phases_component.html.erb
@@ -1,6 +1,6 @@
<% budget.published_phases.each do |phase| %>
- - ">
+
- ">
<%= t("budgets.phase.#{phase.kind}") %>
<%= l(phase.starts_at.to_date, format: :long) if phase.starts_at.present? %>
diff --git a/app/models/budget/phase.rb b/app/models/budget/phase.rb
index d8b59207f..022937ce1 100644
--- a/app/models/budget/phase.rb
+++ b/app/models/budget/phase.rb
@@ -62,6 +62,10 @@ class Budget
in_phase_or_later?("balloting")
end
+ def current?
+ budget.current_phase == self
+ end
+
private
def adjust_date_ranges
diff --git a/app/views/admin/budgets/_form.html.erb b/app/views/admin/budgets/_form.html.erb
index 4702a7ae2..990dfecc5 100644
--- a/app/views/admin/budgets/_form.html.erb
+++ b/app/views/admin/budgets/_form.html.erb
@@ -60,7 +60,7 @@
|
<%= t("budgets.phase.#{phase.kind}") %>
- <% if @budget.current_phase == phase %>
+ <% if phase.current? %>
<%= t("admin.budgets.edit.active") %>
<% end %>
|
diff --git a/spec/system/admin/budgets_spec.rb b/spec/system/admin/budgets_spec.rb
index 0fb597c31..f98142389 100644
--- a/spec/system/admin/budgets_spec.rb
+++ b/spec/system/admin/budgets_spec.rb
@@ -273,7 +273,7 @@ describe "Admin budgets", :admin do
expect(page).to have_content("#{phase.starts_at.to_date} - #{phase.ends_at.to_date}")
expect(page).to have_css(".budget-phase-enabled.enabled")
expect(page).to have_link("Edit phase", href: edit_phase_link)
- expect(page).to have_content("Active") if budget.current_phase == phase
+ expect(page).to have_content("Active") if phase.current?
end
end
end