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

DiegoParadeda · 2024-10-04T20:47:27Z

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 · 2024-10-04T20:47:32Z

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

mileo · 2024-10-07T12:07:08Z

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

marcelsavegnago · 2024-10-07T14:18:52Z

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 · 2024-10-07T15:44:04Z

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 · 2024-10-15T19:05:33Z

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 · 2024-10-22T16:59:21Z

@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 · 2024-10-23T17:00:01Z

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 · 2024-11-11T11:29:38Z

l10n_br_fiscal/tests/test_fiscal_document_generic.py

+
+            # ICMS
+            if line.product_id.icms_origin not in ICMS_ORIGIN_TAX_IMPORTED:
+                self.assertEqual(


Pode fazer um assert da taxa do ICMS ST?

l10n_br_fiscal/tests/test_fiscal_document_generic.py

Tiago370 · 2024-11-14T13:16:10Z

depend de #3422

DiegoParadeda · 2024-11-27T12:54:18Z

l10n_br_fiscal/tests/test_fiscal_document_generic.py

+
+            # ICMS
+            if line.product_id.icms_origin not in ICMS_ORIGIN_TAX_IMPORTED:
+                line.write(


@Tiago370 acho que esse write não pode estar aqui. A ideia é testar o imposto automático. Se você escrever o imposto no teste a validação perde o sentido

DiegoParadeda · 2024-11-27T12:54:36Z

l10n_br_fiscal/tests/test_fiscal_document_generic.py

+
+            # ICMS
+            if line.product_id.icms_origin not in ICMS_ORIGIN_TAX_IMPORTED:
+                line.write(


@Tiago370 acho que esse write não pode estar aqui. A ideia é testar o imposto automático. Se você escrever o imposto no teste a validação perde o sentido

DiegoParadeda · 2024-11-27T12:56:36Z

l10n_br_fiscal/tests/test_fiscal_document_generic.py

+            line.with_company(empresa_lucro_presumido.id)._onchange_fiscal_taxes()
+
+            self.assertTrue("Revenda" in line.fiscal_operation_line_id.name)
+            if "Revenda" in line.fiscal_operation_line_id.name:


O assert da cfop_id precisa acontecer independente se "Revenda" está no nome da operation line. Essa linha é desnecessária

DiegoParadeda · 2024-11-27T12:57:48Z

l10n_br_fiscal/tests/test_fiscal_document_generic.py

+            )._onchange_fiscal_operation_line_id()
+            line.with_company(empresa_lucro_presumido.id)._onchange_fiscal_taxes()
+
+            self.assertTrue("Revenda" in line.fiscal_operation_line_id.name)


Essa linha também é desnecessária já que esse assert já acontece algumas linhas abaixo em: "line.cfop_id.code, 6404"

Tiago370 · 2024-11-27T18:51:59Z

Corrigido

mileo · 2024-11-27T18:55:34Z

@DiegoParadeda @Tiago370 esta pronto para revisão?

Tiago370 · 2024-11-27T23:05:14Z

@mileo da minha parte sim.

DiegoParadeda · 2024-11-29T01:25:31Z

l10n_br_fiscal/tests/test_fiscal_document_generic.py

+            )
+
+            # ICMS
+            if line.product_id.icms_origin not in ICMS_ORIGIN_TAX_IMPORTED:


if line.product_id.icms_origin not in ICMS_ORIGIN_TAX_IMPORTED:

Esse if é desnecessário

DiegoParadeda · 2024-11-29T01:26:46Z

l10n_br_fiscal/tests/test_fiscal_document_generic.py

+            )
+
+            # ICMS
+            if line.product_id.icms_origin not in ICMS_ORIGIN_TAX_IMPORTED:


Esse if é desnecessário

DiegoParadeda · 2024-11-29T01:28:34Z

l10n_br_fiscal/tests/test_fiscal_document_generic.py

+                    " for Venda de Contribuinte p/ Fora do Estado.",
+                )
+                self.assertEqual(
+                    line.icmsst_tax_id.name,


Utilizar registro.name não é uma prática muito legal @Tiago370. Nesse caso você consegue pegar o XML ID do registro e garantir que o teste não vai quebrar se alguém trocar o nome do imposto

Tiago370 · 2024-11-29T09:56:28Z

@DiegoParadeda corrigido.

rvalyi requested a review from renatonlima October 7, 2024 15:47

marcelsavegnago requested review from antoniospneto, douglascstd and WesleyOliveira98 October 15, 2024 19:20

WesleyOliveira98 approved these changes Oct 22, 2024

View reviewed changes

mileo approved these changes Oct 23, 2024

View reviewed changes

OCA-git-bot added approved ready to merge labels Oct 23, 2024

mileo and others added 4 commits November 1, 2024 17:07

[IMP] l10n_br_fiscal: adds icms st demo data

6217161

[IMP] l10n_br_pos: fix POS config test.

3b8077b

[IMP] l10n_br_fiscal: use operation line/product tags

6bd2228

[IMP] l10n_br_fiscal: add product.tag view

267ba5a

Tiago370 force-pushed the product-tag branch from b0a68a1 to e93cbbb Compare November 8, 2024 21:10

mileo marked this pull request as draft November 8, 2024 22:27

Tiago370 force-pushed the product-tag branch 2 times, most recently from b9e08f4 to fc4e040 Compare November 10, 2024 21:52

mileo reviewed Nov 11, 2024

View reviewed changes

DiegoParadeda commented Nov 11, 2024

View reviewed changes

l10n_br_fiscal/tests/test_fiscal_document_generic.py Outdated Show resolved Hide resolved

l10n_br_fiscal/tests/test_fiscal_document_generic.py Outdated Show resolved Hide resolved

l10n_br_fiscal/tests/test_fiscal_document_generic.py Outdated Show resolved Hide resolved

Tiago370 force-pushed the product-tag branch 2 times, most recently from cb467f8 to 25b5a8d Compare November 14, 2024 12:58

Tiago370 force-pushed the product-tag branch 2 times, most recently from cb05c65 to 4f17d7e Compare November 27, 2024 12:21

DiegoParadeda commented Nov 27, 2024

View reviewed changes

Tiago370 force-pushed the product-tag branch from 4f17d7e to e5f6c93 Compare November 27, 2024 13:42

DiegoParadeda commented Nov 29, 2024

View reviewed changes

[IMP] l10n_br_fiscal: add product.tag test

c33ae6e

Tiago370 force-pushed the product-tag branch from e5f6c93 to c33ae6e Compare November 29, 2024 09:54

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

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

DiegoParadeda commented Oct 4, 2024

OCA-git-bot commented Oct 4, 2024

mileo commented Oct 7, 2024 •

edited

Loading

marcelsavegnago commented Oct 7, 2024 •

edited

Loading

mileo commented Oct 7, 2024

DiegoParadeda commented Oct 15, 2024

WesleyOliveira98 left a comment

rvalyi commented Oct 22, 2024

mileo left a comment

OCA-git-bot commented Oct 23, 2024

mileo Nov 11, 2024

Tiago370 commented Nov 14, 2024

DiegoParadeda Nov 27, 2024

DiegoParadeda Nov 27, 2024

DiegoParadeda Nov 27, 2024

DiegoParadeda Nov 27, 2024

Tiago370 commented Nov 27, 2024

mileo commented Nov 27, 2024

Tiago370 commented Nov 27, 2024

DiegoParadeda Nov 29, 2024

DiegoParadeda Nov 29, 2024

DiegoParadeda Nov 29, 2024

Tiago370 commented Nov 29, 2024

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

Are you sure you want to change the base?

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

Conversation

DiegoParadeda commented Oct 4, 2024

Objetivo

OCA-git-bot commented Oct 4, 2024

mileo commented Oct 7, 2024 • edited Loading

marcelsavegnago commented Oct 7, 2024 • edited Loading

mileo commented Oct 7, 2024

DiegoParadeda commented Oct 15, 2024

WesleyOliveira98 left a comment

Choose a reason for hiding this comment

rvalyi commented Oct 22, 2024

mileo left a comment

Choose a reason for hiding this comment

OCA-git-bot commented Oct 23, 2024

Choose a reason for hiding this comment

Tiago370 commented Nov 14, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Tiago370 commented Nov 27, 2024

mileo commented Nov 27, 2024

Tiago370 commented Nov 27, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Tiago370 commented Nov 29, 2024

mileo commented Oct 7, 2024 •

edited

Loading

marcelsavegnago commented Oct 7, 2024 •

edited

Loading