You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the sourcecode of DpdLabelGenerator.php, there are some arbitrary values in the weight check.
If weights are not set in the shop, it assumes that the item weighs 5kg, which leads to unexplainable errors in frontend when creating labels for orders. Please remove this strange default value or at least explain it in documentation (or the error).
Also, weight is entered in KG's in prestashop, then multiplied by 100 (?) in the plugin and checked against a limit of 31,5 * 100, I assume this is supposed to avoid rounding errors, but multiplying by 1000 (to compare in grams) would make more sense.
Thanks for your reply, but the API needs a parcel weight that is in grams and rounded in 10 gram units without decimal delimiter (e.g. 300 equals 3kg). So a 1000 would actually make more sense, but because of how the API was setup, the calculation is right.
That explains! Thanks for commenting on this :-)
Still leaves the part where it randomly assigns weight to invoice lines in tact though.
I could not ship serveral of my parcels because the amount of invoice lines * 5 kg resulted in a weight over 31,5 kg, although the actual package is onlye a few kilo's.
I understand this would be solved by adding weight to all the products, but that should not be strictly necessary, I think.
In the sourcecode of DpdLabelGenerator.php, there are some arbitrary values in the weight check.
If weights are not set in the shop, it assumes that the item weighs 5kg, which leads to unexplainable errors in frontend when creating labels for orders. Please remove this strange default value or at least explain it in documentation (or the error).
Also, weight is entered in KG's in prestashop, then multiplied by 100 (?) in the plugin and checked against a limit of 31,5 * 100, I assume this is supposed to avoid rounding errors, but multiplying by 1000 (to compare in grams) would make more sense.
...
The text was updated successfully, but these errors were encountered: