[14.0][FIX] payment_order: payment_mode na linha de pagamento deve ser related

[antoniospneto]

A forma de pagamento da linha de pagamento não pode ser diferente da ordem de pagamento.
por isso removi a inserção manual e alterei o campo para ser um related.

[rvalyi]

Eu verifiquei no módulo account_payment_order se tinha esse campo mas não, só aqui que usamos, então ta OK para mim fazer essa mudança aqui.

[marcelsavegnago]

/ocabot merge minor

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-2141-by-marcelsavegnago-bump-minor, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 538ee30. Thanks a lot for contributing to OCA. ❤️

[mbcosta]

Olá @netosjb @rvalyi @marcelsavegnago esse campo não estava related devido a uma questão em aberto que pode ser vista no ROADMAP do modulo https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_account_payment_order/readme/ROADMAP.rst

• CNAB de Pagamento, verificar a integração com o PR [12.0] TED / DOC / Transferência / Pagamento de Boletos / Pagamento de Impostos através de cnab240 #972 e a possibilidade de multiplos modos de pagamento na mesma Ordem de Pagamento Feature/cnab multiple payments odoo-brazil/l10n-brazil#112

No PR do odoo-brazil é possível ler sobre o caso de uso que se deseja resolver com uma Ordem de Pagamento com Múltiplos Modos de Pagamento, pelo o que entendo era desnecessário alterar isso já que não causava nenhum problema ou erro e seria melhor deixar da forma anterior para permitir os testes e possível implementação para esse caso de uso.

[antoniospneto]

@mbcosta agora entendi qual era a proposta de ter vários "modos de pagamentos" em uma única ordem de pagamento.
estamos fazendo uma implementação do pagamento que a proposta irá ser um pouco diferente. Nossa proposta irá permitir ter varias formas de pagamentos, utilizando somente um modo de pagamento.

Eu e o Felipe estamos terminando de organizar os commits, em breve vamos abrir uma PR explicando e demostrando a nossa proposta para que todo mundo possa avaliar.

[rvalyi]

tranquilo vamos aguardar essa proposa de vcs @netosjb para avaliar melhor. Pois como nada de pagamentos multiplos tava implementado não tem urgencia, mas valeu @mbcosta por ter lembrado o porque desse design.

[mbcosta]

Legal @netosjb vai ser importante ver a proposta de vcs, é importante notar sobre essa questão que existem dois cenários:

• Um é esse caso aqui falado, para evitar a necessidade de criar várias Ordens de Pagamento porque a Ordem só permite um Modo de Pagto, aí o programa de alguma forma deveria permitir na Ordem vários modos, para assim também no caso do CNAB se criar apenas um arquivo ao invês de vários

• O segundo é quando se tem um Termo de Pagto 30/60/90 onde se quer fazer uma combinação de Modos de Pagto ex.: 30 Dinheiro, 60 No Credito e 90 No Boleto, aí seria um Modo de Pagto multiplo quer dizer um combo, o usuário seleciona UM Modo de Pagto ex.:Combo Dinheiro/Credito/Boleto é o programa na hora de criar os lançamentos coloca em cada account.move.line um Modo de Pagto