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

Atualizando os schemas xml para a nova versão #128

Closed
wants to merge 7 commits into from

Conversation

gersonfs
Copy link
Contributor

Nesse pull request faço a atualização dos schemas xmls para a última versão. Analisando as alterações nos arquivos xsd, acredito que não teremos problemas de quebras de versões antigas. Notem que foi adicionado um campo opcional vAdiant que já está sendo tratado na classe Make, e os campos de número de parcela e data de vencimento são obrigatórios (na classe make eles já eram).

O schema xml baixei daqui: https://dfe-portal.svrs.rs.gov.br/Mdfe/Documentos
05/04/2021
Schema NT 2021.002
Pacote de Schemas que acompanha a NT 2021.002 do MDF-e
Obs: Fiz a correção do tipo string genérico, trocando:
<xs:pattern value="[!-ÿ]{1}[ -ÿ]{0,}[!-ÿ]{1}|[!-ÿ]{1}"/>
por
<xs:pattern value="[!-ÿ]{1}[ -ÿ]*[!-ÿ]{1}|[!-ÿ]{1}"/>

Foram adicionados testes para comprovar o funcionamento. Tomei a liberdade de criar classes Fake para evitar a comunicação com webservices do governo. Também adicionei um certificado fake para conseguir assinar os documentos.

Tomei a liberdade de criar um arquivo para execução automática dos testes no GitHub (.github/workflows/ci.yml) que vão rodar os testes a cada alteração ou PR aqui no projeto futuramente. Podemos usar depois o recurso "matrix" para executar testes em mais versões do PHP. Só executei com a 7.4

@gersonfs
Copy link
Contributor Author

gersonfs commented Jul 8, 2021

Vou fazer um novo PR somente com a atualização do schema xml.

@gersonfs gersonfs closed this Jul 8, 2021
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.

None yet

1 participant