[14.0][IMP] l10n_br_fiscal: add operation line/product tags match

[DiegoParadeda]

Objetivo

Este PR introduz o modelo product.tag para aprimorar a capacidade de combinação automática das linhas de operação fiscal por produto.

[OCA-git-bot]

Hi @renatonlima,
some modules you are maintaining are being modified, check this out!

[mileo]

@DiegoParadeda pode implementar alguns testes e uns dados de demo?

[marcelsavegnago]

De fato sinto falta de algo mais especifico para definir como filtro de operação ou linha de operação fiscal, mas será que precisamos de mais um modelo ?

[mileo]

De fato sinto falta de algo mais especifico para definir como filtro de operação ou linha de operação fiscal, mas será que precisamos de mais um modelo ?

Poderia sugerir alguma outra opção? Das que pensamos e não fica legal:

• Botar mais boleanos / selections
• Alterar a regra de fiscal definition / fiscal op line;
• Usar o fiscal.tax.group, ou outro modelo do fiscal, mas acho que não é pra isso;

Essas tags dão mais versatilidade para casos complexos, onde existem mais de uma CFOP para a mesma operação, por exemplo a exportação com as CFOPs:

• 7102 - Venda de mercadoria adquirida ou recebida de terceiros;
• 7501 - Exportação de mercadorias recebidas com fim específico de exportação;

A ideia é que o usuário nunca tenha que selecionar a linha da operação manualmente.

[DiegoParadeda]

De fato sinto falta de algo mais especifico para definir como filtro de operação ou linha de operação fiscal, mas será que precisamos de mais um modelo ?

@marcelsavegnago tentei usar a ideia dos marcadores (many2many) por simplicidade e praticidade apenas. Me pareceu ser a forma mais tranquila de cobrir vários casos de uso. Só queria ressaltar que o many2many na minha visão é importante.

Agora sobre o modelo em si, acho que daria pra herdar algum product.tags e colocar um campo nele exemplo "is_fiscal_tag" e usar esse modelo já existente com um domínio. Se acharem que fica melhor desse jeito posso alterar.

[WesleyOliveira98]

LGTM

[rvalyi]

@renatonlima pode revisar?

[mileo]

Pra mim isso já esta legal, mas o ideal é colocar testes para garantir que continua funcionando no futuro.

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[mileo]

Pode fazer um assert da taxa do ICMS ST?

[Tiago370]

depend de #3422

[Tiago370]

Corrigido

[mileo]

@DiegoParadeda @Tiago370 esta pronto para revisão?

[Tiago370]

@mileo da minha parte sim.

[Tiago370]

@DiegoParadeda corrigido.