diff --git a/rma/tests/test_rma.py b/rma/tests/test_rma.py index 6b9883f5..a923d291 100644 --- a/rma/tests/test_rma.py +++ b/rma/tests/test_rma.py @@ -671,7 +671,9 @@ class TestRma(common.SavepointCase): wizard._create_picking() res = self.rma_supplier_id.rma_line_ids.action_view_out_shipments() self.assertTrue("res_id" in res, "Incorrect number of pickings" "created") - picking = self.env["stock.picking"].browse(res["res_id"]) + picking = self.rma_supplier_id.rma_line_ids._get_out_pickings() + partner = picking.partner_id + self.assertTrue(partner, "Partner is not defined or False") moves = picking.move_lines self.assertEquals(len(moves), 3, "Incorrect number of moves created") @@ -791,6 +793,8 @@ class TestRma(common.SavepointCase): pickings = self.env["stock.picking"].browse(res["res_id"]) self.assertEquals(len(pickings), 1, "Incorrect number of pickings created") picking_in = pickings[0] + partner = picking_in.partner_id + self.assertTrue(partner, "Partner is not defined or False") moves = picking_in.move_lines self.assertEquals(len(moves), 3, "Incorrect number of moves created") @@ -928,3 +932,20 @@ class TestRma(common.SavepointCase): ).create({}) with self.assertRaisesRegex(ValidationError, "No quantity to transfer"): wizard._create_picking() + + def test_09_supplier_rma_single_line(self): + rma_line_id = self.rma_supplier_id.rma_line_ids[0].id + wizard = self.rma_make_picking.with_context( + { + "active_ids": [rma_line_id], + "active_model": "rma.order.line", + "picking_type": "outgoing", + "active_id": 2, + } + ).create({}) + wizard._create_picking() + picking = self.rma_supplier_id.rma_line_ids[0]._get_out_pickings() + partner = picking.partner_id + self.assertTrue(partner, "Partner is not defined or False") + moves = picking.move_lines + self.assertEqual(len(moves), 1, "Incorrect number of moves created") diff --git a/rma/wizards/rma_make_picking.py b/rma/wizards/rma_make_picking.py index 337be30f..9520b6e4 100644 --- a/rma/wizards/rma_make_picking.py +++ b/rma/wizards/rma_make_picking.py @@ -123,7 +123,7 @@ class RmaMakePicking(models.TransientModel): procurement_data = { "name": line.rma_id and line.rma_id.name or line.name, "group_id": group, - "origin": line.name, + "origin": group and group.name or line.name, "warehouse_id": warehouse, "date_planned": time.strftime(DT_FORMAT), "product_id": item.product_id,