Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

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[:sex]

like image 434
Rabbott Avatar asked Jun 14 '10 16:06

Rabbott


1 Answers

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.
like image 150
Mike Trpcic Avatar answered Sep 16 '22 22:09

Mike Trpcic