diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index 2b37ad312..7bee2117e 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -1173,18 +1173,13 @@ module ApplicationHelper super sources, options end - def content_for(name, content = nil, &block) - @has_content ||= {} - @has_content[name] = true - super(name, content, &block) - end - + # TODO: remove this in 2.5.0 def has_content?(name) - (@has_content && @has_content[name]) || false + content_for?(name) end def sidebar_content? - has_content?(:sidebar) || view_layouts_base_sidebar_hook_response.present? + content_for?(:sidebar) || view_layouts_base_sidebar_hook_response.present? end def view_layouts_base_sidebar_hook_response diff --git a/test/functional/projects_controller_test.rb b/test/functional/projects_controller_test.rb index 79721d589..6f294d6d0 100644 --- a/test/functional/projects_controller_test.rb +++ b/test/functional/projects_controller_test.rb @@ -320,6 +320,16 @@ class ProjectsControllerTest < ActionController::TestCase assert_select 'li', :text => /Development status/ end + def test_show_should_not_display_empty_sidebar + p = Project.find(1) + p.enabled_module_names = [] + p.save! + + get :show, :id => 'ecookbook' + assert_response :success + assert_select '#main.nosidebar' + end + def test_show_should_not_display_hidden_custom_fields ProjectCustomField.find_by_name('Development status').update_attribute :visible, false get :show, :id => 'ecookbook'