12.0 mig l10n_br_account_payment_cobranca (renomeado l10n_br_account_payment_order)

[mileo]

Depende do PR #702

[rvalyi]

@mileo as #702 has been merged, can you also rebase please?

[mileo]

Sim, mas acho melhor revisar o da v10... Conseguimos trabalhar nisso amanhã.

[mileo]

Mas alguns itens que foram revisados aqui podem ser aplicados lá...

[mileo]

Olá pessoal, revisando encontrei alguns issues que nosso time vai começar a trabalhar amanhã, vou postando cada um deles aqui;

Caso encontrem mais algum por favor avisem e tb se forem trabalhar neles....

[mileo]

A baixa de pagamento esta criando uma movimentação de baixa a ser enviada para o banco.

Através dos métodos:

1. create_account_payment_line_baixa

2. register_payment

Entretanto isto só esta sendo criado quando a invoice é paga através do account.payment.

parts/odoo/addons/account/models  (2 usages found)

    account_invoice.py  (1 usage found)

        1099 return self.register_payment(credit_aml)

    account_payment.py  (1 usage found)

        755 self.invoice_ids.register_payment(counterpart_aml)

Deve ser verificado caso a fatura seja paga através do assistente de reconciliação:

Seja ao juntar dois lançamentos contábeis;

Ou através do wizard de reconciliação, ao associar uma linha de um extrato bancário a uma fatura:

parts/odoo/addons/account/models  (8 usages found)

    account_bank_statement.py  (1 usage found)

        700 (new_aml | counterpart_move_line).reconcile()

    account_invoice.py  (1 usage found)

        1086 return (line_to_reconcile + payment_line).reconcile(writeoff_acc_id, writeoff_journal_id)

    account_move.py  (3 usages found)

        471 to_rec.reconcile()

        871 aml_to_balance.reconcile()

        1706 to_clear_aml.reconcile()

    account_payment.py  (1 usage found)

        688 (transfer_credit_aml + transfer_debit_aml).reconcile()

    reconciliation_widget.py  (2 usages found)

        768 (account_move_line + writeoff_lines).reconcile()

        770 account_move_line.reconcile()

parts/odoo/addons/account/wizard  (1 usage found)

    account_invoice_refund.py  (1 usage found)

        77 to_reconcile_lines.filtered(lambda l: l.reconciled == False).reconcile()

Que é um método mais profundo que também é chamado dentro do reconcile

[mileo]

Cobrança via API deve ser integrada ao fluxo normal

O fluxo de cobrança via API deve ser integrado ao fluxo de confirmação da fatura, os botões devem ser removidos, ao se confirmar uma fatura o boleto já deve ser registrado;

[mileo]

Uma linha de pagamento esta sempre sendo criada, mesmo que não seja necessário

Caso haja um modo payment_mode_id, sempre esta sendo criado um linha de pagamento.

@api.multi
def invoice_validate(self):
    result = super(AccountInvoice, self).invoice_validate()
    filtered_invoice_ids = self.filtered(lambda s: s.payment_mode_id)
    if filtered_invoice_ids:
        filtered_invoice_ids.create_account_payment_line()
    return result

[mbcosta]

ola @renatonlima @mileo @sadamo algumas questões sobre mudanças de campos entre as versões 8 e 12:

Na 8 tinhamos o campo bank_id no objeto account.payment.mode o que identificava o Modo do Pagamento a uma Conta Bancaria e consequentemente ao Banco a ser usado, esse campo parece não existir mais, alguém saberia dizer de onde essa informação pode ser obtida na 12 ou se vai ser preciso reimplementar ?

Na 8 eu adicionei alguns campos no account.payment.mode referentes a informações usadas nos boletos boleto_perc_mora "Percentual de Juros de Mora", instrucao_boleto_perc_mora 'Instrução Juros Mora', boleto_perc_multa "Percentual de Multa", instrucao_boleto_perc_multa 'Instrução Multa por Atraso' no objeto account.payment.mode https://github.com/OCA/l10n-brazil/blob/8.0/l10n_br_account_payment_boleto/models/payment_mode.py#L60 isso foi removido, esses campos não são mais necessários ou estariam em outro objeto ?

[mileo]

@mbcosta parte dessas alterações que vc citou, dependem da revisão do #820

Se vc puder dar uma revisada nele agradeço. De toda forma amanhã comento com calma essas questões.

@rvalyi @renatonlima

[mbcosta]

Sobre a questão de onde buscar a conta bancaria e o banco estou usando a conta vinculada ao Diário Bancario.

Estou testando Boleto e Remessa Unicred tenho que incluir dois campos especificos a esse banco:

• POSTO Código do posto da cooperativa de crédito
• BYTE_IDT Byte de identificação do cedente do bloqueto utilizado para compor o nosso número.

Por esse motivo estou pensando em incluir um campo related do Banco dentro do Modo de Pagto ( campo hoje está na Conta Bancaria dentro do Diário ) para permitir tornar esses novos campos invisiveis caso não seja uma conta desse Banco, o que acham ?

[mbcosta]

ola @mileo sobre o que escrevi
"Na 8 eu adicionei alguns campos no account.payment.mode referentes a informações usadas nos boletos boleto_perc_mora "Percentual de Juros de Mora", instrucao_boleto_perc_mora 'Instrução Juros Mora', boleto_perc_multa "Percentual de Multa", instrucao_boleto_perc_multa 'Instrução Multa por Atraso' no objeto account.payment.mode https://github.com/OCA/l10n-brazil/blob/8.0/l10n_br_account_payment_boleto/models/payment_mode.py#L60 isso foi removido, esses campos não são mais necessários ou estariam em outro objeto ?"

Esse campos não deveriam estar no modulo que implementa o Boleto ? Não vejo relação com o modulo do PR que cria Pagamentos/Financeiro, qual seria ?

[renatonlima]

Pessoal,

Esse PR já tem um tempo que esta aberto, como ele já esta funcional, apesar de ter ainda algumas features para serem implementadas, eu vou fazer o merge deste PR e depois pode ser feito outros PRs menores para as novas features.

[renatonlima]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-768-by-renatonlima-bump-nobump, awaiting test results.

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[OCA-git-bot]

Congratulations, your PR was merged at 406884b. Thanks a lot for contributing to OCA. ❤️