0

これらは私の 2 つのモデルで、何も複雑ではありません。

class Invoice
  belongs_to :user
  has_many :invoice_line_items
  accepts_nested_attributes_for :invoice_line_items, allow_destroy: true
end

class InvoiceLineItem
  belongs_to :invoice
end

これは、ネストされた属性用に変更された simple_form です。

<%= simple_form_for(@invoice) do |f| %>
  <%= f.error_notification %>

  <div class="form-inputs">
    <%= f.input :adress_sender %>
    <%= f.input :adress_recipient %>
    <%= f.input :status %>
    <%= simple_fields_for :invoice_line_items do |invoice_line_items_form| %>
      <%= invoice_line_items_form.input :description %>
      <%= invoice_line_items_form.input :price %>
      <%= invoice_line_items_form.input :amount %>
    <% end %>
  </div>  
  <div class="form-actions">
    <%= f.button :submit %>
  </div>
<% end %>

これは、Rails の公式ドキュメントによるとコントローラーです。

class InvoicesController < ApplicationController
  before_action :set_invoice, only: [:show, :edit, :update, :destroy]

  # GET /invoices
  # GET /invoices.json
  def index
    @invoices = current_user.invoices
  end

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

  # GET /invoices/new
  def new
    @invoice = Invoice.new
    @invoice.invoice_line_items.build
  end

  # GET /invoices/1/edit
  def edit
  end

  # POST /invoices
  # POST /invoices.json
  def create
    @invoice = current_user.invoices.new(invoice_params)
    @invoice.invoice_line_items.build
    respond_to do |format|
      if @invoice.save
        format.html { redirect_to @invoice, notice: 'Invoice was successfully created.' }
        format.json { render action: 'show', status: :created, location: @invoice }
      else
        format.html { render action: 'new' }
        format.json { render json: @invoice.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /invoices/1
  # PATCH/PUT /invoices/1.json
  def update
    respond_to do |format|
      if @invoice.update(invoice_params)
        format.html { redirect_to @invoice, notice: 'Invoice was successfully updated.' }
        format.json { head :no_content }
      else
        format.html { render action: 'edit' }
        format.json { render json: @invoice.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /invoices/1
  # DELETE /invoices/1.json
  def destroy
    @invoice.destroy
    respond_to do |format|
      format.html { redirect_to invoices_url }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions and check if the user has access to it.
    def set_invoice
        @invoice = current_user.invoices.find(params[:id])
        @invoice.invoice_line_items.build
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def invoice_params
      params.require(:invoice).permit(:adress_sender, :adress_recipient, :status, :user_id, :customer_id, invoice_line_items: [:description, :price, :amount])
    end
end

請求書の項目が作成されています。ただし、作成日と更新日とIDだけです。説明、価格、金額などのすべてのフィールドが空です。また、allow_destroy が機能していないようです。残念ながら、サーバーコンソールにエラーはありません:/

提案されているように、作成のコンソール ログを編集します。

Started POST "/invoices" for 127.0.0.1 at 2013-10-21 15:36:42 +0200
Processing by InvoicesController#create as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"mFdXPbYF+Lsb1mugWkdykBkJ1iSrzZoREL5Alw6phhQ=", "invoice"=>{"adress_sender"=>"awfawf", "adress_recipient"=>"awfgaw", "status"=>"awgag"}, "invoice_line_items"=>{"description"=>"awga", "price"=>"awgwa", "amount"=>"awg"}, "commit"=>"Create Invoice"}
  User Load (2.1ms)  SELECT "users".* FROM "users" WHERE "users"."id" = 1 ORDER BY "users"."id" ASC LIMIT 1
   (0.3ms)  BEGIN
  SQL (7.0ms)  INSERT INTO "invoices" ("adress_recipient", "adress_sender", "created_at", "status", "updated_at", "user_id") VALUES ($1, $2, $3, $4, $5, $6) RETURNING "id"  [["adress_recipient", "awfgaw"], ["adress_sender", "awfawf"], ["created_at", Mon, 21 Oct 2013 13:36:42 UTC +00:00], ["status", "awgag"], ["updated_at", Mon, 21 Oct 2013 13:36:42 UTC +00:00], ["user_id", 1]]
  SQL (0.9ms)  INSERT INTO "invoice_line_items" ("created_at", "invoice_id", "updated_at") VALUES ($1, $2, $3) RETURNING "id"  [["created_at", Mon, 21 Oct 2013 13:36:42 UTC +00:00], ["invoice_id", 15], ["updated_at", Mon, 21 Oct 2013 13:36:42 UTC +00:00]]
   (4.2ms)  COMMIT
Redirected to http://localhost:3000/invoices/15
Completed 302 Found in 60ms (ActiveRecord: 14.5ms)

アイデアや提案はありますか？

宜しくお願いします。

4

3 に答える 3

1

まず、あなたの質問とは少し関係ありません.set_invoice呼び出しの場合、破棄する場合でも常に新しいinvoice_line_itemを作成しています。特定の請求書にすべてのinvoice_line_itemsを含めようとしている場合は、@invoice.includes(:invoice_line_items).where(id: params[:id])これらのアイテムを熱心にロードするために、おそらく次のようなことを行う必要があります。

あなたの質問については... incoive_paramsについては、そうあるinvoice_line_itemsべきですinvoice_line_items_attributes（更新に:idを含めることもお勧めします）。

それを試して、それが機能するかどうかを確認してください。@invoice.invoice_line_items.buildまた、作成アクションでは、上記のコマンドにネストされたフォームのパラメーターが既に含まれているため、追加する必要はないと思います。これを行う@invoice.invoice_line_items.buildと、@invoice の別のinvoice_line_item を作成しているように見えますが、パラメーターを音声化していません。

（少し急いで返信を書きました。意味があることを願っています。）

def create
  @invoice = current_user.invoices.new(invoice_params)
  @invoice.invoice_line_items.build
  respond_to do |format|
    if @invoice.save
      format.html { redirect_to @invoice, notice: 'Invoice was successfully created.' }
      format.json { render action: 'show', status: :created, location: @invoice }
    else
      format.html { render action: 'new' }
      format.json { render json: @invoice.errors, status: :unprocessable_entity }
    end
  end
end

# Never trust parameters from the scary internet, only allow the white list through.
def invoice_params
  params.require(:invoice).permit(:adress_sender, :adress_recipient, :status, :user_id, :customer_id, invoice_line_items_attributes: [:id, :description, :price, :amount])
end

```

補足: 住所の綴りが間違っています。これがレールの複数形化に問題を引き起こすかどうかはわかりません。

于 2013-10-21T14:01:44.040 に答える
0

以下をお試しください。お役に立てば幸いです。

def new
    @invoice = Invoice.new
    @invoice.invoice_line_items.build
  end

  def create
    @invoice = current_user.invoices.create(invoice_params)

    respond_to do |format|
      if @invoice.save
        format.html { redirect_to @invoice, notice: 'Invoice was successfully created.' }
        format.json { render action: 'show', status: :created, location: @invoice }
      else
        format.html { render action: 'new' }
        format.json { render json: @invoice.errors, status: :unprocessable_entity }
      end
    end
  end
于 2013-10-21T13:54:52.327 に答える