-
-
Notifications
You must be signed in to change notification settings - Fork 248
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: add icms difal regulation #2501
base: 14.0
Are you sure you want to change the base?
Conversation
Hi @renatonlima, |
Ainda vou colocar alguns testes e corrigir a mensagem de um dos usererror. |
f59c454
to
324b6c8
Compare
@felipemotter chegou fazer rebase ? |
324b6c8
to
3d03d2e
Compare
Eu vi que na PR #2461 você alterou os valores do DIFAL, o que você acha da proposta do @felipemotter de deixar essa essa definição em um modelo odoo ao invés de ser estatico no código ? |
cc @aug1804 |
8262372
to
bc8a169
Compare
ping @douglascstd |
ad726e8
to
2e58a13
Compare
ping @aug1804 pode fazer a revisão funcional por favor ? |
2e58a13
to
5d05ef3
Compare
5d05ef3
to
582f305
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Poderia configurar a empresa para pegar por padrão a configuração definida nos dados?
Creio que o default no campo possa ser uma boa opção.
This PR has the |
@felipemotter assim que possível faça um rebase por favor. |
7c83c6c
to
a928bcb
Compare
@marcelsavegnago rebase feito, esperando os testes passarem... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Pessoal eu vou revisar esse PR |
a928bcb
to
513a38e
Compare
Conseguiu revisar? |
Grande @renatonlima consegue dar uma atenção nesta PR por favor ? |
@douglascstd se puder revisar esta tbm seria bom. |
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
…features1 PR OCA#2501 adiciona melhoria no difal
513a38e
to
bc2a41d
Compare
Hi @renatonlima, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sera se isso talvez não impacta muitas empresas e talvez poderia ser um modulo extra opcional. @renatonlima algum posicionamento sobre isso? |
@rvalyi esse DIFAL vai ser calculado em certos cenários, e sim.. é possível separar todo o Difal das regras gerais, mas vejo que ele faz parte do calculo de ICMS....
Essa tabela de Difal é uma proposta da Engenere, acho válido entrar para o conjunto de cálculo do ICMS que já existe hoje. E o grande problema hoje é que cada estado pode legislar e arbitrar se é simples ou não, com isso uma tabela em um modelo como proposto, pode ajudar a manter atualizado quando necessário. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Realmente de acordo com o chatGPT, é comum mesmo, até para empresas do simples: Nisso vamos ver com calma a implementação com @renatonlima . Importante tb fazer o que seria uma espécie de "regra de ouro" dos modulos OCA, ou seja botar para correr do modulo l10n_br_fiscal o que tem nada para fazer dentro antes de exigir mais funcionalidades, aquela coisa né... #3290 (assim como fizemos com #3012) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Me parece Ok.. Neste caso, não sei se caberia separar.. pelo menos não me parece pratico neste caso..... masssssss pode ser falha cognitiva minha heheheh.
@rvalyi você tem razão, o DIFAL só vai ser necessário para empresas que operam com Revenda de mercadoria/Venda e que fazem essa venda para outros estados. Nesta proposta desta PR, é possível configurar quando é base simples ou base dupla conforme eventual modificação de legislação futura. Na minha simples opinião, se for possível e viável, adotar essa sequência para tornar possível corrigir o cálculo do DIFAL por configuração pela interface:
Sem essa PR, atualmente, só é possível corrigir alterando o código que faz essa seleção baseado em uma constante: l10n-brazil/l10n_br_fiscal/constants/icms.py Line 120 in 9e9aefb
Em tempo: Sobre as respostas do ChatGPT, eles estão um pouco defasado com relação ao uso da base Dupla, pois parece que não está considerando o Convênio ICMS nº 236, de 27 de dezembro de 2021:
|
@rvalyi @felipemotter Será que podemos avançar com essa PR para resolver a questão do Difal? |
@douglascstd esse projeto era para um antigo cliente e eu ando muito sem tempo por questões pessoais, se alguém quiser pegar isso para seguir em frente, tudo certo. |
Atualmente, usamos constantes no código para definir os tipos de base para o DIFAL, mas isso pode causar algumas complicações.
Depois que o Convênio ICMS 236/2021 (https://www.confaz.fazenda.gov.br/legislacao/convenios/2021/CV236_21) entrou em vigor no ano passado, as bases para vendas em todos os estados deveriam ser simples e únicas. No entanto, no nosso código, ainda temos estados com bases simples e outros com bases duplas. Não tenho certeza se as empresas ainda seguem esse padrão ou se o nosso sistema está desatualizado.
Além disso, há algumas exceções. Conforme explicado pela nossa contadora, em Minas Gerais, mesmo tendo assinado o acordo, em certas inspeções fiscais eles exigem que o cálculo do DIFAL seja feito com base dupla.
Resumindo, não estamos totalmente seguros sobre o que é certo ou errado nesse caso. Por isso, decidimos permitir que cada empresa ajuste a configuração de acordo com suas necessidades e diretrizes.
Fiz uma atualização para definir todas as bases dos estados como simples, seguindo o convênio citado, mas não estabeleci isso como padrão. Acreditamos que cada empresa deve verificar e decidir se essa é a melhor forma de trabalhar para ela.