diff --git a/app/components/budgets/groups_and_headings_component.rb b/app/components/budgets/groups_and_headings_component.rb
index b16357e4b..115695034 100644
--- a/app/components/budgets/groups_and_headings_component.rb
+++ b/app/components/budgets/groups_and_headings_component.rb
@@ -8,6 +8,6 @@ class Budgets::GroupsAndHeadingsComponent < ApplicationComponent
private
def price(heading)
- tag.span(budget.formatted_heading_price(heading))
+ tag.span(budget.formatted_heading_price(heading)) if budget.show_money?
end
end
diff --git a/app/components/budgets/single_heading_component.html.erb b/app/components/budgets/single_heading_component.html.erb
index 05fb1898d..a8550e402 100644
--- a/app/components/budgets/single_heading_component.html.erb
+++ b/app/components/budgets/single_heading_component.html.erb
@@ -1,4 +1,4 @@
<%= title %>
-
<%= price %>
+ <%= price %>
diff --git a/app/components/budgets/single_heading_component.rb b/app/components/budgets/single_heading_component.rb
index d9711775a..c35068694 100644
--- a/app/components/budgets/single_heading_component.rb
+++ b/app/components/budgets/single_heading_component.rb
@@ -16,6 +16,6 @@ class Budgets::SingleHeadingComponent < ApplicationComponent
end
def price
- budget.formatted_heading_price(heading)
+ tag.p budget.formatted_heading_price(heading) if budget.show_money?
end
end
diff --git a/spec/system/budgets/budgets_spec.rb b/spec/system/budgets/budgets_spec.rb
index 3e49b59c9..b0a8ce7f9 100644
--- a/spec/system/budgets/budgets_spec.rb
+++ b/spec/system/budgets/budgets_spec.rb
@@ -119,6 +119,34 @@ describe "Budgets" do
end
end
+ scenario "Hide money on single heading budget" do
+ budget = create(:budget, :finished, :hide_money)
+ heading = create(:budget_heading, budget: budget)
+
+ visit budgets_path
+
+ within("#budget_info") do
+ expect(page).to have_content heading.name
+ expect(page).not_to have_content "€"
+ end
+ end
+
+ scenario "Hide money on multiple headings budget" do
+ budget = create(:budget, :finished, :hide_money)
+ heading1 = create(:budget_heading, budget: budget)
+ heading2 = create(:budget_heading, budget: budget)
+ heading3 = create(:budget_heading, budget: budget)
+
+ visit budgets_path
+
+ within("#budget_info") do
+ expect(page).to have_content heading1.name
+ expect(page).to have_content heading2.name
+ expect(page).to have_content heading3.name
+ expect(page).not_to have_content "€"
+ end
+ end
+
scenario "No budgets" do
Budget.destroy_all