ForgeFlow / ddmrp

DDMRP
GNU Affero General Public License v3.0
14 stars 17 forks source link

[FIX] Empty buffer on PO line when receiving in 2 steps #60

Closed max3903 closed 6 years ago

max3903 commented 6 years ago

@guewen, @jbeficent, @lreficent, @jgrandguillaume If you have a better way to solve this, I am open to suggestions.

JordiBForgeFlow commented 6 years ago

I think that the approach of this PR is not the right one. You should define a separate module in OCA/stock-logistics-workflow called 'stock_orderpoint_purchase_link'.

Then define in purchase.order.line a field 'orderpoint_ids' as many2many. Then pass on the orderpoint from the procurement rule. Equivalent to what we did with stock requests here:

JordiBForgeFlow commented 6 years ago

Then, add to the ddmrp this new module as dependency, and benefit from the orderpoint_ids now available in the PO line to obtain the execution_priority_level

LoisRForgeFlow commented 6 years ago

I have adapted this as per the new approach see https://github.com/Eficent/ddmrp/pull/63