0

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) がレンダリングされていないことに気付いたので、ディレクトリ app/views/errors を作成し、すべてのエラー静的ページをそこに配置して、今すぐレンダリングします。できます。

ありがとうアンドリュー。

4

1 に答える 1

1

メソッドのtemplate_existsパラメーターは、2 番目のパラメーターであるプレフィックスが配列であることを示しています。通常、Rails のメソッドは、そうでない場合は配列に変換することで両方を受け入れるため、これは少し珍しいことです。

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"]のはうまくいくはずですか？(そしてそれに応じて残りの文字列補間を変更します)

于 2011-09-07T09:10:37.467 に答える