@@ -15,7 +15,7 @@ class OrdersController < Spree::BaseController create.after do params[:products].each do |product_id,variant_id| quantity = params[:quantity].to_i if !params[:quantity].is_a?(Array) - quantity = params[:quantity][variant_id].to_i if params[:quantity].is_a?(Array) + quantity = params[:quantity][variant_id].to_i if params[:quantity].is_a?(Array) @order.add_variant(Variant.find(variant_id), quantity) if quantity > 0 end if params[:products] @@ -24,7 +24,7 @@ class OrdersController < Spree::BaseController @order.add_variant(Variant.find(variant_id), quantity) if quantity > 0 end if params[:variants] - @order.save + @order.save # store order token in the session session[:order_token] = @order.token app/controllers/orders_controller.rb @@ -13,13 +13,13 @@ describe OrdersController do it "should add the variant to the order" do controller.stub!(:object).and_return(@order) @order.should_receive(:add_variant).with(@variant, 2) - post :create, :variants => {"345" => 2} + post :create, :variants => {"345" => 2} end - + it "should not set the state" do @order.should_not_receive(:state=) post :create, :id => @order.number, :quantities => {456 => "123=1"}, :order => {:state => "paid"} - end + end end describe "update" do spec/controllers/orders_controller_spec.rb 97c50aaedbdeb8d2247ac53447d09b95eb882899 David North davidnorth@gmail.com http://github.com/akitaonrails/spree/commit/7f3fa6b4ed7caa69a8b2c495d93ba4d7d13fecff 7f3fa6b4ed7caa69a8b2c495d93ba4d7d13fecff 2009-04-15T19:31:29-07:00 2009-02-27T18:57:35-08:00 Fix that if you tried to add a product to the cart that had variants but you forgot to pick one, you got an application error. Client side validation still required. fb4284e53537418c016b1cc2e1255df8e2ed759a Sean Schofield schof@apache.org