audrius/rma
1
0
Fork 1
mirror of https://github.com/OCA/rma.git synced 2025-02-16 17:11:47 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge PR #310 into 13.0

Browse Source 
Signed-off-by pedrobaeza
This commit is contained in:
OCA-git-bot
2022-09-21 09:20:06 +00:00
parent aadecb72bd 1459bf2484
commit cfc396b7b1
3 changed files with 13 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
rma_sale/models/rma.py
View File

@@ -35,7 +35,7 @@ class Rma(models.Model):
def _compute_allowed_picking_ids(self):
domain = [("state", "=", "done"), ("picking_type_id.code", "=", "outgoing")]
for rec in self:
# if rec.partner_id:
if rec.partner_id:
commercial_partner = rec.partner_id.commercial_partner_id
domain.append(("partner_id", "child_of", commercial_partner.id))
if rec.order_id:
@@ -48,7 +48,7 @@ class Rma(models.Model):
if rec.order_id:
order_move = rec.order_id.order_line.mapped("move_ids")
rec.allowed_move_ids = order_move.filtered(
lambda r: r.picking_id == self.picking_id
lambda r: r.picking_id == self.picking_id and r.state == "done"
).ids
else:
rec.allowed_move_ids = self.picking_id.move_lines.ids

5
rma_sale/wizard/sale_order_rma_wizard.py
View File

@@ -152,6 +152,7 @@ class SaleOrderLineRmaWizard(models.TransientModel):
r.sale_line_id == record.sale_line_id
and r.sale_line_id.product_id == record.product_id
and r.sale_line_id.order_id == record.order_id
and r.state == "done"
)
)
else:
@@ -169,7 +170,9 @@ class SaleOrderLineRmaWizard(models.TransientModel):
line = record.order_id.order_line.filtered(
lambda r: r.product_id == record.product_id
)
record.allowed_picking_ids = line.mapped("move_ids.picking_id")
record.allowed_picking_ids = line.mapped("move_ids.picking_id").filtered(
lambda x: x.state == "done"
)
def _prepare_rma_values(self):
self.ensure_one()

6
rma_sale/wizard/sale_order_rma_wizard_views.xml
View File

@@ -25,7 +25,11 @@
options="{'no_create': True}"
/>
<field name="allowed_picking_ids" invisible="1" />
<field name="picking_id" options="{'no_create': True}" />
<field
name="picking_id"
options="{'no_create': True}"
required="1"
/>
<field name="operation_id" />
</tree>
</field>