The question mark (or rather the query params part of the URL) sadly never makes it into ApplicationController#cache_page which means the conditional in there is useless as it is right now. Can we check @request?

We should really remove /theme from the route space. If we remove the default :controller/:action/... line, it'll break at least a couple article actions, but does anything else depend on it? Perhaps adding a /articles/:action line in place of :controler/:action would do the trick?

Alternately, we could just put a conditional on :controller to make it not equal theme.

What happened to the description of the ticket? My link turned into an image tag...

Wow, functional tests are amazingly unhappy without the default route.

I'm going to commit a route that deep-sixes all /theme/* URLs. This isn't the ideal fix for this, but removing the default :controller/:action/:id line breaks too many things for this phase of development. Once we get 2.5 out and we're more willing to break things, then we should come up with a cleaner fix.

(In r474) Blocked access to theme_controller directly via /theme. (closes #243)

See #252 for the milestone-4.0 cleanup associated with this.

Instead of the fix in r474 you could have added a condition on the anything route to make sure :controller didn't equal theme, something like :controller => /^(?!theme$).*$/.