3

我有以下模型:團隊,成員,作業,角色has_many上的Rails驗證計數限制:通過

團隊模型has_many成員。每個成員在任務中都有許多角色。角色分配是Captain和Runner。我還使用會員模型安裝了設計和CanCan。

我需要做的是限制每個隊伍最多有1名隊長和5名選手。

我發現這example,它似乎工作後,一些定製,但更新('隊/ 1 /成員/ 4 /編輯')。它不適用於創建('teams/1/members/new')。但是我的其他驗證(驗證:role_ids,:presence => true )對更新和創建都有效。任何幫助,將不勝感激。

更新:我發現這個example似乎與我的問題類似,但我似乎無法讓它適用於我的應用程序。

似乎問題的根源在於驗證之前和驗證過程中如何執行計數(或大小)。

例如:

當更新記錄...... 它檢查,看看有多少運動員有一個團隊,並返回一個計數。 (即5)然後,當我選擇要添加到成員的角色時,它從數據庫中取出已知的計數(即5)並添加建議的改變(即1),然後運行驗證檢查。 (Team.find(self.team_id).members.runner.count> 5)這很好,因爲它返回6和6> 5的值,所以建議的更新失敗而不保存並給出錯誤。

當我嘗試對球隊創建一個新的成員... 它檢查,看看有多少運動員有一個團隊,並返回一個計數。 (即5)然後,當我選擇要添加到成員的角色時,它從數據庫中獲取已知的計數(即5),然後運行驗證檢查而不包括分解建議的更改。這是行不通的,因爲它返回5個已知runner和5 = 5的值,所以建議的更新通過,並且新成員和角色被保存到數據庫中,沒有錯誤。

會員型號:

class Member < ActiveRecord::Base 
    devise :database_authenticatable, :registerable, 
     :recoverable, :rememberable, :trackable, :validatable 

    attr_accessible :password, :password_confirmation, :remember_me 
    attr_accessible :age, :email, :first_name, :last_name, :sex, :shirt_size, :team_id, :assignments_attributes, :role_ids 
    belongs_to :team 
    has_many :assignments, :dependent => :destroy 
    has_many :roles, through: :assignments 
    accepts_nested_attributes_for :assignments 

    scope :runner, joins(:roles).where('roles.title = ?', "Runner") 
    scope :captain, joins(:roles).where('roles.title = ?', "Captain") 

    validate :validate_runner_count 
    validate :validate_captain_count 
    validates :role_ids, :presence => true 

    def validate_runner_count 
    if Team.find(self.team_id).members.runner.count > 5 
     errors.add(:role_id, 'Error - Max runner limit reached') 
    end 
    end 

    def validate_captain_count 
    if Team.find(self.team_id).members.captain.count > 1 
     errors.add(:role_id, 'Error - Max captain limit reached') 
    end 
    end 

    def has_role?(role_sym) 
    roles.any? { |r| r.title.underscore.to_sym == role_sym } 
    end 

end 

會員控制器:

class MembersController < ApplicationController 
    load_and_authorize_resource :team 
    load_and_authorize_resource :member, :through => :team 

    before_filter :get_team 
    before_filter :initialize_check_boxes, :only => [:create, :update] 

    def get_team 
    @team = Team.find(params[:team_id]) 
    end 

    def index 
    respond_to do |format| 
     format.html # index.html.erb 
     format.json { render json: @members } 
    end 
    end 

    def show 
    respond_to do |format| 
     format.html # show.html.erb 
     format.json { render json: @member } 
    end 
    end 

    def new 
    respond_to do |format| 
     format.html # new.html.erb 
     format.json { render json: @member } 
    end 
    end 

    def edit 

    end 

    def create 
    respond_to do |format| 
     if @member.save 
     format.html { redirect_to [@team, @member], notice: 'Member was successfully created.' } 
     format.json { render json: [@team, @member], status: :created, location: [@team, @member] } 
     else 
     format.html { render action: "new" } 
     format.json { render json: @member.errors, status: :unprocessable_entity } 
     end 
    end 
    end 

    def update 
    respond_to do |format| 
     if @member.update_attributes(params[:member]) 
     format.html { redirect_to [@team, @member], notice: 'Member was successfully updated.' } 
     format.json { head :no_content } 
     else 
     format.html { render action: "edit" } 
     format.json { render json: @member.errors, status: :unprocessable_entity } 
     end 
    end 
    end 

    def destroy 
    @member.destroy 

    respond_to do |format| 
     format.html { redirect_to team_members_url } 
     format.json { head :no_content } 
    end 
    end 

    # Allow empty checkboxes 
    # http://railscasts.com/episodes/17-habtm-checkboxes 
    def initialize_check_boxes 
    params[:member][:role_ids] ||= [] 
    end 

end 

_form部分

<%= form_for [@team, @member], :html => { :class => 'form-horizontal' } do |f| %> 

    #... 

    # testing the count... 
    <ul> 
    <li>Captain - <%= Team.find(@member.team_id).members.captain.size %></li> 
    <li>Runner - <%= Team.find(@member.team_id).members.runner.size %></li> 
    <li>Driver - <%= Team.find(@member.team_id).members.driver.size %></li> 
    </ul> 

    <div class="control-group"> 
     <div class="controls"> 
     <%= f.fields_for :roles do %> 
     <%= hidden_field_tag "member[role_ids][]", nil %> 
     <% Role.all.each do |role| %> 
      <%= check_box_tag "member[role_ids][]", role.id, @member.role_ids.include?(role.id), id: dom_id(role) %> 
      <%= label_tag dom_id(role), role.title %> 
     <% end %> 
     <% end %> 
    </div> 
    </div> 

    #... 
<% end %> 

回答

1

嘗試

class Member < ActiveRecord::Base 
    ... 

    def validate_runner_count 
    if self.team.members.runner.count > 5 
     errors.add(:role_id, 'Error - Max runner limit reached') 
    end 
    end 

    def validate_captain_count 
    if self.team.members.captain.count > 1 
     errors.add(:role_id, 'Error - Max captain limit reached') 
    end 
    end 
end 
+0

餘噸該解決方案在運行的同時,仍然只適用於更新,而不適用於創建。感謝您的嘗試。 **注意:**我對我發佈的原始模型做了一些小修改,以修復一個錯誤。 – Jeremy

+0

使用.size而不是.count – Azi