Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0][IMP] l10n_br_fiscal: allow operation tax base override #3452

Open
wants to merge 1 commit into
base: 14.0
Choose a base branch
from

Conversation

DiegoParadeda
Copy link
Contributor

Objetivo

Este PR adiciona a possibilidade de modificar o comportamento da composição da base de cálculo do imposto, definida pelo campo base_with_additional_values no modelo l10n_br_fiscal.tax.group, permitindo que essa definição seja sobrescrita por meio da Fiscal Operation Line.

O principal objetivo é configurar Linhas de Operação específicas onde os valores de freight_value, insurance_value e other_value sejam ou não incluídos na base de cálculo dos impostos, sem a necessidade de criar novos grupos de imposto ou múltiplas definições de impostos.

Exemplo

A operação de Importação para comercialização é um exemplo dessa situação. Nessa operação, os impostos são calculados de forma diferente do padrão nacional, fazendo com que o IPI, COFINS e PIS não incluam certos custos adicionais da compra.

No tax.group:
image

Na operation.line:
image

@OCA-git-bot
Copy link
Contributor

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

@DiegoParadeda DiegoParadeda force-pushed the feature/other-values-tax-base-operation branch from 6254e2a to 612ae89 Compare October 18, 2024 20:44
@marcelsavegnago
Copy link
Member

marcelsavegnago commented Oct 20, 2024

Consegue aumentar a cobertura de teste ?. Inclusive no l10n_br_account talvez varios casos de uso sendo testados afim de garantir que não tenhamos nenhuma eventual regressão e que os lançamentos contábeis estão corretos. Sendo assim, acha valido considerar incluir este caso nos testes do l10n_br_account?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants