I have a simple has_many/belongs_to relationship between Report and Chart. The issue I'm having is that my Chart model is a parent that has children.
So in my Report model I have
class Report < ActiveRecord::Base
has_many :charts
end
And my Chart model is a parent, where Pie, Line, Bar all inherit from Chart. I'm not sure where the belongs_to :report belongs within the chart model, or children of chart model. I get errors when I attempt to access chart.report because the object is of type "Class"
undefined local variable or method `report' for #< Class:0x104974b90>
The Chart model uses STI so its pulling say.. 'Gender' from the chart_type column in the charts table.. what am I missing?
EDIT
Chart
/ \
Pie Line
/ \
/ \
Gender Sex
I am (using STI) instantiating an object of type Gender, or Sex. Hopefully this helps a bit more.
I have a feeling that its caused by
@chart.update_attributes(params[:chart])
because when submitted its actually params[:chart] its params[:gender] or params[:se开发者_如何学运维x]
Your problem is that you have one controller receiving all these different types of models. When you use the Rails form helpers, they're taking the object type from the instance and using that to populate the params. This means that you have params[:gender]
instead of params[:chart]
You can fix this by overriding the method that Rails uses to generate the form. Put the following code in your base chart class:
def self.model_name
name = "chart"
name.instance_eval do
def plural; pluralize; end
def singular; singularize; end
end
return name
end
Now, any class that is a child of Chart will be passed into your receiving action as params[:chart]
Keep in mind that changing the object name as I've outlined above could break functionality of some plugins/gems that rely on it. You should look into having multiple controllers. Instead of having a ChartsController
to receive all of the data, have a GenderChartsController
and a LineChartsController
. This provides two benefits:
- You don't need to change the object name
- If you need custom code for a specific type of chart in the future, you don't have to add conditional statements to make it work, as the different controllers have segregated it for you.
You should be able to use just
class Report < ActiveRecord::Base
has_many :charts
end
class Chart < ActiveRecord::Base
belongs_to :report
end
and call
@chart.report
@pie.report
@line.report
@bar.report
If all the chart types belong to a report, then place
belongs_to :report
in the parent model ( that is Chart Model in your case I am guessing).
Your chart_type
column needs to be renamed to just type
, unless you explicitly overwrote Base.inheritance_column
.
Did you explicitly rename the single table inheritance column to chart_type
?
精彩评论