DynamoMTL / spree_chimpy

Spree/MailChimp Integration
BSD 3-Clause "New" or "Revised" License
35 stars 123 forks source link

500 Server Error editing order. #21

Closed rterbush closed 8 years ago

rterbush commented 11 years ago

Attempting to access an order through the admin panel errors out with the following.

Started GET "/admin/orders/R781673870/edit" for 174.29.117.52 at 2013-06-18 07:02:57 -0600
Processing by Spree::Admin::OrdersController#edit as HTML
  Parameters: {"id"=>"R781673870"}
spree_chimpy: Attempting to remove order R781673870
spree_chimpy: Adding order R781673870
Completed 500 Internal Server Error in 1186ms

Hominid::APIError (<330> Category Ids must be integers, not "".):
  vendor/bundle/ruby/1.9.1/gems/hominid-3.0.5/lib/hominid/api.rb:35:in `rescue in method_missing'
  vendor/bundle/ruby/1.9.1/gems/hominid-3.0.5/lib/hominid/api.rb:32:in `method_missing'
  vendor/bundle/ruby/1.9.1/bundler/gems/spree_chimpy-d819392b5ea9/lib/spree/chimpy/interface/orders.rb:15:in `add'
  vendor/bundle/ruby/1.9.1/bundler/gems/spree_chimpy-d819392b5ea9/lib/spree/chimpy/interface/orders.rb:30:in `sync'
  vendor/bundle/ruby/1.9.1/bundler/gems/spree_chimpy-d819392b5ea9/lib/spree_chimpy.rb:101:in `perform'
  vendor/bundle/ruby/1.9.1/bundler/gems/spree_chimpy-d819392b5ea9/lib/spree_chimpy.rb:89:in `handle_event'
  vendor/bundle/ruby/1.9.1/bundler/gems/spree_chimpy-d819392b5ea9/lib/spree/chimpy/engine.rb:27:in `block (2 levels) in <class:Engine>'
  vendor/bundle/ruby/1.9.1/gems/activesupport-3.2.13/lib/active_support/notifications/fanout.rb:47:in `call'
johanb commented 11 years ago

Thanks for reporting this @rterbush. I'll look into this tonight.

johanb commented 11 years ago

@rterbush could you clarify which version of Spree you're using? I'm assuming 2.0.3 correct ?

rterbush commented 11 years ago

Correct 2-0-stable. Sorry this report is a bit thin but wanted to capture it while I am seeing it. Let me know what I can do to help.

johanb commented 11 years ago

Today I've encountered a similar issue in 1-3-stable. Investigating

rterbush commented 11 years ago

@johanb I suspect that PR that I sent awhile back applies to 1-3-stable as well.

braidn commented 8 years ago

Hey all! Going to close this due to age. Feel free to reopen if it is still an issue in your apps. Thanks again.