2009-10-16 2 views
16

Я использую Single Table Inheritance для управления различными типами проектов.STI и form_for problem

Модели:

class Project < ActiveRecord::Base 
end 

class SiteDesign < Project 
end 

class TechDesign < Project 
end 

Редактировать действие от projects_controller:

def edit 
    @project = Project.find(params[:id]) 
end 

Просмотр edit.html.erb:

<% form_for(@project, :url => {:controller => "projects",:action => "update"}) do |f| %> 
    ... 
    <%= submit_tag 'Update' %> 
<% end %> 

Update действие projects_controller:

def update 
    @project = Project.find(params[:id]) 
    respond_to do |format| 
     if @project.update_attributes(params[:project]) 
     @project.type = params[:project][:type] 
     @project.save 
     flash[:notice] = 'Project was successfully updated.' 
     format.html { redirect_to(@project) } 
     format.xml { head :ok } 
     else 
     format.html { render :action => "edit" } 
     format.xml { render :xml => @project.errors, :status => :unprocessable_entity } 
     end 
    end 
    end 

Тогда я сделать некоторые изменения вступления TechDesign на представлении редактирования и получаю ошибку:

NoMethodError in ProjectsController#update 

You have a nil object when you didn't expect it! 
You might have expected an instance of ActiveRecord::Base. 
The error occurred while evaluating nil.[] 

В parametrs, очевидно, что вместо имени параметра проекта я tech_design Параметры:

{"commit"=>"Update", 
"_method"=>"put", 
"authenticity_token"=>"pd9Mf7VBw+dv9MGWphe6BYwGDRJHEJ1x0RrG9hzirs8=", 
"id"=>"15", 
"tech_design"=>{"name"=>"ech", 
"concept"=>"efds", 
"type"=>"TechDesign", 
"client_id"=>"41", 
"description"=>"tech"}} 

Как Исправить это?

ответ

27

Вот источник вашей проблемы. Это устанавливает @project как экземпляр объекта TechDesign.

def edit 
    @project = Project.find(params[:id]) 
end 

Вы можете гарантировать, что все работает так, как вы хотите, указав: проект для имени в форме для вызова.

<% form_for(:project, @project, :url => {:controller => "projects",:action => "update"}) do |f| %> 
    ... 
    <%= submit_tag 'Update' %> 
<% end %> 
+0

Спасибо за ваш ответ. После этих ошибок у меня возникла эта ошибка: Никаких действий не ответили 1. Действия: создать, уничтожить, отредактировать, индексировать, обновить, показать и обновить – ebsbk

+1

После добавления этого map.connect "projects /: id/edit", : controller => "projects",: action => "edit" to routes.rb ошибка disapared. Вопрос в том, почему это не сработало без него, если map.resources: проекты уже представлены в маршрутах? – ebsbk

+11

В Rails 3.1, напишите выражение form_for как <% = form_for @project,: as =>: project do | f | %> – Cygwin98

2

Случайное замечание: Если вы используете одного таблицы наследования (ИППП), и забудьте удалить инициализировать метод из ваших определений подкласса вы получите подобный «объект ноль, когда вы не ожидали» исключение.

Например:

class Parent < ActiveRecord::Base 
end 

class Child < Parent 
    def initialize 
    # you MUST call *super* here or get rid of the initialize block 
    end 
end 

В моем случае, я использовал мой IDE для создания дочерних классов и IDE создал метод инициализации. Взял меня навсегда, чтобы разыскать ...

6

Для Rails 3

<% form_for(@project, :as => :project, :url => {:controller => "projects",:action => "update"}) do |f| %> 
... 
    <%= submit_tag 'Update' %> 
<% end %> 
1

Для Rails 4, я подтвердил, единственное, что, казалось, работать для меня, чтобы явно указать URL и тому AS параметры:

<% form_for(@project, as: :project, url: {controller: :projects, action: :update}) do |f| %> 
    ... 
<% end %> 

Кажется уродливым для меня!