From 492ecd60f65afd58e89190b54d5431fe80cda5cb Mon Sep 17 00:00:00 2001 From: Jared Kipe Date: Mon, 4 Oct 2021 14:09:18 -0700 Subject: [PATCH] [FIX] delivery_hibou: context on button inside tree view for cancel, price can be Decimal --- delivery_hibou/models/stock.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/delivery_hibou/models/stock.py b/delivery_hibou/models/stock.py index 5f940728..52e9c670 100644 --- a/delivery_hibou/models/stock.py +++ b/delivery_hibou/models/stock.py @@ -12,7 +12,12 @@ class StockQuantPackage(models.Model): picking_id = self._context.get('active_id') picking_model = self._context.get('active_model') if not picking_id or picking_model != 'stock.picking': - raise UserError('Cannot cancel package other than through shipment/picking.') + params = self._context.get('params') + if params: + picking_id = params.get('id') + picking_model = params.get('model') + if not picking_id or picking_model != 'stock.picking': + raise UserError('Cannot cancel package other than through shipment/picking.') return self.env['stock.picking'].browse(picking_id) def send_to_shipper(self): @@ -116,7 +121,7 @@ class StockPicking(models.Model): res = res[0] if carrier.free_over and self.sale_id and self.sale_id._compute_amount_total_without_delivery() >= carrier.amount: res['exact_price'] = 0.0 - carrier_price = res['exact_price'] * (1.0 + (self.carrier_id.margin / 100.0)) + carrier_price = float(res['exact_price']) * (1.0 + (self.carrier_id.margin / 100.0)) carrier_prices.append(carrier_price) tracking_number = '' if res['tracking_number']: