Skip to content

Commit

Permalink
[16.0][FIX] product_packaging_level: Preserve sequence order
Browse files Browse the repository at this point in the history
  • Loading branch information
p-tombez authored and santostelmo committed Mar 6, 2024
1 parent f5708e8 commit 7e2593e
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions product_packaging_level/models/product_packaging.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@

class ProductPackaging(models.Model):
_inherit = "product.packaging"
_order = "product_id, level_sequence"

sequence = fields.Integer(compute="_compute_sequence", store=True, readonly=False)
packaging_level_id = fields.Many2one(
"product.packaging.level",
required=True,
Expand All @@ -19,12 +19,6 @@ class ProductPackaging(models.Model):
barcode_required_for_gtin = fields.Boolean(
readonly=True, compute="_compute_barcode_required_for_gtin"
)
level_sequence = fields.Integer(
string="Level Sequence",
related="packaging_level_id.sequence",
readonly=True,
store=True,
)
qty_per_level = fields.Char(
compute="_compute_qty_per_level", string="Qty per package level"
)
Expand Down Expand Up @@ -61,6 +55,11 @@ def _check_one_packaging_level_per_product(self):
).format(product.display_name)
)

@api.depends("packaging_level_id")
def _compute_sequence(self):
for packaging in self:
packaging.sequence = packaging.packaging_level_id.sequence

@api.depends("packaging_level_id", "packaging_level_id.has_gtin", "qty")
def _compute_barcode_required_for_gtin(self):
for packaging in self:
Expand Down

0 comments on commit 7e2593e

Please sign in to comment.