2014-10-09 179 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