spree-contrib / spree_multi_currency

Provides UI to allow configuring multiple currencies in Spree.
http://guides.spreecommerce.org
BSD 3-Clause "New" or "Revised" License
41 stars 118 forks source link

replace match with post because cannot install in rails 4.0.2 #11

Closed donnykurnia closed 10 years ago

donnykurnia commented 10 years ago

In rails 4.0.2, using match without :via will produce RuntimeError

/home/donny/.rvm/gems/ruby-2.0.0-p353@s5unity-spree/gems/actionpack-4.0.2/lib/action_dispatch/routing/mapper.rb:191:in `normalize_conditions!': You should not use the `match` method in your router without specifying an HTTP method. (RuntimeError)
If you want to expose your action to both GET and POST, add `via: [:get, :post]` option.
If you want to expose your action to GET, use `get` in the router:
  Instead of: match "controller#action"
  Do: get "controller#action"

Reference: http://guides.rubyonrails.org/routing.html#http-verb-constraints

peterberkenbosch commented 10 years ago

added to 2-1-stable and master. Thanks.