de-escalate unnecessary logger warnings

Author: Nathan Keyes

app/models/cms/view_context.rb

@@ -39,7 +39,7 @@ def initialize(controller, attributes_to_assign)
 
  # We want content_for to be called on the controller's view, not this inner view
  def content_for(name, content=nil, &block)
- Rails.logger.warn "content_for(#{name}, #{content}, block) called."
+ Rails.logger.info "content_for(#{name}, #{content}, block) called."
  @controller.instance_variable_get("@template").content_for(name, content, &block)
  end
 

lib/cms/acts/content_page.rb

@@ -52,7 +52,7 @@ def page_title=(title)
  # Before filter that determines if the current user can access a specific section.
  def check_access_to_section
  user = current_user
- logger.warn "Checking that current_user '#{user.login}' has access to view section with path '#{self.class.in_section}'."
+ logger.info "Checking that current_user '#{user.login}' has access to view section with path '#{self.class.in_section}'."
  unless user.able_to_view?(self.class.in_section)
  store_location
  raise Cms::Errors::AccessDenied
@@ -75,7 +75,7 @@ module ClassMethods
  # ...
  #
  def requires_permission_for_section(path, options={})
- logger.warn "Setting path #{path}"
+ logger.info "Setting path #{path}"
  @section_path = path
  before_filter :check_access_to_section, options
  end

lib/cms/behaviors/attaching.rb

@@ -294,7 +294,7 @@ def multiple_attachments
  #
  # ActiveRecord Callback
  def save_associated_attachments
- logger.warn "save_associated_attachments #{attachments}"
+ logger.info "save_associated_attachments #{attachments}"
  attachments.each do |a|
  a.save if a.changed?
  end

test/unit/models/page_test.rb

@@ -566,10 +566,10 @@ def setup
  end
 
  test "Creating a new block to a page should update all existing connectors to the new page version." do
- Rails.logger.warn "Creating a new connector"
+ Rails.logger.info "Creating a new connector"
  @page.create_connector(create(:html_block), "main")
  expected_version = 3
- Rails.logger.warn "Done"
+ Rails.logger.info "Done"
  connectors = @page.connectors.for_page_version(expected_version)
  assert_equal expected_version, connectors[0].page_version, "There should be two connectors with the same version as the page"
  assert_equal expected_version, connectors[1].page_version