2012-02-10 80 views
0

嗨,我知道這很愚蠢,但我真的得到這個時,我編輯任何條目。更新操作觸發插入查詢?

我有一個費用模型和一個expense_line_item和一個paid_line_item模型,它創建正確,當我創建一個新的條目,但在編輯以前的條目它增加了一個新的條目,而不是更新,我。即在更新操作時,它觸發插入查詢。這裏是我的代碼:

我的控制器:

def new 
     @menu = 'Expenses' 
     @page_name = 'Record New Expenses' 
    @expense = Expense.new 
     @expense.expense_line_items.build 
     @expense.paid_line_items.build 
     @expense.voucher_number = "EXP"+Time.now.to_i.to_s 
     @from_accounts = TransactionType.fetch_from_accounts(current_company.id, 'payments') 
     @to_accounts = TransactionType.fetch_to_accounts(current_company.id, 'payments') 

    respond_to do |format| 
     format.html # new.html.erb 
     format.xml { render :xml => @expense } 
    end 
    end 

    # GET /expenses/1/edit 
    def edit 
    @menu = 'Expenses' 
    @page_name = 'Edit Expenses Entry' 
    @expense = Expense.find(params[:id]) 
     @from_accounts = TransactionType.fetch_from_accounts(current_company.id, 'payments') 
     @to_accounts = TransactionType.fetch_to_accounts(current_company.id, 'payments') 
    end 

    # POST /expenses 
    # POST /expenses.xml 
    def create 
    @expense = Expense.new(params[:expense]) 
     @expense.created_by = current_user.id 
     @expense.company_id = current_company.id 

    respond_to do |format| 
     if @expense.save 
     format.html { redirect_to(@expense, :notice => 'Expense was successfully created.') } 
     format.xml { render :xml => @expense, :status => :created, :location => @expense } 
     else 
     @menu = 'Expenses' 
     @page_name = 'Record New Expenses' 
       @from_accounts = TransactionType.fetch_from_accounts(current_company.id, 'payments') 
       @to_accounts = TransactionType.fetch_to_accounts(current_company.id, 'payments')     
     format.html { render :action => "new" } 
     format.xml { render :xml => @expense.errors, :status => :unprocessable_entity } 
     end 
    end 
    end 

    # PUT /expenses/1 
    # PUT /expenses/1.xml 
    def update 
    @expense = Expense.find(params[:id]) 

    respond_to do |format| 
     if @expense.update_attributes(params[:expense]) 
     format.html { redirect_to(@expense, :notice => 'Expense was successfully updated.') } 
     format.xml { head :ok } 
     else 
     @menu = 'Expenses' 
     @page_name = 'Edit Expenses Entry' 
     format.html { render :action => "edit" } 
     format.xml { render :xml => @expense.errors, :status => :unprocessable_entity } 
     end 
    end 
    end 

我的模型:

expense model: 


class Expense < ActiveRecord::Base 
     has_many :expense_line_items 
     has_many :paid_line_items 

     accepts_nested_attributes_for :expense_line_items, :reject_if => lambda {|a| a[:account_id].blank? } , :allow_destroy => true 
     accepts_nested_attributes_for :paid_line_items, :reject_if => lambda {|a| a[:account_id].blank? }, :allow_destroy => true 

     #validations 
     validates_presence_of :expense_date, :voucher_number 
     validates_presence_of :expense_line_items 
     validates_associated :expense_line_items 
     validates_presence_of :paid_line_items 
     validates_associated :paid_line_items 


    end 

expense_line_item:

class ExpenseLineItem < ActiveRecord::Base 
    belongs_to :expense 
end 

paid_line_item:

class PaidLineItem < ActiveRecord::Base 
    belongs_to :expense 
end 

我的形式:

<%= form_for(@expense) do |f| %> 

<% @expense.expense_line_items.each_with_index do |expense_line_item, index| %> 
         <%= render "expense_line_items", :expense_line_item => expense_line_item, :index => index %> 
        <% end %> 
        <tr id="row_link" valign="top"> 
         <td valign="top" colspan="6"> 
          <%= link_to "Add new row",{:action => :add_row, :index => @expense.expense_line_items.size}, :remote => true %> 
         </td> 
        </tr> 
<% @expense.paid_line_items.each_with_index do |paid_line_item, index| %> 
        <%= render "paid_line_items", :paid_line_item => paid_line_item, :index => index %> 
        <% end %> 
        <tr id="to_row_link" valign="top"> 
        <td valign="top" colspan="6"> 
         <%= link_to "Add new row",{:action => :add_to_row, :index => @expense.paid_line_items.size}, :remote => true %> 
        </td> 
        </tr> 
<% end %> 

我很沮喪,在此先感謝。

+0

上面的代碼我沒有看到任何明顯的錯誤。控制器看起來沒問題,所以問題可能在於傳遞給更新的@params。如果沒有看到其他觀點,我們無法確定這些參數是什麼。你可以發佈他們嗎? (expense_line_item和paid_line_item的意見) – u2622 2012-02-11 03:00:05

+0

你可以在更新函數的開頭添加「raise params.inspect」並粘貼輸出嗎? – Rahul 2012-02-11 04:32:43

+0

謝謝@rahul我發現這個錯誤是由於我忘記在我的訂單項中添加的代碼。我必須爲我的訂單項添加一個隱藏的ID字段。感謝您的支持。 – Ravindra 2012-02-22 08:47:03

回答

0

我發現了這個問題的解決方案。當我嘗試更新一個lineitem時,它將它作爲一個新的,因此我必須通過隱藏的lineitem_id更新操作。我有下面的代碼使用 :

<%= hidden_field_tag "expense[expense_line_items_attributes][#{index}][id]",expense_line_item.id%> 

,它爲我工作。