ruby-on-railsrubycontent-management-systemrefinery

Refinery CMS inquiries - NoMethodError for content_for when rendering :body content


I'm using the 'refinerycms-inquiries' gem in my app to render an inquiry on my contact page.

The problem that I'm getting is that although I can use the <%= raw @page.content_for(:body) %> code on my other pages to render the content, I have no idea why I'm getting this in the inquiries#new view. Maybe someone can help me solve this more quicker, because I've looked for a couple of hours on the internet for an answer and looking through the source code.

This is the error:

 NoMethodError in Refinery/inquiries/inquiries#new 
undefined method `content_for' for nil:NilClass

Extracted source (around line #3):

1: <% content_for :body do %>
2:     <div>
3:       <%= raw @page.content_for(:body) %>
4:     </div>
5: <% end %>
6: 

I have to mention I'm a bit new to refinery and RoR.

This is my refinery/inquiries/inquiries/new.html.erb file:

<% content_for :body do %>
    <div class="contact">
      <%= raw @page.content_for(:body) %>
    </div>
<% end %>

<% content_for :body_content_left do %>
    <div class='inquiries'>
      <%= form_for [refinery, :inquiries, @inquiry] do |f| %>
          <%= render :partial => "/refinery/admin/error_messages",
                     :locals => {
                             :object => @inquiry,
                             :include_object_name => true
                     } %>
          <div class="field">
            <%= f.required_label :name, :class => 'placeholder-fallback' %>
            <%= f.text_field :name, :class => 'text', :required => 'required', :placeholder => t('name', :scope => 'activerecord.attributes.refinery/inquiries/inquiry') %>
          </div>
          <div class="field">
            <%= f.required_label :email, :class => 'placeholder-fallback' %>
            <%= f.email_field :email, :class => 'text email', :required => 'required', :placeholder => t('email', :scope => 'activerecord.attributes.refinery/inquiries/inquiry') %>
          </div>
          <div class="field">
            <%= f.label :phone, :class => 'placeholder-fallback' %>
            <%= f.text_field :phone, :class => 'text phone', :placeholder => t('phone', :scope => 'activerecord.attributes.refinery/inquiries/inquiry') %>
          </div>
          <div class='field message_field'>
            <%= f.required_label :message, :class => 'placeholder-fallback' %>
            <%= f.text_area :message, :rows => 8, :required => 'required', :placeholder => t('message', :scope => 'activerecord.attributes.refinery/inquiries/inquiry') %>
          </div>
          <div class="actions">
            <%= f.submit t('.send') %>
          </div>
      <% end %>
    </div>
<% end %>
<%= render :partial => "/refinery/content_page" %>

This is my inquiries_controller.rb:

module Refinery
  module Inquiries
    class InquiriesController < ::ApplicationController

      before_filter :find_page, :only => [:create, :new]

      def thank_you
        @page = ::Refinery::Page.find_by_link_url("/contact/thank_you")
      end

      def new
        @inquiry = ::Refinery::Inquiries::Inquiry.new
      end

      def create
        @inquiry = ::Refinery::Inquiries::Inquiry.new(params[:inquiry])

        if @inquiry.save
          if @inquiry.ham?
            begin
              ::Refinery::Inquiries::InquiryMailer.notification(@inquiry, request).deliver
            rescue
              logger.warn "There was an error delivering an inquiry notification.\n#{$!}\n"
            end

            begin
              ::Refinery::Inquiries::InquiryMailer.confirmation(@inquiry, request).deliver
            rescue
              logger.warn "There was an error delivering an inquiry confirmation:\n#{$!}\n"
            end if ::Refinery::Inquiries::Setting.send_confirmation?
          end

          redirect_to refinery.thank_you_inquiries_inquiries_path
        else
          render :action => 'new'
        end
      end

      protected

      def find_page
        @page = ::Refinery::Page.find_by_link_url("/contact")
      end

    end
  end
end

And here is the config/routes.rb part that is generated by the 'refinerycms-inquiries' gem:

Refinery::Core::Engine.routes.draw do
    namespace :inquiries, :path => '' do
      get '/contact', :to => 'inquiries#new', :as => 'new_inquiry'

      resources :contact,
                :only => :create,
                :as => :inquiries,
                :controller => 'inquiries' do
        get :thank_you, :on => :collection
      end

      namespace :admin, :path => 'refinery' do
        resources :inquiries, :only => [:index, :show, :destroy] do
          get :spam, :on => :collection
          get :toggle_spam, :on => :member
        end

        scope :path => 'inquiries' do
          resources :settings, :only => [:edit, :update]
        end
      end
    end
  end

Solution

  • Looks like I've found the problem.

    It turns out that ::Refinery::Page.find_by_url method from the inquiries_controller returned nil and I had to rewrite the find_page and thank_you methods like this:

      def thank_you
         @page = ::Refinery::Page.find_by_path("/contact/thank_you")
      end
    
      def find_page
         @page = ::Refinery::Page.find_by_path(/contact")
      end
    

    in order for the :body and :side_body content to be rendered in my contact view. I've also noticed that by this modification and don't need to explicitly specify in my refinery/inquiries/inquiries/new.html.erb file to render the content_for(:body).

    So the new view will look like the default one from the gem:

    <% content_for :body_content_left do %>
        <div class='inquiries'>
          <%= form_for [refinery, :inquiries, @inquiry] do |f| %>
              <%= render :partial => "/refinery/admin/error_messages",
                         :locals => {
                                 :object => @inquiry,
                                 :include_object_name => true
                         } %>
              <div class="field">
                <%= f.required_label :name, :class => 'placeholder-fallback' %>
                <%= f.text_field :name, :class => 'text', :required => 'required', :placeholder => t('name', :scope => 'activerecord.attributes.refinery/inquiries/inquiry') %>
              </div>
              <div class="field">
                <%= f.required_label :email, :class => 'placeholder-fallback' %>
                <%= f.email_field :email, :class => 'text email', :required => 'required', :placeholder => t('email', :scope => 'activerecord.attributes.refinery/inquiries/inquiry') %>
              </div>
              <div class="field">
                <%= f.label :phone, :class => 'placeholder-fallback' %>
                <%= f.text_field :phone, :class => 'text phone', :placeholder => t('phone', :scope => 'activerecord.attributes.refinery/inquiries/inquiry') %>
              </div>
              <div class='field message_field'>
                <%= f.required_label :message, :class => 'placeholder-fallback' %>
                <%= f.text_area :message, :rows => 8, :required => 'required', :placeholder => t('message', :scope => 'activerecord.attributes.refinery/inquiries/inquiry') %>
              </div>
              <div class="actions">
                <%= f.submit t('.send') %>
              </div>
          <% end %>
        </div>
    <% end %>
    <%= render :partial => "/refinery/content_page" %>
    

    Hope this helps if anyone encounters the same problem.