2012-02-10 82 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%> 

,它为我工作。