diff --git a/addons/delivery/models/delivery_carrier.py b/addons/delivery/models/delivery_carrier.py index 543c0feb0b7..f48eb8d9835 100644 --- a/addons/delivery/models/delivery_carrier.py +++ b/addons/delivery/models/delivery_carrier.py @@ -118,7 +118,7 @@ class DeliveryCarrier(models.Model): order = SaleOrder.browse(order_id) if self.delivery_type not in ['fixed', 'base_on_rule']: try: - computed_price = self.get_shipping_price_from_so(order)[0] + computed_price = self.get_shipping_price_from_so(order) self.available = True except ValidationError as e: # No suitable delivery method found, probably configuration error @@ -145,15 +145,15 @@ class DeliveryCarrier(models.Model): # TODO define and handle exceptions that could be thrown by providers - def get_shipping_price_from_so(self, orders): - ''' For every sales order, compute the price of the shipment + def get_shipping_price_from_so(self, order): + ''' Compute the price of the order shipment - :param orders: A recordset of sales orders - :return list: A list of floats, containing the estimated price for the shipping of the sales order + :param order: A recordset of a sales order + :return list: A float, the estimated price for the shipping of the sales order ''' self.ensure_one() if hasattr(self, '%s_get_shipping_price_from_so' % self.delivery_type): - return getattr(self, '%s_get_shipping_price_from_so' % self.delivery_type)(orders) + return getattr(self, '%s_get_shipping_price_from_so' % self.delivery_type)(order) def send_shipping(self, pickings): ''' Send the package to the service provider diff --git a/addons/delivery/models/sale_order.py b/addons/delivery/models/sale_order.py index ccfed1a2e74..4ff2ab101ed 100644 --- a/addons/delivery/models/sale_order.py +++ b/addons/delivery/models/sale_order.py @@ -55,7 +55,7 @@ class SaleOrder(models.Model): if carrier.delivery_type not in ['fixed', 'base_on_rule']: # Shipping providers are used when delivery_type is other than 'fixed' or 'base_on_rule' - price_unit = order.carrier_id.get_shipping_price_from_so(order)[0] + price_unit = order.carrier_id.get_shipping_price_from_so(order) else: # Classic grid-based carriers carrier = order.carrier_id.verify_carrier(order.partner_shipping_id)