Rails 3.1 e pagine statiche

Question

Sono solo nel bel mezzo di aggiornamento di un'applicazione di grandi dimensioni da Rails 3 a Rails 3.1 e ha colpito un problema con la mia implementazione del controller pagine:

  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

Questo ha funzionato in Rails 3.0. Qualcosa deve dei cambiato con il metodo template_exists. Ecco il controllore:

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

Codice Soluzione:

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

Ho anche notato che non era il rendering il punto di vista di errore (ad esempio: /public/404.html) così ho creato una cartella app / views / errori e mettere tutte le pagine di errore statiche in là e solo li rende ora. Esso funziona.

Grazie Andrew.

Answer 1

I parametri del metodo template_exists indicano che il secondo parametro, prefisso, dovrebbe essere un array. Normalmente i metodi Rails accettano sia convertendo qualcosa da una matrice, se non, quindi questo è un po 'insolito.

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

Questo metodo è anche alias come 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

Così facendo view_prefix = ["pages"] dovrebbe funzionare? (E modificando l'interpolazione stringa rimanente conseguenza)