[8.0] New option of grouping one_sale_one_purchase

[drl9319]

I create new option (One Sale, One Purchase) in procurement_purchase_no_groping to add new funcionality.
This funcionality make that: One sale order create only one purchase order for supllier with all related lines. This funcion make that only groping purchase order from de same sale order.

[rubencabrera]

+1
Works and solves a common problem, good one!

[anajuaristi]

Good point. Let us review code and we will merge it asap. @oihane @agaldona @alfredo Daniel, thank you veru much for contributing Ana

[pedrobaeza]

This module has been proposed now to be in OCA. Maybe you should propose there also the improvement after the merge. Anyway, Travis is red, so this is the first thing to fix.

[codecov-io]

Codecov Report

Merging #1352 into 8.0 will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              8.0    #1352      +/-   ##
==========================================
+ Coverage   58.74%   58.76%   +0.02%     
==========================================
  Files         269      269              
  Lines        8075     8079       +4     
==========================================
+ Hits         4744     4748       +4     
  Misses       3331     3331

Impacted Files	Coverage Δ
...nt_purchase_no_grouping/models/product_category.py	100% <ø> (ø)	✅
...t_purchase_no_grouping/models/procurement_order.py	100% <ø> (ø)	✅
...ment_purchase_no_grouping/models/purchase_order.py	100% <100%> (ø)	✅

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e02141e...d9acf54. Read the comment docs.

[oihane]

@drl9319 is this correct? It seems to be a misspell

[drl9319]

This is an Error because arsg is a list an in Python if I assign list = list is by reference, for that this works ok. But I clean the code now.

[drl9319]

Now the error is for the function is in old api, but this module before has this funcion (in old api) and was be in the master branch

[drl9319]

What happens with the pull request?

[carlosap92]

It works fine! +1

[pedrobaeza]

Again, it's better to move the efforts to OCA instead here: OCA/purchase-workflow#343

[jesuspablo]

+1
good job..

[anajuaristi]

Travis failed so we can't merge until error is solved.
Sorry but we are quite strict on this.