1.3-era reference to @order.shipment is ambiguous in light of 2.0's multiple shipment redesign

spree-contrib / spree_print_invoice

Create a PDF invoice for Spree orders.

http://guides.spreecommerce.org

BSD 3-Clause "New" or "Revised" License

91 stars 239 forks source link

1.3-era reference to @order.shipment is ambiguous in light of 2.0's multiple shipment redesign #28

Closed dpritchett closed 10 years ago

dpritchett commented 11 years ago

Specifically a problem in the address partial:

# Orders no longer have a single :shipping_method, what should this invoice even look like? 
data2 << [@order.shipping_method.try(:name), @order.shipping_method.try(:name)]

radar commented 10 years ago

This has been fixed in 2-0-stable now.