[14][IMP] shopfloor: Ensure destination location on moves for allow move create

[mt-software-de]

Ensure that the destination gets also set on the move. When a move is created within the single pack or single product transfer scenario, the move ends up with the default destination location of the operation type.
To ensure that the destination location of the move gets also set, we need to set it also on the move not only on the move.line.

cc @jbaudoux @i-vyshnevska

[OCA-git-bot]

Hi @mmequignon, @simahawk, @sebalix, @guewen,
some modules you are maintaining are being modified, check this out!

[jbaudoux]

What's the purpose of sync_src_location ?

[mt-software-de]

What's the purpose of sync_src_location ?
I updated the description.

[jbaudoux]

Without syncing the src location, the moves will end up with the warehouse view location as src location and with actual scanned location as dest location.

@mt-software-de Isn't it something that should be tackled at move creation? When you scan the source location and product/package to move, the move should be created on that location and not on the picking type source location. Same if you plan the moves, should be planned on the right source location.

[mt-software-de]

Without syncing the src location, the moves will end up with the warehouse view location as src location and with actual scanned location as dest location.

@mt-software-de Isn't it something that should be tackled at move creation? When you scan the source location and product/package to move, the move should be created on that location and not on the picking type source location. Same if you plan the moves, should be planned on the right source location.

I checked it again the source location was never be the problem. I was on allow move create the dest location on the move which was missing. And only for the single product transfer because there are no package level.

[jbaudoux]

Looking better. Just a few left overs

[jbaudoux]

cc @rousseldenis