2017-01-25 9 views
1

私はたくさんのグーグルで行いましたが、許可された属性にidを追加することに戻ってきましたが、それは私のケースでは機能しません: 複数の空の行ネストされたフォームを編集するときに関連テーブルの中の) フォームを編集するたびに、関連する空のレコードが2倍になります。入れ子になったフォームを編集するときに重複を作成するレール

コントローラ:

class EnquiriesController < ApplicationController 
    before_action :authenticate_user! 
    load_and_authorize_resource 
    # before_action :set_enquiry, only: [:show, :edit, :update, :destroy] 
    #11-1-2017 Devise geinstalleerd, nog verder afmaken!(oa onderstaande uncommenten) 


    # GET /enquiries 
    # GET /enquiries.json 
    def index 
    # @enquiries = Enquiry.all 
    @enquirie = current_user.enquiries 
    end 

    # GET /enquiries/1 
    # GET /enquiries/1.json 
    def show 
    end 

    # GET /enquiries/new 
    def new 
    @enquiry = Enquiry.new 
    @enquiry.enquirymeasures.build 
     #@enquiry.measurements.build 
    @enquiry.tools.build 
    @enquiry.build_applicant 
    @enquiry.signatures.build 
    @enquiry.gasmeters.build 
     #@enquiry.enquirymeasures.build.build_measurement 
    end 

    # GET /enquiries/1/edit 
    def edit 
    #nog op kunnen slaan!!!! 1-01-17 Marco 
    @enquiry = Enquiry.find(params[:id]) 
    @enquiry.enquirymeasures.build 

    @enquiry.tools.build 
    @enquiry.build_applicant 
    @enquiry.signatures.build 
    @enquiry.gasmeters.build 

    end 

    # POST /enquiries 
    # POST /enquiries.json 
    def create 
    @enquiry.user_id = current_user.id 
    @enquiry = Enquiry.new(enquiry_params) 
    #@enquiry.enquirymeasures.build 

    respond_to do |format| 
     if @enquiry.save 
     format.html { redirect_to @enquiry, notice: 'Enquiry was successfully created.' } 
     format.json { render :show, status: :created, location: @enquiry } 
     else 
     format.html { render :new } 
     format.json { render json: @enquiry.errors, status: :unprocessable_entity } 
     end 
    end 
    end 

    #approved_enquiry_notification 

    # PATCH/PUT /enquiries/1 
    # PATCH/PUT /enquiries/1.json 
    def update 
    @enquiry = Enquiry.find(params[:id]) 
    respond_to do |format| 
     if @enquiry.update(enquiry_params) 
     format.html { redirect_to @enquiry, notice: 'Enquiry was successfully updated.' } 
     format.json { render :show, status: :ok, location: @enquiry } 
     else 
     format.html { render :edit } 
     format.json { render json: @enquiry.errors, status: :unprocessable_entity } 
     end 
    end 
    end 

    # DELETE /enquiries/1 
    # DELETE /enquiries/1.json 
    def destroy 
    @enquiry.destroy 
    respond_to do |format| 
     format.html { redirect_to enquiries_url, notice: 'Enquiry was successfully destroyed.' } 
     format.json { head :no_content } 
    end 
    end 

    private 
    # Use callbacks to share common setup or constraints between actions. 
    def set_enquiry 
     @enquiry = Enquiry.find(params[:id]) 
    end 

    # Never trust parameters from the scary internet, only allow the white list through. 
    #25-1 MG alle attributes hadden ook :id 
    def enquiry_params 
     params.require(:enquiry).permit(:id, :reference, :location, :description, :date, :amount, :approved, enquirymeasures_attributes: [:id, :responsible, :done, :needed, :measurement_id, :user], 
             tools_attributes: [:id, :handtool, :other, :motorvehicle, :compressor, :ramp, :scaffold, :crane, :ladder, :generator, :tankladder], 
             applicant_attributes: [:id, :name, :email, :contractor_id], 
             signatures_attributes: [:id, :date, :signature, :representative_id], 
             gasmeters_attributes: [:id, :date, :tester, :signature, :oxigen, :o_needed, :o_continu, :explosives, :e_needed, :e_continu, :mat1, :mat1_needed, :mat1_continu, :mat2, :mat2_needed, :mat2_continu, :mat3, :mat3_needed, :mat3_continu], 
             controls_attributes: [:id, :enquirycheck, :workspacecheck, :enquiry_id]).merge(user_id: current_user.id) 


    end 



end 

モデル: メイン:

関連モデルの
class Enquiry < ActiveRecord::Base 
    has_many :enquirymeasures, dependent: :destroy 
    accepts_nested_attributes_for :enquirymeasures, :reject_if => lambda { |a| a[:responsible].blank? }, :allow_destroy => true 


    has_many :tools, dependent: :destroy 
    accepts_nested_attributes_for :tools 

    has_many :controls, dependent: :destroy 
    accepts_nested_attributes_for :controls 

    has_one :applicant, dependent: :destroy 
    accepts_nested_attributes_for :applicant 

    has_one :contractor, through: :applicant 

    has_many :signatures, dependent: :destroy 
    accepts_nested_attributes_for :signatures 
    has_many :representatives , through: :signatures, :source => :representatives 

    has_many :gasmeters, dependent: :destroy 
    accepts_nested_attributes_for :gasmeters 

    belongs_to :user 

    #voor de goedkeuring mail 
    after_create :send_confirmation_mail 
    def send_confirmation_mail 
    ModelMailer.new_enquiry_mail(self).deliver 
    end 

    #after_update :send_approved_mail 
    #def send_approved_mail 
    # if params[:approved] == '1' 
    # if :approved == '1' 
    # ModelMailer.enquiry_approved_mailer(self).deliver 
    # end 
    #end 

end 

1:

class Tool < ActiveRecord::Base 
    belongs_to :enquiry 
end 

私はそれので、全体_formが含まれていませんでした非常に長いです(300行)が、あなたに考えを与える:

<div class="form-group "> 

     <div class="panel-group" id="accordion" role="tablist" aria-multiselectable="true"> 

      <%= f.fields_for :applicant do |a| %> 

       <div class="panel-group" id="accordion" role="tablist" aria-multiselectable="true"> 
       <div class="panel panel-default"> 
        <div class="panel-heading" role="tab" id="headingOne"> 
        <h4 class="panel-title"> 
         <a role="button" data-toggle="collapse" data-parent="#accordion" href="#collapseOne" aria-expanded="true" aria-controls="collapseOne"> 
         Aanvraag 
         </a> 
        </h4> 
        </div> 
        <div id="collapseOne" class="panel-collapse collapse in" role="tabpanel" aria-labelledby="headingOne"> 
        <div class="panel-body"> 
         <%= f.label :Referentie, :class => "col-sm-2 control-label" %> 
         <%= f.text_field :reference, :class => "form-control" %> <br> 

         <%= f.label :Locatie, :class => "col-sm-2 control-label" %> 
         <%= f.select :location, [['Chemiepark', 'chemiepark'], ['Winschoten', 'winschoten'], ['Farmsum', 'farmsum']], :class => "form-control" %> 
         <br> 
         <br> 

         <%= f.label :Omschrijving, :class => "col-sm-2 control-label" %> 
         <%= f.text_area :description, :class => "form-control" %> <br> 

         <%= f.label :Aantal_uitvoerenden, :class => "col-sm-2 control-label" %> 
         <%= f.number_field :amount, :class => "form-control" %> <br> 

         <%= f.label :Datum %><br> 
         <%= f.datetime_select :date %> <br> 

         <%= a.label :Aanvrager %><br> 
         <%= a.text_field :name %> <br> 

         <%= a.label :email %><br> 
         <%= a.text_field :email %> <br> 

         <%= a.label :Aannemer %><br> 
         <%= a.collection_select(:contractor_id, Contractor.all, :id, :name, prompt: true) %> 
        </div> 
        </div> 
       </div> 
       </div> 

      <% end %> 

答えて

0

ネストされた各属性にreject_if句を追加する必要があります。また、フォームでネストされた属性を黙って拒否しないようにする場合は、必要なフィールドの個々のモデルに検証を追加できます。必要に応じて指定したもののエラーを取り消します。サイドノートでは

has_many :tools, dependent: :destroy 
accepts_nested_attributes_for :tools, reject_if: :all_blank 

has_many :controls, dependent: :destroy 
accepts_nested_attributes_for :controls, reject_if: :all_blank 

has_one :applicant, dependent: :destroy 
accepts_nested_attributes_for :applicant, reject_if: :all_blank 

has_one :contractor, through: :applicant 

has_many :signatures, dependent: :destroy 
accepts_nested_attributes_for :signatures, reject_if: :all_blank 

has_many :representatives , through: :signatures, :source => :representatives 

has_many :gasmeters, dependent: :destroy 
accepts_nested_attributes_for :gasmeters, reject_if: :all_blank 

、私は、ネストされたフォームで作業する場合cocoon gemは非常に有用であることがわかってきました。

+0

ありがとう、私はあなたの解決を試みた、それはほぼ正しい結果を与えたので、私は少しそれを微調整し、今は動作するようです。私はall_blankを次のように変更しました:reject_if => lambda {| a | a [:handtool] .blank? }、:allow_destroy =>テーブルごとに必要なフィールドを使用するとtrueになります。ので、ガスメータのためにそれは[:テスター]などありがとう! – Marco

0

あなたはこのような何かをする必要があります

def update 
    @enquiry = Enquiry.find(params[:id]) 
    if @enquiry.update_attributes(enquiry_params) 
    # Handle a successful update. 
    else 
    render 'edit' 
end 

とenquiry_paramsからIDを削除します。

+0

すべてを削除します:idはenquiry_paramsからですか?メインテーブルのidとif @enquiry.update_attributes(enquiry_params)を使って、テストしました。ネストされたテーブルごとに1つのレコードが追加されます – Marco

+0

既存のレコードを更新するだけなので、ネストされたパラメータでIDを受け入れる必要はありません。だから、私はすべてのidを入れ子のパラメタから削除すると助けになると思います。 – trueinViso

関連する問題