Question

Currently I verify that there are no duplicated Members when trying to create a new Member and add it to a Team.

members_controller.rb

def create
  @team = current_team
  player = Player.find(params[:player_id])
  @member = @team.add_player(player.id)

  respond_to do |format|
     if @member.save
        format.html { redirect_to(@team, :notice => 'Member was successfully added.') }
        format.js { @current_member = @member }
        format.xml  { render :xml => @member, 
        :status => :created, :location => @member } 
     else
        format.html { redirect_to(@team, :notice => 'Member already exists.') }
        format.xml  { render :xml => @member.errors, 
        :status => :unprocessable_entity }
     end
   end
 end

team.rb

def add_player(player_id)
    current_member = members.build(:player_id => player_id)
    current_member
end

I want to add some logic to my add_player method in team.rb that checks various properties of the player that is being added. This action will require multiple failure messages, other than ‘Member already exists.’ How do I do this in the Model layer?

Answer 1

The controller logic to display all errors from base worked. However, I was not able to add errors from the add_player method as Ben suggested. I instead created separate custom validations as such:

Team.rb

validate validation_name

def validation_name
  if condition
    errors.add_to_base "Error Message"
  end
end