Repository navigation
fix: Adiciona eventos 211120, 112150 e 110150 ao NFeTipoEvento - #209
Merged
adrbarros merged 1 commit intoNov 9, 2026
Merged
Conversation
Os XSDs desses eventos (e211120, e112150 e e110150) já estão em NFe.AppTeste/Schemas, mas o enum não tinha os valores, então um retConsSitNFe com algum deles não desserializava. Os descEvento de 211120 e 112150 só são aceitos acentuados pelo XSD, por isso entram em DescEventoSomenteComAcento.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No review do #189, o @danilobreda apontou: "Faltam 211120 e 112150: os XSDs estão no repositório, mas não há enum nem método" (comentário). O mesmo vale para o 110150, que também tem XSD em
NFe.AppTeste/Schemase não estava no enum.Sem esses valores no
NFeTipoEvento, umretConsSitNFe(ou distribuição DF-e) que traga um desses eventos quebra na desserialização:Mudanças
NFeTipoEvento: adicionadosTeNfeAtorInteressadoNFe(110150),TeNfeAtualizacaoDataPrevisaoEntregaNFe(112150) eTeNfeDestinacaoConsumoPessoalNFe(211120). ODescriptionde cada um é exatamente odescEventoaceito pelo XSD:Extevento.DescEventoSomenteComAcento: incluídos 211120 e 112150, porque os XSDs deles só aceitam a descrição acentuada (mesmo tratamento dos outros eventos da reforma). O 110150 não precisa, a descrição não tem acento.NFe.Utils.Testes/Consulta/RetConsSitNFeEventosNovosTestes.cs, reaproveitando oMontarRetConsSitNFedo teste de CT-e: desserializa umretConsSitNFecom cadatpEventoe confere queRemoverAcentosPreservandoDescEventomantém odescEventoaceito pelo XSD.Ficou de fora: métodos de envio desses eventos no
ServicosNFe(aqui é só o enum) e o 110300, que já está no #208.Testes
'112150' is not a valid value for NFeTipoEvento, e o mesmo para 211120 e 110150) e passam com a correção (net10.0 e net48).NFe.Utils.Testes: as 6 falhas restantes por framework (URLs SVRS de autorização e QR Code) já existem no master e não têm relação com esta mudança.DFe.Testes: 257/257.