Skip to content

Adicionando novos eventos RTC (Reforma Tributária) ao modulo de NFe - #189

Merged
danilobreda merged 6 commits into
Hercules-NET:masterfrom
Threads-creator:ajustes_reforma_tributaria_credito_debito
Sep 13, 2026
Merged

danilobreda merged 6 commits into
Hercules-NET:masterfrom
Threads-creator:ajustes_reforma_tributaria_credito_debito

Conversation

@Threads-creator

Copy link
Copy Markdown

Mudanças

  • Adicionando os novos eventos da reforma tributária seguindo a NT_2025.002_v1.51_RTC. Implementado os topicos de 8.3 a 8.17 do documento;
  • Atualizando o projeto de testes de NFe e os schemas xsd;
  • Ajustando o xsd de tiposBasicos_v_1.0.3 (é uma versão modificada por mim para conter campos necessários aos eventos que a SEFAZ não disponibilizou originalmente no site dela)

@Threads-creator

Copy link
Copy Markdown
Author

Testei em ambiente de homologação apontando para o ES e funcionou. Quanto aos outros Estados e ambiente de produção, não sei se esta funcionando.

@danilobreda

Copy link
Copy Markdown
Collaborator

Agradeço a contribuição, aguarde analise pra aprovação.

@danilobreda

danilobreda commented Sep 13, 2026 •

Copy link
Copy Markdown
Collaborator

O que precisa corrigir:

  1. Regressão e quebra de API em gControleEstoque (usado pelo evento 112130, que já existe no master):
    • vIBS/vCBS passaram de decimal para decimal?, com ShouldSerialize baseado no campo interno.
    • Quem não atribuía valor antes recebia 0.00 no XML. Agora o elemento some e o XSD rejeita o evento.
    • O getter usa GetValueOrDefault() e nunca devolve null, o que confunde quem usa a classe.
    • Código como decimal x = g.vIBS deixa de compilar.
  2. Faltam 211120 e 112150: os XSDs estão no repositório, mas não há enum nem método.
  3. RemoverAcentos=true quebra 12 dos 14 eventos: o descEvento perde os acentos e o XSD rejeita o valor. Testado. O padrão é false, mas vale tratar.
  4. Enum ServicoNFe: os 14 valores novos entraram no meio do enum, então os membros seguintes mudam de número. Por exemplo, NFeAutorizacao passa de 17 para 31. Isso quebra compatibilidade binária para quem guarda o valor inteiro. O PR anterior do 112130 já tinha feito o mesmo.
  5. Menores:
    • resEvento.CNPJ passou de ulong para string. Tem justificativa (CNPJ alfanumérico), mas quebra API.
    • Cerca de 1.000 linhas repetem o mesmo template.
    • Todas as UFs apontam para a SVRS. Segue o padrão do 112130, mas o autor só testou homologação no ES.
    • Adiciona BOM em 8 arquivos .cs.

@danilobreda

Copy link
Copy Markdown
Collaborator

vou aprovar e fazer as melhorias eu mesmo em outro pullrequest.

@danilobreda
danilobreda self-requested a review September 13, 2026 20:23
danilobreda added a commit that referenced this pull request Sep 13, 2026
Adicionando novos eventos RTC (Reforma Tributária) ao modulo de NFe - #189
@danilobreda
danilobreda merged commit 210d22a into Hercules-NET:master Sep 13, 2026
@danilobreda

Copy link
Copy Markdown
Collaborator

@Threads-creator obrigado pela contribuição!

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.

2 participants