Alright, that makes perfect sense. Is there anything I need to to fix this? Do I need to remove this from my PR?
On Feb 17, 2013, at 11:21 PM, Ryan Bigg wrote:

> In core/app/controllers/spree/locale_controller.rb:
> 
> > @@ -10,7 +10,7 @@ def set
> >        else
> >          flash[:error] = t(:locale_not_changed)
> >        end
> > -      redirect_back_or_default(root_path)
> > +      redirect_back_or_default(spree.root_path)
> That wouldn't be a problem because the routes that would be used would still be for the Spree engine, not the application. It's the routes which define the routing helper contexts.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>