pero-ict-solutions / spree-static-content

Extention to manage the static pages for your spree shop
BSD 3-Clause "New" or "Revised" License
89 stars 398 forks source link

Fixes a tiny bug when calling expire_page in production environment #18

Open mtylty opened 13 years ago

mtylty commented 13 years ago

When RAILS_ENV=production, calling expire_page (i.e. when updating a static page), results in a 404 (by calling a non-existant route)... This is due to the fact that the route passed to expire_page has a controller specified without a trailing slash. This results in the actual controller being called is 'admin/static_content' while it is evident (just by looking at the folder structure), that the 'static_content' controller is meant to be outside the admin namespace...

Hope you'll pull it :)

Keep up the good work.

Matteo

romul commented 13 years ago

Thank you. I applied your patch to spree fork - https://github.com/spree/spree_static_content/commit/a70594fb6d05e73acf426072b5ff2af269cf8a3a