Вопрос

Я просто нахожусь в середине обновления большого применения с Rails 3 до Rails 3.1 и вычеркнул проблему с моей реализацией контроллера страниц:

  when templates doesnt exist
    should render the 404 page (FAILED - 1)

Failures:

  1) PagesController automatic paths when templates doesnt exist should render the 404 page
     Failure/Error: get 'base_page_processor', :base_page => 'something_that_doesnt_exist'
     NoMethodError:
       undefined method `map' for "pages":String
     # ./app/controllers/pages_controller.rb:5:in `base_page_processor'
     # ./spec/controllers/pages_controller_spec.rb:37:in `block (3 levels) in <top (required)>'

Finished in 0.10557 seconds
4 examples, 1 failure

Failed examples:

rspec ./spec/controllers/pages_controller_spec.rb:36 # PagesController automatic paths when templates doesnt exist should render the 404 page

Это работало в Rails 3.0. Что -то должно измениться с помощью метода Template_exists. Вот контроллер:

class PagesController < ApplicationController
  def base_page_processor
    view_prefix = "pages"

    if params[:base_page].present? && template_exists?(params[:base_page], view_prefix)
      render "#{view_prefix}/#{params[:base_page]}"
    else
      #TODO : Notify missing url via email error or error notification service
      render '/public/404.html', :status => 404
    end
  end
end

Код решения:

class PagesController < ApplicationController
  def base_page_processor
    view_prefix = ["pages"]

    if params[:base_page].present? && template_exists?(params[:base_page], view_prefix)
      render "#{view_prefix[0]}/#{params[:base_page]}"
    else
      #TODO : Notify missing url via email error or error notification service
      render '/errors/404.html', :status => 404
    end
  end
end

Я также заметил, что это не производило представления ошибок (т.е.: /public/404.html), поэтому я создал приложение/виды/ошибки каталога и поместил все статические страницы ошибки и просто показываю их сейчас. Оно работает.

Спасибо, Эндрю.

Это было полезно?

Решение

А template_exists Параметры метода показывают, что второй параметр, префикс, должен быть массивом. Обычно методы рельсов принимают оба путем преобразования чего -либо в массив, если нет, так что это немного необычно.

exists?(name, prefixes = [], partial = false, keys = [])

Этот метод также подселен как Template_exists?

# File actionpack/lib/action_view/lookup_context.rb, line 93
def exists?(name, prefixes = [], partial = false, keys = [])
  @view_paths.exists?(*args_for_lookup(name, prefixes, partial, keys))
end

Так что делать view_prefix = ["pages"] должно сработать? (и соответственно изменение оставшейся строковой интерполяции)

Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top