openboleto / OpenCnabPHP

Biblioteca multibanco para geração e leitura de arquivos Cnab240 e Cna400 de remessa e retorno
MIT License
191 stars 118 forks source link

Geracao de remessa para o sicredi #219

Closed fabriciolangermt closed 2 years ago

fabriciolangermt commented 2 years ago

Ajustes nos campos ($meta) para os registros 1 e 2 Correcao do calculo do digito verificar para o campo nosso numero do registro 2 que estava diferente do registro 1 Seria interessante padronizar essa funcao, e unir as duas? visto que tem a mesma funcao (modulo11) nos dois arquivos?

Rctnet commented 2 years ago

Quanto a usar uma só, acredito que seja o certo, não me lembro agora do porque de ter dicado com duas classes iguais em arquivos diferentes, acredito que o ideal seja criar uma classe estática Utils com os métodos estáticos , ai nos registros chamaríamos ela por Utils::modulo11($numero) por exemplo, se tiver interesse em fazer, tem meu apoio

fabriciolangermt commented 2 years ago

Acredito que seria o melhor tambem, vou implementar e gero um pull request