[14.0][IMP] l10n_br_purchase_stock: Tornando a Fatura criada pela Ordem de Seleção/stock.picking semelhante a do Pedido de Compra/purchase.order, evitando "glue modules"

[mbcosta]

Improve create invoice from purchase stock.

Ao criar uma Fatura/account.move a partir de uma Ordem de Seleção/stock.picking que tenha um Pedido de Compra/purchase.order relacionado o modulo agora chama os métodos que criam os Dicionários de Dados a serem usados para criar a Fatura a partir do Pedido de Compra, tanto do purchase.order quanto o purchase.order.line, assim a Fatura criada a partir do stock.picking deve ser o mais semelhante possível com a criada a partir do purchase.order, alguns campos não são copiados por serem informados na Ordem de Seleção, mais detalhes no código.

Com isso qualquer módulo que implementa algum novo campo na Fatura criada a partir do Pedido de Compra também deverá ser carregado quando for criado a partir da Ordem de Seleção, com isso evitando a necessidade de "glue modules" (pequenos módulos criados apenas para evitar dependências indiretas).

As Linhas de Seção e Notas que existem no Pedido de Compras também passaram a ser criadas na Fatura a partir da Ordem de Seleção, mantendo a mesma Sequencia das Linhas mesmo quando é criada uma Fatura Agrupada

Foram incluídas Linhas de de Seção e Notas nos Dados de Demonstração e teste para verificar se a Fatura foi criada com essas linhas.

Essas alterações também estão sendo feitas no modulo l10n_br_sale_stock no PR de extração #2955 para o sale_stock_picking_invoicing OCA/account-invoicing#1025 onde a alteração nesse caso está sendo feita.

Se necessário posso ver de separar o PR para facilitar a revisão.

cc @rvalyi @renatonlima @marcelsavegnago @mileo

[rvalyi]

show @mbcosta ! Porem, aproveitando ja para comentar algum detalhe tecnico: vc costuma botar mensagem de commit grandes e isso deve ser evitado: na primeira linha tem que botar uma mensagem resumida de menos de 50 chars para facilitar para quem mantem o projeto e que tem que lidar com muitos commits. Veja o guideline da OCA: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message

Isso eh padrao do proprio git, nem da OCA. Sei que pode ser dificil com 50 chars e ainda botando o nome do modulo, mas tem que se esforçar nisso. Botar a mensagem em ingles a principio ajuda a ficar menor. Nas outras linhas eh possivel detalhar mais.

cc outras pessoas que fazem isso tb: @renatonlima @DiegoParadeda @mileo

[mbcosta]

@rvalyi corrigi as mensagens dos commits para o tamanho 80, valeu por avisar sobre essa questão, vou passar a deixar as mensagens no tamanho esperado

[rvalyi]

@rvalyi corrigi as mensagens dos commits para o tamanho 80, valeu por avisar sobre essa questão, vou passar a deixar as mensagens no tamanho esperado

Na verdade eu errei em escrever 80 inicialmente. O limite eh 50 chars para mensagem principal/primeira linha mesmo... Ai por examplo: qdo vc olha as lista dos commits no Github ou em qualquer ferramenta/log: ter mensagens curtas ajuda a saber do que se trata, onde dar rebase ou procurar algo errado. Qdo nao te resumido, tem que navegar em cada item para saber, ai perde tempo...

[mbcosta]

certo @rvalyi alterado para 50

[marcelsavegnago]

LGTM

[rvalyi]

LGTM, limpeza muito boa @mbcosta
La na frente se a gente quiser ligar com a lógica da nota importada pelo XML, isso deve simplificar tb.

[rvalyi]

/ocabot merge major

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-3299-by-rvalyi-bump-major, awaiting test results.

[OCA-git-bot]

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