[14.0][REF][l10n_br_fiscal][l10n_br_fiscal_edi] l10n_br_fiscal_edi extraction from l10n_br_fiscal 1/2

[rvalyi]

Uns anos atras foi enfiado as pressas (...) bastante código no modulo l10n_br_fiscal sem se perguntar muito se poderia ter sido mais modular... Ai que ta hoje o modulo l10n_br_fiscal, alem de ser o maior da OCA inteira pesa nada menos do que uns 20k linhas de código (python e XML se descontar o que é XML de data):

cloc l10n_br_fiscal  #  atual:
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
CSV                             24              0              0          32260
XML                             82           1150            471          16137
Python                         110           2603            828          12964
PO File                          1           1654           6255           3339
HTML                             1             86              6            455
reStructuredText                 6            123             48            187
-------------------------------------------------------------------------------
SUM:                           224           5616           7608          65342
-------------------------------------------------------------------------------

Bem na real muito desse código de eventos e tal que nem sempre ta tão bom assim (...) poderia sim ter sido adicionado de forma mais modular, ou pelo menos melhorado durante esses anos por quem adicionou em vez de querer começar mil outras coisas....

Não se assusta, esse PR é apenas um Proof Of Concept testado na 14.0, mas a gente queria fazer isso apenas na v16.
A ideia num primeiro momento, seria na v16, jogar o que eu tirei do l10n_br_fiscal num novo modulo que poderia se chamar l10n_br_fiscal_edi e que teria as coisas para interagir com documentos eletronicos e que são comuns entre os vários documentos eletronicos. Nisso, depois de alguns amends, nenhuma funcionalidade seria perdida e a dezena de modulos que eu desativei inicialmente como l10n_br_nfe, l10n_br_nfse* apenas iriam depender desse novo modulo l10n_br_fiscal_edi (que eu ainda não botei) e funcionariam normalmente. Apenas teria que cuidar de uns detalhes de herança no l10n_br_fiscal_edi para adicionar tudo de forma correta, mas seria tb a oportunidade de limpar tb um código com camadas que se misturaram demais entre documentos eletronicos ou não.

O @renatonlima já tinha começado a bolar uma boa melhorada dessas coisas que eu extrai do l10n_br_fiscal ja na época da v12, so não deu tempo de fazer o refactor ainda. Mais uma oportunidade de perceber que não adianta querer queimar a largada e fazer as coisas as pressas de qualquer jeito, porque depois acaba custando sempre 10x mais quando tem que fazer scripts de migração, e compatibilizar um montão de módulos que começaram a depender de coisas mais ou menos. E ainda complica os cherry-picks entre as branches...

cloc l10n_br_fiscal  # depois, sem limpar muito ainda:
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
CSV                             24              0              0          32252
XML                             77           1131            505          15818
Python                         103           2472            859          12214
PO File                          1           1654           6255           3339
HTML                             1             86              6            456
reStructuredText                 6            123             48            187
-------------------------------------------------------------------------------
SUM:                           212           5466           7673          64266
-------------------------------------------------------------------------------

tira umas 1200 linhas.

[OCA-git-bot]

Hi @renatonlima,
some modules you are maintaining are being modified, check this out!

[rvalyi]

ai pessoal ta passando os testes. Eu tive que desativar 11 modulos dos 59 do repo que passariam então a depender do novo l10n_br_fiscal_edi (que eu ainda nao botei):

  ^l10n_br_account_nfe/|
  ^l10n_br_delivery_nfe/|
  ^l10n_br_fiscal_closing/|
  ^l10n_br_ie_search/|
  ^l10n_br_nfe/|
  ^l10n_br_nfse/|
  ^l10n_br_nfse_barueri/|
  ^l10n_br_nfse_focus/|
  ^l10n_br_nfse_ginfes/|
  ^l10n_br_nfse_paulistana/|
  ^l10n_br_pos_nfce/|

Vou conservar esse PR e essa branch como base e fazer outra branch com a adição do l10n_br_fiscal_edi e a re-ativação desses 11 modulos. Teve tb uns ajustes nos testes do l10n_br_fiscal e nos wizards do l10n_br_account, coisas que a gente consegue ajustar depois.

[rvalyi]

o PR completo com o l10n_br_fiscal_edi e os módulos todos instaláveis de volta continua aqui #3012

[rvalyi]

fechando esse e deixando o foco neste PR onde os trabalhos de extraçao foram levados adiante #3012