Avoid name clashing when including modules

That's what I think... There might be more like this around, but I haven't
hit any other...

I've grep'd for includes on spree and found nothing, but my grep could be
wrong.
On Jun 23, 2014 5:37 AM, "John Hawthorn" <notifications@github.com> wrote:

> In core/app/models/spree/order.rb:
>
> > @@ -3,8 +3,8 @@
> >
> >  module Spree
> >    class Order < ActiveRecord::Base
> > -    include Checkout
> > -    include CurrencyUpdater
> > +    include Spree::Order::Checkout
> > +    include Spree::Order::CurrencyUpdater
>
> Rails' autoloading is a bit strange. It relies on const_missing, which is
> run when an undefined constant is referenced. If there is a ::Checkout
> constant already defined (which would happen if it had been referenced
> already before this file had been loaded), it would be a valid constant for
> ruby to find here and const_missing won't be triggered.
>
> Most won't need this change, but I don't think there's harm in including
> it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/spree/spree/pull/4844/files#r14065079>.
>