tryton-env: Improve moves synchronisation on drop shipment

tryton-env: Improve moves synchronisation on drop shipment

By using an origin drop as synchronisation key, the code is simplified but also
it allows to follow when products are changed.
This allows also to synchronise lots.
The origin drop is filled on the customer moves when the drop shipment is set
to waiting by using the purchase request as link.

issue11700

[reviewbot, 2022-09-12 12:18:19]

patch is not applicable
URL: https://codereview.tryton.org/411701003

[reviewbot, 2022-09-12 12:44:45]

checks OK
URL: https://codereview.tryton.org/411701003

[pokoli, 2022-09-12 12:53:37]

https://codereview.tryton.org/411701003/diff/441441003/modules/sale_supply_dr...
File modules/sale_supply_drop_shipment/stock.py (left):

https://codereview.tryton.org/411701003/diff/441441003/modules/sale_supply_dr...
modules/sale_supply_drop_shipment/stock.py:503: from_location=move.to_location,
I think we should keep the to_location from the source move when we have a
template.

[ced, 2022-09-12 14:40:51]

https://codereview.tryton.org/411701003/diff/441441003/modules/sale_supply_dr...
File modules/sale_supply_drop_shipment/stock.py (left):

https://codereview.tryton.org/411701003/diff/441441003/modules/sale_supply_dr...
modules/sale_supply_drop_shipment/stock.py:503: from_location=move.to_location,
On 2022/09/12 12:53:37, pokoli wrote:
> I think we should keep the to_location from the source move when we have a
> template. 

Done.

[reviewbot, 2022-09-12 14:48:26]

checks OK
URL: https://codereview.tryton.org/411701003

[pokoli, 2022-09-12 16:08:49]

https://codereview.tryton.org/411701003/diff/421691003/modules/sale_supply_dr...
File modules/sale_supply_drop_shipment/stock.py (right):

https://codereview.tryton.org/411701003/diff/421691003/modules/sale_supply_dr...
modules/sale_supply_drop_shipment/stock.py:508: move.from_location =
template.from_location
this should be template.to_location

https://codereview.tryton.org/411701003/diff/421691003/modules/sale_supply_dr...
modules/sale_supply_drop_shipment/stock.py:509: move.to_location =
template.to_location
this should be: self.customer.customer_location

[ced, 2022-09-12 17:13:17]

https://codereview.tryton.org/411701003/diff/421691003/modules/sale_supply_dr...
File modules/sale_supply_drop_shipment/stock.py (right):

https://codereview.tryton.org/411701003/diff/421691003/modules/sale_supply_dr...
modules/sale_supply_drop_shipment/stock.py:508: move.from_location =
template.from_location
On 2022/09/12 16:08:49, pokoli wrote:
> this should be template.to_location

why?

https://codereview.tryton.org/411701003/diff/421691003/modules/sale_supply_dr...
modules/sale_supply_drop_shipment/stock.py:509: move.to_location =
template.to_location
On 2022/09/12 16:08:49, pokoli wrote:
> this should be: self.customer.customer_location

why?

[rietveld-bot_tryton.org, 2022-09-26 22:29:55]

New changeset aa298e7213e0 by Cédric Krier in branch 'default':
Improve moves synchronisation on drop shipment
https://hg.tryton.org/modules/sale_supply_drop_shipment/rev/aa298e7213e0

[rietveld-bot_tryton.org, 2022-09-26 22:30:11]

New changeset c516c469500c by Cédric Krier in branch 'default':
Improve moves synchronisation on drop shipment
https://hg.tryton.org/modules/stock/rev/c516c469500c

[rietveld-bot_tryton.org, 2022-09-26 22:30:27]

New changeset c799a13dbc19 by Cédric Krier in branch 'default':
Improve moves synchronisation on drop shipment
https://hg.tryton.org/tryton-env/rev/c799a13dbc19