SilverCard / DanfeSharp

Biblioteca C# para geração de DANFE em PDF.
MIT License
97 stars 81 forks source link

"O tpEmis {ide.tpEmis} não é suportado." #17

Closed willianflopes closed 5 years ago

willianflopes commented 5 years ago

O método CreateFromXml(ProcNFe procNfe) da classe DanfeViewModelCreators contém a linha:

if (FormasEmissaoSuportadas.Contains(model.TipoEmissao))
{
throw new NotSupportedException($"O tpEmis {ide.tpEmis} não é suportado.");
}

Quando a emissão é normal e está dentro do tipo de emissão suportada ele lança a exceção acima. A linha do if não teria de ser a abaixo?

if (!FormasEmissaoSuportadas.Contains(model.TipoEmissao)) //observe o ! negando o if

SilverCard commented 5 years ago

Teria.

Realmente é um erro de digitação, a intenção é verificar os formatos suportados.

Vou fazer um novo commit em breve, Obrigado :)


From: willianflopes notifications@github.com Sent: Tuesday, December 18, 2018 3:18:05 PM To: SilverCard/DanfeSharp Cc: Subscribed Subject: [SilverCard/DanfeSharp] "O tpEmis {ide.tpEmis} não é suportado." (#17)

O método CreateFromXml(ProcNFe procNfe) da classe DanfeViewModelCreators contém a linha:

if (FormasEmissaoSuportadas.Contains(model.TipoEmissao))

{

throw new NotSupportedException($"O tpEmis {ide.tpEmis} não é suportado.");

}

Quando a emissão é normal e está dentro do tipo de emissão suportada ele lança a exceção acima. A linha do if não teria de ser a abaixo?

if (!FormasEmissaoSuportadas.Contains(model.TipoEmissao)) //observe o ! negando o if

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/SilverCard/DanfeSharp/issues/17, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJsmleTJzz4lhTHDMrJIWq2yQklzhBw_ks5u6SNNgaJpZM4ZYvDa.

SilverCard commented 5 years ago

Corrigido em 01e2117b38193fc8f97599c5f0e08cb97bd0d9a9, mais um vez obrigado.

willianflopes commented 5 years ago

Eu que agradeço por você fornecer o código! Muito obrigado pela disposição! Excelente trabalho.