[Spree Upgrade] Handle references to order.shipping_method_id

[luisramos0]

I am not sure yet, but I believe this column should be dropped.

We need to handle all references to it:

• app/services/order_factory.rb
• lib/open_food_network/order_cycle_management_report.rb
• spec/features/consumer/shopping/orders_spec.rb
• spec/models/spree/order_spec.rb
• spec/services/order_factory_spec.rb
• db/migrate/20121005015852_populate_order_default_shipping_method.rb

There are 2 more references in separate issues: #2681 and #2682

[luisramos0]

I am going through each reference:

• app/services/order_factory.rb and spec/services/order_factory_spec.rb
  - Moved this refactoring taks to [Spree Upgrade] Fix subscriptions.shipping_method_id (13 broken specs) #2696 as it's subscriptions specific
• lib/open_food_network/order_cycle_management_report.rb
  - already addressed as part of [Spree Upgrade] Fixed inexistent order#shipping_method= in several specs #2680
• spec/features/consumer/shopping/orders_spec.rb
  - handled in new PR [Spree Upgrade] Fixed updates of order.shipping_method_id and improved order, stock_location and shipment factories #2762
• spec/models/spree/order_spec.rb
  - handled in new PR [Spree Upgrade] Fixed updates of order.shipping_method_id and improved order, stock_location and shipment factories #2762
• db/migrate/20121005015852_populate_order_default_shipping_method.rb
  - this is old and not relevant