audrius/stock-logistics-warehouse
1
0
Fork 1
mirror of https://github.com/OCA/stock-logistics-warehouse.git synced 2025-01-21 14:27:28 +02:00
Code Issues Packages Projects Releases Wiki Activity

[FIX] stock_quant_manual_assign: fix access error issues

Browse Source 
When not all features are activated in inventory (locations, lots,
packages and owners) selecting a quant would raise an access error.

It is not possible to directly read the related fields, e.g
`self.lot_id`, because they are stored fields and have a group
restriction, so an access error would raise. To work around it,
we should access these fields from the quant:
`self.quant_id.lot_id`.
This commit is contained in:
Lois Rilo
2022-07-12 09:30:19 +02:00
committed by Alexis de Lattre
parent 7f47c3c17a
commit cd2942bca6
1 changed files with 13 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
stock_quant_manual_assign/wizard/assign_manual_quants.py
View File

@@ -130,6 +130,11 @@ class AssignManualQuantsLines(models.TransientModel):
required=True,
ondelete="cascade",
)
# Fields below are storable in order to be able to order by them. However,
# there is a side effect: It is not possible to directly read the related
# fields, e.g `self.lot_id`, because they are stored fields and have a group
# restriction, so an access error would raise. To work around it, we should
# access these fields from the quant: `self.quant_id.lot_id`.
location_id = fields.Many2one(
comodel_name="stock.location",
string="Location",
@@ -224,10 +229,10 @@ class AssignManualQuantsLines(models.TransientModel):
if float_compare(self.qty, 0.0, precision_digits=precision_digits) > 0:
available_quantity = quant._get_available_quantity(
move.product_id,
self.location_id,
lot_id=self.lot_id,
package_id=self.package_id,
owner_id=self.owner_id,
self.quant_id.location_id,
lot_id=self.quant_id.lot_id,
package_id=self.quant_id.package_id,
owner_id=self.quant_id.owner_id,
)
if (
float_compare(
@@ -239,9 +244,9 @@ class AssignManualQuantsLines(models.TransientModel):
move._update_reserved_quantity(
self.qty,
available_quantity,
self.location_id,
lot_id=self.lot_id,
package_id=self.package_id,
owner_id=self.owner_id,
self.quant_id.location_id,
lot_id=self.quant_id.lot_id,
package_id=self.quant_id.package_id,
owner_id=self.quant_id.owner_id,
strict=True,
)