[trunk/7.0/6.1] mrp: outgoing picking not created in subcontracted production

[Jordi Llinares - KM Sistemas]

Deep testing production with subcontracting, I found the following issue:

I have created a supplier location linked to a supplier address. I have created a routing where production location is that supplier's location (let's call it 'external routing'). I have a manufacturable product with a BoM, to which I may have decided to link the external routing or not (blank). And that is the point, because it makes the mrp.production method '_make_production_internal_shipment' behave correctly or not:

If the product's BoM has my 'external routing' linked, _make_production_internal_shipment creates an outgoing picking for raw materials. Right.

But if the product's BoM has no routing, _make_production_internal_shipment will create an internal picking, EVEN if I have chosen my 'external routing' in the manufacturing order.

Shouldn't _make_production_internal_shipment look first at the production's own routing, rather than going directly to bom_id.routing?

I attach a quite simple proposal to fix this. Hope it helps.

Greetings.

[Jordi Llinares - KM Sistemas (jorgellinares-deactivatedaccount)]

Please, ignore last changed line of my patch file. I crossed code of both 6.1 and 7.0 versions. Sorry.