diff --git a/delivery_stamps/__manifest__.py b/delivery_stamps/__manifest__.py
index 382e8d60..e10a2eff 100644
--- a/delivery_stamps/__manifest__.py
+++ b/delivery_stamps/__manifest__.py
@@ -3,7 +3,7 @@
{
'name': 'Stamps.com (USPS) Shipping',
'summary': 'Send your shippings through Stamps.com and track them online.',
- 'version': '13.0.1.0.0',
+ 'version': '13.0.1.1.0',
'author': "Hibou Corp.",
'category': 'Warehouse',
'license': 'OPL-1',
diff --git a/delivery_stamps/models/delivery_stamps.py b/delivery_stamps/models/delivery_stamps.py
index c9a1829d..9f1c284a 100644
--- a/delivery_stamps/models/delivery_stamps.py
+++ b/delivery_stamps/models/delivery_stamps.py
@@ -145,11 +145,13 @@ class ProviderStamps(models.Model):
ret_val.ContentType = self._stamps_content_type()
return ret_val
- def _get_stamps_shipping_multi(self, service, date_planned, order=False, picking=False):
+ def _get_stamps_shipping_multi(self, service, date_planned, order=False, picking=False, package=False):
if order:
weight = sum([(line.product_id.weight * line.product_qty) for line in order.order_line]) or 0.0
- else:
+ elif not package:
weight = picking.shipping_weight
+ else:
+ weight = package.shipping_weight or package.weight
weight = self._stamps_convert_weight(weight)
shipper = self.get_shipper_warehouse(order=order, picking=picking)
@@ -162,7 +164,7 @@ class ProviderStamps(models.Model):
ret_val.ShipDate = date_planned.strftime('%Y-%m-%d') if date_planned else date.today().isoformat()
ret_val.From = self._stamps_address(service, shipper)
ret_val.To = self._stamps_address(service, recipient)
- ret_val.PackageType = self._stamps_package_type()
+ ret_val.PackageType = self._stamps_package_type(package=package)
ret_val.WeightLb = weight
ret_val.ContentType = 'Merchandise'
return ret_val
@@ -204,7 +206,13 @@ class ProviderStamps(models.Model):
if not all((from_partner.zip, to_partner.zip)):
raise ValidationError('Stamps needs ZIP/PostalCode. From: ' + str(from_partner.zip) + ' To: ' + str(to_partner.zip))
- for package in picking.package_ids:
+ picking_packages = picking.package_ids
+ package_carriers = picking_packages.mapped('carrier_id')
+ if package_carriers:
+ # only ship ours
+ picking_packages = picking_packages.filtered(lambda p: p.carrier_id == self and not p.carrier_tracking_ref)
+
+ for package in picking_packages:
weight = self._stamps_convert_weight(package.shipping_weight)
l, w, h = self._stamps_package_dimensions(package=package)
@@ -221,7 +229,7 @@ class ProviderStamps(models.Model):
ret_val.WeightLb = weight
ret_val.ContentType = self._stamps_content_type(package=package)
ret.append((package.name + ret_val.ShipDate + str(ret_val.WeightLb) + self._stamps_hash_partner(to_partner), ret_val))
- if not ret:
+ if not ret and not package_carriers:
weight = self._stamps_convert_weight(picking.shipping_weight)
l, w, h = self._stamps_package_dimensions()
@@ -310,6 +318,8 @@ class ProviderStamps(models.Model):
package_labels = []
shippings = self._stamps_get_shippings_for_picking(service, picking)
+ if not shippings:
+ continue
company, from_partner, to_partner = self._stamps_get_addresses_for_picking(picking)
customs = None
@@ -451,14 +461,23 @@ class ProviderStamps(models.Model):
except WebFault as e:
raise ValidationError(e)
- def stamps_rate_shipment_multi(self, order=None, picking=None):
+ def stamps_rate_shipment_multi(self, order=None, picking=None, packages=None):
+ if not packages:
+ return self._stamps_rate_shipment_multi_package(order=order, picking=picking)
+ else:
+ rates = []
+ for package in packages:
+ rates += self._stamps_rate_shipment_multi_package(order=order, picking=picking, package=package)
+ return rates
+
+ def _stamps_rate_shipment_multi_package(self, order=None, picking=None, package=None):
self.ensure_one()
date_planned = fields.Datetime.now()
if self.env.context.get('date_planned'):
date_planned = self.env.context.get('date_planned')
res = []
service = self._get_stamps_service()
- shipping = self._get_stamps_shipping_multi(service, date_planned, order=order, picking=picking)
+ shipping = self._get_stamps_shipping_multi(service, date_planned, order=order, picking=picking, package=package)
rates = service.get_rates(shipping)
for rate in rates:
price = float(rate.Amount)
@@ -484,6 +503,7 @@ class ProviderStamps(models.Model):
if carrier:
res.append({
'carrier': carrier,
+ 'package': package or self.env['stock.quant.package'].browse(),
'success': True,
'price': price,
'error_message': False,
diff --git a/delivery_stamps/views/delivery_stamps_view.xml b/delivery_stamps/views/delivery_stamps_view.xml
index 4320c0e2..3508b348 100644
--- a/delivery_stamps/views/delivery_stamps_view.xml
+++ b/delivery_stamps/views/delivery_stamps_view.xml
@@ -32,7 +32,7 @@
product.packaging
-
+