开发者

Rails object inheritance with belongs_to

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:

  1. You don't need to change the object name
  2. 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?

0

上一篇:

下一篇:

精彩评论

暂无评论...
验证码 换一张
取 消

最新问答

问答排行榜