(BREAKING CHANGE) Refatora atributos cClassTrib e cClassTribIS para string #131
Conversation
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
Adiciona campos privados `_cClassTrib` e `_cClassTribIS` para encapsulamento e métodos auxiliares `SetcClassTrib` e `SetcClassTribIS` para definir os valores a partir de inteiros, formatando-os como strings de 6 dígitos (`D6`)
Collaborator
|
Como conversado no grupo dos mantenedores, aceito este PR com dor no coração 😢 Motivo: Sefaz aceitou XMLs com 000000 que está fora dos ClassTrib oficiais removendo completamente a possibilidade de ser um ENUM. Como a formatação do ClassTrib é um D6 foi discutido a possibilidade de usar INT e formatar como um proxy. Entretanto, como nem tudo são flores a chance da Sefaz enfiar uma letra nesse ClassTrib a médio prazo é enorme, então optou-se por string. Peço desculpas em nome dos colaboradores, mas entendemos que quebrar isso agora é melhor do que quebrar depois. Isso vai afetar também outras ferramentas como a impressão de ESCPOS, justamente por isso estou aceitando em um sábado o PR para já corrigir tudo para segunda-feira. |
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.
Adiciona campos privados
_cClassTribe_cClassTribISpara encapsulamento e métodos auxiliaresSetcClassTribeSetcClassTribISpara definir os valores a partir de inteiros, formatando-os como strings de 6 dígitos (D6)Motivo: Foi definido por conta de problemas com o ENUM em alguns casos. Por enquanto a classe do ENUM foi mantido no código fonte para não quebrar usuários mais a fundo.