2014-10-09 68 views
0

需要一些幫助在這裏:-)施普雷錯誤使用裝飾與原代碼的時候

我試圖用裝飾來擴展Order類,但我得到一個錯誤回來,甚至當我使用與源代碼完全相同的代碼。例如:

order_decorator.rb(該方法是完全一樣的源,我只是使用裝飾)

Spree::Order.class_eval do 

def update_from_params(params, permitted_params, request_env = {}) 
    success = false 
    @updating_params = params 
    run_callbacks :updating_from_params do 
    attributes = @updating_params[:order] ? @updating_params[:order].permit(permitted_params).delete_if { |k,v| v.nil? } : {} 

    # Set existing card after setting permitted parameters because 
    # rails would slice parameters containg ruby objects, apparently 
    existing_card_id = @updating_params[:order] ? @updating_params[:order][:existing_card] : nil 

    if existing_card_id.present? 
     credit_card = CreditCard.find existing_card_id 
     if credit_card.user_id != self.user_id || credit_card.user_id.blank? 
     raise Core::GatewayError.new Spree.t(:invalid_credit_card) 
     end 

     credit_card.verification_value = params[:cvc_confirm] if params[:cvc_confirm].present? 

     attributes[:payments_attributes].first[:source] = credit_card 
     attributes[:payments_attributes].first[:payment_method_id] = credit_card.payment_method_id 
     attributes[:payments_attributes].first.delete :source_attributes 
    end 

    if attributes[:payments_attributes] 
     attributes[:payments_attributes].first[:request_env] = request_env 
    end 

    success = self.update_attributes(attributes) 
    set_shipments_cost if self.shipments.any? 
    end 

    @updating_params = nil 
    success 
end 

end 

當運行該代碼,熱潮從未發現@updating_params [:訂單] [:existing_card],即使我選擇了現有的卡。因此,我永遠無法使用預先存在的卡和僞造網關來完成交易(取而代之的是空白空白錯誤)。

我試圖使用撬的方法結合在order_decorator.rb和注意到[:existing_card]是實際上在@updating_params「電平,而不是在@updating_params [:爲了]的水平。

當我刪除裝飾器時,原始代碼正常工作。

有人可以向我解釋我的代碼有什麼問題嗎?

感謝,

回答

0

要重新定義的方法是不是真的Order類的方法。這是由Order類中的Checkout模塊混合的方法。 你可以在這裏看到:https://github.com/spree/spree/blob/master/core/app/models/spree/order/checkout.rb

嘗試做你想做的是這樣的:

的代碼創建文件app/models/spree/order/checkout.rb

Spree::Order::Checkout.class_eval do 
    def self.included(klass) 
    super 
    klass.class_eval do 
     def update_from_params(params, permitted_params, request_env = {}) 
     ... 
     ... 
     ... 
     end 
    end 
    end 
end