operating-unit icon indicating copy to clipboard operation
operating-unit copied to clipboard

[FIX] sale_stock_operating_unit: Reverted change in onchange_team_id

Open baguenth opened this issue 1 month ago • 1 comments

Hey @AaronHForgeFlow,

in the migration of this module to v16.0 I think there was a mistake: https://github.com/OCA/operating-unit/pull/797

  • The obvious issue is, that the function onchange_team_id was just converted from onchange to compute, without adding a for loop. This will break, if we have to compute for multiple sale orders.
  • Anyway, I am not sure if we should have changed it to compute in the first place. Actually the functionality should still be given by using the onchange method. We do the same in sale_operating_unit https://github.com/OCA/operating-unit/blob/16.0/sale_operating_unit/models/sale_order.py#L28.

baguenth avatar Nov 14 '25 15:11 baguenth

Hey @alan196 and @Chionne27,

this is maybe also interesting for you, since you have been involved in https://github.com/OCA/operating-unit/pull/797.

baguenth avatar Nov 18 '25 16:11 baguenth

@AaronHForgeFlow Can we integrate? :)

baguenth avatar Nov 21 '25 08:11 baguenth