Nfe cleanup

[rvalyi]

Isso eh relacionado as mudancas da nfelib aqui: https://github.com/akretion/nfelib
( historico das mudancas nas fontes aqui https://github.com/akretion/nfelib/commits/master ).

postando aqui de novo a explicacao que eu tinha postado no Telegram e Hangout sobre essa limpeza da nfelib:

entao, eu atualizei o nfelib da forma que eu acho correcto para manter e com um minimo de testes e com todos os esquemas usados hoje no erpbrasil.edocs https://github.com/akretion/nfelib
a coisa que eu considero fundamental para essas libs que usam o generateDS, eh que seja atualizado a cada commit/add de esquema esse tipo de receita do bolo de como refaz de novo do 0:
https://github.com/akretion/nfelib#gerir-a-lib-novamente--processo-de-release
isso eh fundamental se atualizar um esquema ou o generateDS, para poder atualizar sem criar regressoes com alguma gambiarra que nao foi documentada
um outro ponto fondamental eh separar a receita do bolo, testes e extencoes da parte da parte do codigo gerido
entao o codigo gerido, que fica navigavel para ler o codigo ou instalar com pip install do github ou pro release, fica na branch padrao do codigo gerido, no caso:
master_gen_v4_00
MAS
se algum precisar revisar uma contribuicao ou uma alteracao (imagina vc atualiza a prod depois de 6 meses o que sera que mudou?), ai as coisas ficam muito claro na branch master QUE NAO TEM O CODIGO GERIDO:
https://github.com/akretion/nfelib/commits/master
ai vc ve cada commit e pode olhar o diff fica perfeitamente legivel
outro ponto fondamental: ficar DRY e leve
considerando os xsd do Distrito Federal e MDe, a lib de vcs tem todo isso:
https://github.com/akretion/nfelib/tree/master_gen_v4_00/nfelib/v4_00

a minha tem TODAS classes usadas pelo erpbrasil.edocs e so tem 30% dos arquivos https://github.com/akretion/nfelib/tree/master_gen_v4_00/nfelib/v4_00
no caso do Mde talvez precisaria replicar um gambiarra que fizeram, mas teria que ver. Com o generateDS atualizado pode ser que varias gambiarras nao sejam mais necessarias
mas a ideia e nao atrasar os merge de l10n_br_nfe e GINFES na OCA por conta da ultimas coisas de mde
vamos fazer o merge do grosso das coisas primeiro e depois as melhorias
entao para ser menor, o ponto eh que qdo vc tem o xsd envNfe.xsd e retEnvNFe.xsd, esse segundo xsd inclue tudo que vc usa no erpbrasil.edocs ja
se vc pegar os arquivos py dos 2 vc dobra 95% do codigo de forma inutil
se vc depoiis quer rodar o Odoo em instancia de 512 Mb de RAM, se vc multiplicar o tamanho das libs fiscais assim pelo numero de libs usadas e pelo numero de workers aquil tem a sua importancia
com essas libs verbosa se vc bota muito arquivos de bobeira vc rapidamente dobra a quantidade de codigo que ta rodando comparando com o codigo do Odoo
entao meu ponto seria vcs da KMEE limparem a nfselib um pouco dessa forma
realmente eu nao quero ser chato, tudo bem se nao tem um historico igual da nfelib
tudo bem se tem 5x menos teste
tudo bem se tem gambiarras em vez de subclasses na GINFES
MAS no minimo teria que ter a receita do bolo repetitivel na branch master e que seja diferente da branch gerida, um teste minimo de importacao das classes que usa
ser gerido com generateDS atualizado e nao de 2 anos atras
senao eh falacia que vc pode dar suporte nisso como vc daria num modulo OCA qualquer
vou escrever um RFC sobre isso
mas ai seria isso e botar um Travis no erpbrasil.edoc
ai a KMEE fazer essa limpeza de casa das dependencias dos PRs dela.

[rvalyi]

Legal que vc aprovou @mileo . Compartilhei o PR qto antes, porem ele nao foi testado em produção ainda. Tem testes na nfelib mas nao no erpbrasil.edoc. E outro ponto, seria bom vcs testarem a parte de transmissao de MDe porque eu nao inclui a mudanca manual que vcs fizeram na geracao de MDe, teria que ver se ainda seria preciso ja que oo generateDS tem 2 anos de bugfix e pode ser que nao seja mais precisao.

[mileo]

Legal que vc aprovou @mileo . Compartilhei o PR qto antes, porem ele nao foi testado em produção ainda. Tem testes na nfelib mas nao no erpbrasil.edoc. E outro ponto, seria bom vcs testarem a parte de transmissao de MDe porque eu nao inclui a mudanca manual que vcs fizeram na geracao de MDe, teria que ver se ainda seria preciso ja que oo generateDS tem 2 anos de bugfix e pode ser que nao seja mais precisao.

Sim, estou começando a montar o ambiente para testar tudo, inclusive a nfse.

[mileo]

Favor reabrir o PR