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][WIP] l10n_br_fiscal, nfe: Adição da tag cBenef no XML #3561

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

Conversation

corredato
Copy link
Contributor

Esta PR adiciona a tag cBenef no XML, considerando que, a partir de 01/02/2024, será obrigatório o preenchimento dessa tag no estado de Santa Catarina. Na tag de informações adicionais do produto, deverão ser informados os códigos de benefício fiscal, caso existam mais de um.

Algumas melhorias podem ser feitas futuramente:

  1. Controle da visibilidade do campo do código de benefício fiscal se baseando no state_id da empresa e do cst/csosn informado no imposto
  2. Introduzir os códigos de benefício fiscal de cada estado que preenche o cBenef (DF, GO, PR, RJ, RS e SC)

@OCA-git-bot
Copy link
Contributor

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

@rvalyi
Copy link
Member

rvalyi commented Dec 19, 2024

@corredato eu vejo que vc adiciona esses modelos ao nivel do módulo l10n_br_fiscal. Porem temos mixin geridos para a tag cBenef no módulo l10n_br_nfe_spec que poderia ser extendido no módulo l10n_br_nfe. Sera que não seria melhor implementar isso apenas no módulo l10n_br_nfe? Vai ser usado fora do contexto da NFe esse modelo? cc @renatonlima @mileo @marcelsavegnago @antoniospneto

@corredato
Copy link
Contributor Author

@corredato eu vejo que vc adiciona esses modelos ao nivel do módulo l10n_br_fiscal. Porem temos mixin geridos para a tag cBenef no módulo l10n_br_nfe_spec que poderia ser extendido no módulo l10n_br_nfe. Sera que não seria melhor implementar isso apenas no módulo l10n_br_nfe? Vai ser usado fora do contexto da NFe esse modelo? cc @renatonlima @mileo @marcelsavegnago @antoniospneto

Acho interessante manter a implementação no fiscal por se tratar de uma configuração base para empresas de alguns estados, assim como a configuração de impostos e alíquotas nas linhas das operações, não parece mais intuitivo manter no fiscal?

@rvalyi
Copy link
Member

rvalyi commented Dec 19, 2024

eu diria que o unico critério é se vai usar em outros documentos fiscais sem ser NFe. Se não for o caso tem que ir no l10n_br_nfe. Em termo de ergonomia pode ter exactemente a mesma usabilidade. Mas uma coisa super importante é que o módulo l10n_br_fiscal ja esta MUITO acima do peso, já é disparado o maior módulo dos 3000 módulos da OCA então para enfiar qualquer coisa a mais dentro sem ser outro módulo tem que ter justificativa muito boa...

A tendência é mais que a gente ainda ta na luta para fatiar ele mais do que para aceitar mais funcionalidades nele...

@corredato
Copy link
Contributor Author

eu diria que o unico critério é se vai usar em outros documentos fiscais sem ser NFe. Se não for o caso tem que ir no l10n_br_nfe. Em termo de ergonomia pode ter exactemente a mesma usabilidade. Mas uma coisa super importante é que o módulo l10n_br_fiscal ja esta MUITO acima do peso, já é disparado o maior módulo dos 3000 módulos da OCA então para enfiar qualquer coisa a mais dentro sem ser outro módulo tem que ter justificativa muito boa...

Entendo... A tag do cBenef deve ser preenchida também na NFCe, é o suficiente para manter no fiscal?

@rvalyi
Copy link
Member

rvalyi commented Dec 19, 2024

eu diria que o unico critério é se vai usar em outros documentos fiscais sem ser NFe. Se não for o caso tem que ir no l10n_br_nfe. Em termo de ergonomia pode ter exactemente a mesma usabilidade. Mas uma coisa super importante é que o módulo l10n_br_fiscal ja esta MUITO acima do peso, já é disparado o maior módulo dos 3000 módulos da OCA então para enfiar qualquer coisa a mais dentro sem ser outro módulo tem que ter justificativa muito boa...

Entendo... A tag do cBenef deve ser preenchida também na NFCe, é o suficiente para manter no fiscal?

não! pois o esquema xsd da NFCe é o mesmo da NFe, eu diria que é mais um motivo de botar no módulo l10n_br_nfe (herdando o mixin nfe.40.cbenef que ja existe e vem do módulo l10n_br_nfe_spec)

@antoniospneto
Copy link
Contributor

antoniospneto commented Dec 19, 2024

02/2024 já passou, o @renatonlima ja tinha feito uma implementação para isso, o cadastro do benefício fiscal fica junto a regulamentação do ICMS, isso não atende o caso de vocês?

@rvalyi
Copy link
Member

rvalyi commented Dec 19, 2024

02/2024 já passou, o @renatonlima ja tinha feito uma implementação para isso, o cadastro do benefício fiscal fica junto a regulamentação do ICMS, isso não atende o caso de vocês?

ver #2461 e #3354

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.

4 participants