CODEWITHSUNDEEP
ruby-on-railsruby
Tarscher
Tarscher

Reputation: 1933

inheritance in controllers

I use inheritance in my model. An event has different types:

Event < activity
Event < training
Event < game

I want to set session data to every event type like

game.user_id = session[:user_id]
training.user_id = session[:user_id]
activity.user_id = session[:user_id]

I want to avoid writing @game.user_id = session[:user_id] , ..., ... in every create method in the controller of activity, game and training

Someone knows how to approach this best.

Thanks

Upvotes: 0

Views: 352

Answers (3)

Josh Rickard
Josh Rickard

Reputation: 1603

Generally you'll want to use the built-in scoping that Rails provides. Just to flesh out what @Radar already posted:

class ApplicationController < ActionController::Base
  before_filter :find_current_user

  private
    def find_current_user
      @current_user = User.find( session[:user_id] )
    end
end

class EventsController < ApplicationController
  def create
    @event = @current_user.events.build( params[:event] )
    @event.save!
  end
end

This assumes that you have setup the associations in your model:

class User
  has_many :events
end

class Event
  belongs_to :user
end

This is also a rather handy mechanism if you need to restrict what a user can see or edit:

class EventsController < ApplicationController
  def index
    @events = @current_user.events # only fetch current users events
  end

  def update
    @event = @current_user.events.find( params[:id] ) # can't update other user's events
    @event.update_attributes!( params[:event] )
  end
end

Upvotes: 0

Ryan Bigg
Ryan Bigg

Reputation: 107728

Don't use game.user_id, instead you can do this:

game = current_user.games.build(params[:game])
if game.save
  # do something
else
  # do something else
end

Repeat for your other controllers too!

The associations guide may be helpful too.

Upvotes: 0

erik
erik

Reputation: 6436

Perhaps you are looking for a before_filter that resides in your ApplicationController? Then in each controller, you can set the before_filter to run on create actions.

ApplicationController
  def set_user_ids
    game.user_id = session[:user_id]
    training.user_id = session[:user_id]
    activity.user_id = session[:user_id]
  end
  ...
end

OneController < ApplicationController
  before_filter :set_user_ids, :only => [:create]
  ...
end

TwoController < ApplicationController
  before_filter :set_user_ids, :only => [:create]
  ...
end

Upvotes: 3

Related Questions