DPS Carregado a Partir do XML - Instanciando Propriedades Indevidamente

[marcianobandeira]

Describe the bug
Meu cenário é o seguinte, eu gero o XML com todas as informações e coloco ele em uma fila.
Posteriormente, o serviço que consome a fila realiza o envio do XML gerado anteriormente para o WebService.
O problema, é que ao usar o método DPS.Load(), notei que são criadas instancias de propriedades que não estão presentes no XML. No meu teste em questão, está sendo instanciado "dps.Informacoes.Substituida" com informações em branco, porém, essa propriedade, deveria estar nula.

To Reproduce
Para reproduzir basta criar um DPS, salvar o XML em uma String e depois fazer o DPS.Load em uma nova instancia de DPS

Expected behavior
O comportamento esperado, seria que a proprieade viesse nula já que não existe no XML.

Screenshots

Additional context
Uma possibilidade que poderia ser interessante, é se o método "EnviarAsync" que hoje exige uma instancia da classe DPS, pudesse ter um overload aceitando um XML string.

[marcianobandeira]

Gerei um PR com a proposta de alteração, que permitiria o envio do xml da seguinte forma

[cassianolucas]

Na minha visão parece que fere um pouco da arquitetura a PR #50 , se você analisar, existe um fluxo de assinatura e gravação em disco dentro do EnviarAsync, neste caso deve sempre ter um xml assinado para que o fluxo funcione corretamente.
a criação da instância deve ter relação com o projeto e a classe ou até a classe.

[marcianobandeira]

De fato, essa proposta de correção é minimamente invasiva, não ocasiona quebra no código que ja funciona hoje, apenas injeta a possibilidade de manipular a geração do xml.
Contudo, a resolução do problema de fato, sobre a serialização, envolveria alterar o codigo do projeto OpenAC.Net.Core.

[marcianobandeira]

Aliás, a proposta nao é de correção, mas sim de abrir a possibilidade de injetar o xml diretamente

[marcianobandeira]

Estou fazendo um ajuste aqui pensando na proteção da arquitetura

[marcianobandeira]

@cassianolucas quando puder de uma olhada por favor.
Pensando na arquiteura, o DPS vai continuar gerando o xml normalmente mas vai invocar a rotina que permite manipular o xml.
No meu caso aqui, eu vou simplesmente substituir completamente o xml

[WalissonPires]

Estou passando pela mesma situação. Eu implementei a correção no projeto OpenAC.Net.DFe.Core nesse PR.
Ja no projeto OpenAC.Net.NFSe.Nacional eu crei esse PR.

[WalissonPires]

A correção efetiva em OpenAC.Net.NFSe.Nacional só podera ser feita após o PR no projeto core for aceito e uma nova versão do pacote nuget for criada.

[cassianolucas]

Ainda não consegui parar para analisar, mas deu uma olhada rápida no projeto core, e a principio não era para instânciar o objeto quando o valor é nulo e a propriedade está anotada com atributo NaoObrigatorio

[WalissonPires]

Ainda não consegui parar para analisar, mas deu uma olhada rápida no projeto core, e a principio não era para instânciar o objeto quando o valor é nulo e a propriedade está anotada com atributo NaoObrigatorio

Durante a serialização faz todo sentido verificar o atributo NaoObrigatorio. Mas durante a desserialização eu não acho que deveria levar ele em consideração. Porque mesmo que uma propriedade esteja marcada como Obrigatorio e no xml não tive o dado será criado uma instância com valores "inválidos".

[cassianolucas]

Ainda não consegui parar para analisar, mas deu uma olhada rápida no projeto core, e a principio não era para instânciar o objeto quando o valor é nulo e a propriedade está anotada com atributo NaoObrigatorio

Durante a serialização faz todo sentido verificar o atributo NaoObrigatorio. Mas durante a desserialização eu não acho que deveria levar ele em consideração. Porque mesmo que uma propriedade esteja marcada como Obrigatorio e no xml não tive o dado será criado uma instância com valores "inválidos".

também concordo. acredito que seu PR resolva este fluxo.