rvalyi
commented
7 years ago Ola, eu estou avaliando esse PR e ja posso dar um retorno ate o modulo l10n_br_account_payment_boleto e suas dependencias:
Nao tenho problema fundamental com o modulo, porem alguns detalhes:
- l10n_br_account_payment_boleto depende do modulo account_due_list_payment_mode. Porem essa dependencia ja eh trazida pelo modulo l10n_br_account_payment_mode. Portanto a inutil trazer a dependencia de novo. Obrigado @mbcosta por ter levantado isso.
- no arquivo l10n_br_account_payment_boleto/models/account_move_line.py, o campo date_payment_created que esta initializado com
move_line.date_payment_created = date.today()parece inutil. Porque nao usar o campo create_date do objeto account.move.line que ja existe? Nao eh mais correcto usar a data que o account.move.line foi gerado (o create_date)? Novamente obrigado a @mbcosta por ter levantado isso. - A ausencia de alguns campos obrigatorios pode criar erros nao explicitos na hora de gerar os boletos. Em vez disso, esses errors deveriam ser explicitos,
- Em l10n_br_account_payment_boleto/boleto/document.py temos
self.boleto.numero_documento = move_line.name.encode('utf-8'). Porem parece que o campo numero_documento deve ser um numero. Se usamos o name ele pode ser outra coisa do que um numero e pode ainda vir o string 'False' o que trava depois na geraçao do boleto. Nao poderiamos usar:self.boleto.numero_documento = move_line.id? Eu mudei esse ponto e resolvi o ponto precendente nesse PR: https://github.com/kmee/odoo-brazil-banking/pull/20 - Nao eh possivel editar o campo boleto_convenio no objeto payment.mode. Eu acho que eh preciso ter esse campo na visao. Por examplo quis accrescentar boletos Itau e nao pude faze-lo pela interface.
Eu tou revisando agora a parte da geraçao de arquivos de remessa e importaçao de retorno, vou fazer feedback dos outros modulos em breve. cc @mbcosta @renatonlima @mileo @Hendrixcosta
hendrixcosta
:+1: