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

Spree routes override #8

Closed chipiga closed 14 years ago

chipiga commented 14 years ago

Your last commit http://github.com/PeterBerkenbosch/spree-static-content/commit/ae903a1f4b10f3b38491c71e73e27d5990db16de makes spree process only static pages. Here is a part of my log

Processing StaticContentController#show (for 127.0.0.1 at 2010-03-03 09:36:04) [GET] Parameters: {"action"=>"show", "path"=>["products"], "controller"=>"static_content"} Cache miss: Spree::Config ({}) Preference Load (0.5ms) SELECT * FROM preferences WHERE (preferences.owner_id = 1 AND preferences.owner_type = 'Configuration') Configuration Columns (1.8ms) SHOW FIELDS FROM configurations Configuration Load (0.5ms) SELECT * FROM configurations WHERE (configurations.id = 1) CACHE (0.0ms) SELECT * FROM configurations WHERE (configurations.id = 1) CACHE (0.0ms) SELECT * FROM configurations WHERE (configurations.id = 1) CACHE (0.0ms) SELECT * FROM configurations WHERE (configurations.id = 1) Cache write (will save 7.05ms): Spree::Config Page Columns (1.8ms) SHOW FIELDS FROM pages Page Load (0.4ms) SELECT * FROM pages WHERE (pages.slug = '/products') AND (pages.visible = 1) ORDER BY position ASC LIMIT 1 CACHE (0.0ms) SELECT * FROM pages WHERE (pages.slug = '/products') AND (pages.visible = 1) ORDER BY position ASC LIMIT 1 Rendering /Users/pasha/Sites/darite/public/404.html (404) Completed in 163ms (View: 2, DB: 9) | 404 Not Found [http://localhost/products]

Looks like this route map.static '*path', :controller => 'static_content', :action => 'show' override others. Commit rollback solves the problem.

peterberkenbosch commented 14 years ago

oh dear.... I see it... fill fix this asap!