Issue 11036

Title
Package weight and volume are not included in Total Weight or Total Volume
Priority
bug
Status
resolved
Nosy list
ced, mrichez, reviewbot, roundup-bot, tbruyere
Assigned to
ced
Keywords
review

Created on 2021-12-09.15:53:51 by mrichez, last changed 5 hours ago by roundup-bot.

Messages

New changeset c761581bfa94 by Cédric Krier in branch '6.2':
Add packing weight to total weight of package
https://hg.tryton.org/modules/stock_shipment_measurements/rev/c761581bfa94

New changeset 3d4e67f89818 by Cédric Krier in branch '6.0':
Add packing weight to total weight of package
https://hg.tryton.org/modules/stock_shipment_measurements/rev/3d4e67f89818

New changeset 0fa5e75050e4 by Cédric Krier in branch '5.0':
Add packing weight to total weight of package
https://hg.tryton.org/modules/stock_shipment_measurements/rev/0fa5e75050e4
New changeset 6e8713859a08 by Cédric Krier in branch 'default':
Add packing weight to total weight of package
https://hg.tryton.org/tryton-env/rev/6e8713859a08
New changeset b1228f8f5b6e by Cédric Krier in branch 'default':
Add packing weight to total weight of package
https://hg.tryton.org/modules/stock_shipment_measurements/rev/b1228f8f5b6e
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2022-01-06.17:52:54

Indeed we are actually missing the package weight to be added to the total_weight since it was added by issue10145.
So review373001002 is indeed for a new feature issue11123.
And here is review384031002 which adds the packaging weight to the total_weight.

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2022-01-05.22:09:36

For me we are just missing a field that sum the weight of the packages.
And optionally to display the measurements on the form view of the shipments.

Author: [hidden] (mrichez)
Date: 2021-12-10.15:09:17

I opened a discuss to have some thoughts about weight fields : https://discuss.tryton.org/t/improvments-concerning-some-details-on-packages/4846

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-12-10.10:28:40
On 2021-12-10 10:25, Maxime Richez wrote:
> Maybe we could have 2 weights (gross weight and net weight) ? (it could be useful for Customs purpose too)

I think we need first to analyse which one is needed for shipping
service. I prefer to not add field "just in case".
Author: [hidden] (mrichez)
Date: 2021-12-10.10:25:02

Maybe we could have 2 weights (gross weight and net weight) ? (it could be useful for Customs purpose too)

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-12-10.10:06:24

Indeed the _carrier_weight is used only to compute in advance the carrier cost (even if we do not use packages).
But the total_weight is used for the shipping service to create labels.
So it is actually a bug that the package weight is not included in the total_weight. But I think we must ensure that shipping service to which we send this weight is actually expecting the total weight with the package weight and not the total weight of the content only.

Author: [hidden] (mrichez)
Date: 2021-12-10.09:47:22

Indeed _carrier_weight has the right weight but it seems this information is not used on the labels. (and on the form too)
Is _carrier_weight just used to calculate shipment costs ?

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-12-09.23:11:12

For me _parcel_weight computes correctly so the carrier has the right weight, so it is only about the display that why for me it is just a feature.

Author: [hidden] (mrichez)
Date: 2021-12-09.20:10:50

The weight is missing the package weight which could be ridiculous in some cases but becomes a problem when using for instance a pallet.

One of our test case was :

  • Main package: Pallet (100x120x15cm) - Weight : 18 Kg
  • Child: Box (weight 200 gr) linked to move with product A - Qty: 500 - Weight of product: 10 gr

Total weight displayed: 5 Kg (500 x 10gr)

Total weight expected: 5 Kg + 18 Kg (weight of pallet) + 200 gr (weight of box) = 23,200 Kg

And i'm pretty sure carrier needs to know the total weight (pallet included) and total volume...

PS.: we customize to add 2 fields on package (start number and end number) this way we could define only one package type to link with move but with those numbers it defines a quantity of packages.

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2021-12-09.16:07:34

I do not think it is a bug. For me the current behavior does not generate wrong outcome. The weight is correct for the carrier etc.

Author: [hidden] (mrichez)
Date: 2021-12-09.15:57:23

I think the method get_total_measurements (https://hg.tryton.org/modules/stock_shipment_measurements/file/default/stock.py#l279) should take into account "packaging_weight"

Author: [hidden] (mrichez)
Date: 2021-12-09.15:53:51

We define a weight and dimensions on a package.
When using this package on shipment out, the total weight/volume displayed doesn't take care of weight/volume of package (only the weight of the products on the move).

History
Date User Action Args
2022-01-22 21:18:19roundup-botsetkeyword: - backport
messages: + msg73404
2022-01-20 19:01:23roundup-botsetmessages: + msg73351
2022-01-20 19:01:20roundup-botsetmessages: + msg73350
nosy: + roundup-bot
status: testing -> resolved
2022-01-06 18:13:21reviewbotsetmessages: + msg73166
2022-01-06 17:52:54cedsetkeyword: + backport
messages: + msg73164
priority: feature -> bug
reviews: 373001002 -> 384031002
type: feature request -> behavior
2022-01-05 22:21:02reviewbotsetmessages: + msg73139
nosy: + reviewbot
2022-01-05 22:10:12cedsetkeyword: + review
reviews: 373001002
status: in-progress -> testing
2022-01-05 22:09:36cedsetassignedto: ced
keyword: - backport
messages: + msg73138
priority: bug -> feature
status: chatting -> in-progress
type: behavior -> feature request
2021-12-10 15:09:17mrichezsetmessages: + msg72098
2021-12-10 10:28:42cedsetmessages: + msg72091
2021-12-10 10:25:02mrichezsetmessages: + msg72090
2021-12-10 10:06:24cedsetkeyword: + backport
messages: + msg72089
priority: feature -> bug
2021-12-10 09:47:22mrichezsetmessages: + msg72088
2021-12-09 23:11:12cedsetmessages: + msg72087
2021-12-09 20:10:50mrichezsetmessages: + msg72086
2021-12-09 16:07:34cedsetmessages: + msg72077
nosy: + ced
priority: bug -> feature
status: unread -> chatting
2021-12-09 15:57:23mrichezsetmessages: + msg72075
2021-12-09 15:53:51mrichezcreate