[16.0][REF] l10n_br_purchase: default load demo #3644
Conversation
|
Hi @rvalyi, @renatonlima, |
antoniospneto
changed the title
|
Ola @antoniospneto valeu por olhar isso! Pois simplificando isso a gente abaixa bastante a barreira de entrada no projeto. O @mbcosta ja tinha simplificado muito no #3629 mais pode ser que funcione de forma ainda mais simples com essa forma que vc ta propondo aqui. Vou dar uma pensada melhor amanhã... Porem tem que verificar essa mesmo dos COA estar carregados e se tá tão robusto quanto o que o @mbcosta fez... Tem que verificar o cenário do tipo: vc instala o l10n_br_purchase e só depois instala o l10n_br_coa_generic. A CI instala todos módulos do repo antes de rodar os testes então isso não vai acontecer na CI MAS, pro zé mané que vai baixar a localização e tentar instalar de qualquer jeito, com certeza esse tipo de cenário vai acontecer... |
PR para prova de conceito.
@rvalyi o que você acha dessa proposta em alternativa a PR #3629
Por hora apliquei apenas no módulo purchase, para exemplificar.
A vatagem é que eliminamos totalmete o uso de hooks, deixando o carregamento de dados totalmente de forma padrão.
O que eu fiz foi utilizar o contexto no XML pra definir a empresa ativa/corrente.
Outro detalhe é que eu acho que não tem a necessidade de verificar se os planos de contas estão instalados, quando carregado os dados demos é uma exigencia todos os COAs estarem instalados não?