[14.0][FIX+IMP] l10n_br_sale_stock: Inform Fields that should not be used from Sale 'prepare' methods, tests #3619
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Inform Fields that should not be used from Sale 'prepare' methods, depends OCA/account-invoicing#1906
Esse PR deve resolver o problema reportado aqui #3616 , que é o programa deve usar os valores dos campos Número do Documento e Série informados no stock.picking e não no sale.order, acredito que a melhor solução para resolver isso é criando dois método que retornam os Campos que não usados dos métodos 'prepare' do Sale e como isso pode ser tanto um problema da Localização também pode acabar sendo um problema para outras Localizações ou mesmo para módulos customizados será melhor resolver a nível do sale_stock_picking_invocing ao invés de resolver apenas na Localização Brasileira, então agora basta informar qualquer campo que venha do Sale mas que não deverá ser usado para criar a Fatura herdando o método _get_fields_not_used_from_sale para os campos do sale.order ou o _get_fields_not_used_from_sale_line para os campos do sale.order.line.
Inclui um teste para esse caso, assim evitamos regressões, mas como o PR depende do OCA/account-invoicing#1906 o CI aqui vai ficar com erro até ocorrer o merge.
cc @OCA/local-brazil-maintainers @DiegoParadeda