[FIX] pep8 + lint

l10n_br_account_payment_cobranca/models/account_invoice.py

@@ -261,7 +261,9 @@ def create_api_account_payment_line(self):
 
                 new_payorder = False
                 if not payorder:
-                    payorder = apoo.create(inv._prepare_new_payment_order(payment_mode))
+                    payorder = apoo.create(
+                        inv._prepare_new_payment_order(payment_mode)
+                    )
                     new_payorder = True
                     payorder.name += "_api"
                     payorder.active = False
@@ -384,10 +386,11 @@ def create_account_payment_line_baixa(self):
 
                 new_payorder = False
                 if not payorder:
-                    payorder = apoo.create(inv._prepare_new_payment_order(payment_mode))
+                    payorder = apoo.create(
+                        inv._prepare_new_payment_order(payment_mode)
+                    )
                     new_payorder = True
                 result_payorder_ids.append(payorder.id)
-                action_payment_type = payorder.payment_type
                 count = 0
                 for line in applicable_lines.filtered(
                     lambda x: x.payment_mode_id == payment_mode

l10n_br_account_payment_cobranca/models/account_move_line.py

@@ -4,8 +4,7 @@
 
 import logging
 
-from odoo import _, api, fields, models
-from odoo.exceptions import UserError
+from odoo import api, fields, models
 
 _logger = logging.getLogger(__name__)
 

l10n_br_account_payment_cobranca/models/l10n_br_cnab.py

@@ -89,23 +89,24 @@
     RETORNO_400_BAIXA,
 ]
 
-# COD_REGISTROS_REJEITADOS_CNAB400 -> USADO QUANDO HA CODIGO DE OCORRENCIA 03 NA POSIÇÃO 109-110
+# COD_REGISTROS_REJEITADOS_CNAB400 -> USADO QUANDO HA CODIGO DE OCORRENCIA 03
+# NA POSIÇÃO 109-110
 COD_REGISTROS_REJEITADOS_CNAB400 = {
     3: "AG. COBRADORA - CEP SEM ATENDIMENTO DE PROTESTO NO MOMENTO",
     4: "ESTADO - SIGLA DO ESTADO INVÁLIDA",
-    5: "DATA VENCIMENTO - PRAZO DA OPERAÇÃO MENOR QUE PRAZO MÍNIMO OU MAIOR QUE O MÁXIMO",
+    5: "DATA VENCIMENTO - PRAZO DA OPERAÇÃO MENOR QUE PRAZO MÍNIMO OU MAIOR QUE O MÁXIMO",  # noqa
     7: "VALOR DO TÍTULO - VALOR DO TÍTULO MAIOR QUE 10.000.000,00",
     8: "NOME DO PAGADOR - NÃO INFORMADO OU DESLOCADO",
     9: "AGENCIA/CONTA - AGÊNCIA ENCERRADA",
     10: "LOGRADOURO - NÃO INFORMADO OU DESLOCADO",
     11: "CEP - CEP NÃO NUMÉRICO OU CEP INVÁLIDO",
-    12: "SACADOR / AVALISTA - NOME NÃO INFORMADO OU DESLOCADO (BANCOS CORRESPONDENTES)",
+    12: "SACADOR / AVALISTA - NOME NÃO INFORMADO OU DESLOCADO (BANCOS CORRESPONDENTES)",  # noqa
     13: "ESTADO/CEP - CEP INCOMPATÍVEL COM A SIGLA DO ESTADO",
-    14: "NOSSO NÚMERO - NOSSO NÚMERO JÁ REGISTRADO NO CADASTRO DO BANCO OU FORA DA FAIXA",
+    14: "NOSSO NÚMERO - NOSSO NÚMERO JÁ REGISTRADO NO CADASTRO DO BANCO OU FORA DA FAIXA",  # noqa
     15: "NOSSO NÚMERO - NOSSO NÚMERO EM DUPLICIDADE NO MESMO MOVIMENTO",
     18: "DATA DE ENTRADA - DATA DE ENTRADA INVÁLIDA PARA OPERAR COM ESTA CARTEIRA",
     19: "OCORRÊNCIA - OCORRÊNCIA INVÁLIDA",
-    21: "AG. COBRADORA - CARTEIRA NÃO ACEITA DEPOSITÁRIA CORRESPONDENTE ESTADO DA AGÊNCIA DIFERENTE DO ESTADO DO PAGADOR AG. COBRADORA NÃO CONSTA NO CADASTRO OU ENCERRANDO",
+    21: "AG. COBRADORA - CARTEIRA NÃO ACEITA DEPOSITÁRIA CORRESPONDENTE ESTADO DA AGÊNCIA DIFERENTE DO ESTADO DO PAGADOR AG. COBRADORA NÃO CONSTA NO CADASTRO OU ENCERRANDO",  # noqa
     22: "CARTEIRA - CARTEIRA NÃO PERMITIDA (NECESSÁRIO CADASTRAR FAIXA LIVRE)",
     26: "AGÊNCIA/CONTA - AGÊNCIA/CONTA NÃO LIBERADA PARA OPERAR COM COBRANÇA",
     27: "CNPJ INAPTO - CNPJ DO BENEFICIÁRIO INAPTO DEVOLUÇÃO DE TÍTULO EM GARANTIA",
@@ -118,34 +119,34 @@
     42: "NOSSO NÚMERO - NOSSO NÚMERO FORA DE FAIXA",
     52: "AG. COBRADORA - EMPRESA NÃO ACEITA BANCO CORRESPONDENTE",
     53: "AG. COBRADORA - EMPRESA NÃO ACEITA BANCO CORRESPONDENTE - COBRANÇA MENSAGEM",
-    54: "DATA DE VENCTO - BANCO CORRESPONDENTE - TÍTULO COM VENCIMENTO INFERIOR A 15 DIAS",
+    54: "DATA DE VENCTO - BANCO CORRESPONDENTE - TÍTULO COM VENCIMENTO INFERIOR A 15 DIAS",  # noqa
     55: "DEP/BCO CORRESP - CEP NÃO PERTENCE À DEPOSITÁRIA INFORMADA",
     56: "DT VENCTO/BCO CORRESP - VENCTO SUPERIOR A 180 DIAS DA DATA DE ENTRADA",
-    57: "DATA DE VENCTO - CEP SÓ DEPOSITÁRIA BCO DO BRASIL COM VENCTO INFERIOR A 8 DIAS",
+    57: "DATA DE VENCTO - CEP SÓ DEPOSITÁRIA BCO DO BRASIL COM VENCTO INFERIOR A 8 DIAS",  # noqa
     60: "ABATIMENTO - VALOR DO ABATIMENTO INVÁLIDO",
     61: "JUROS DE MORA - JUROS DE MORA MAIOR QUE O PERMITIDO",
     62: "DESCONTO - VALOR DO DESCONTO MAIOR QUE VALOR DO TÍTULO",
-    63: "DESCONTO DE ANTECIPAÇÃO - VALOR DA IMPORTÂNCIA POR DIA DE DESCONTO (IDD) NÃO PERMITIDO",
+    63: "DESCONTO DE ANTECIPAÇÃO - VALOR DA IMPORTÂNCIA POR DIA DE DESCONTO (IDD) NÃO PERMITIDO",  # noqa
     64: "DATA DE EMISSÃO - DATA DE EMISSÃO DO TÍTULO INVÁLIDA",
     65: "TAXA FINANCTO - TAXA INVÁLIDA (VENDOR)",
     66: "DATA DE VENCTO - INVALIDA/FORA DE PRAZO DE OPERAÇÃO (MÍNIMO OU MÁXIMO)",
     67: "VALOR/QTIDADE - VALOR DO TÍTULO/QUANTIDADE DE MOEDA INVÁLIDO",
     68: "CARTEIRA - CARTEIRA INVÁLIDA OU NÃO CADASTRADA NO INTERCÂMBIO DA COBRANÇA",
     69: "CARTEIRA - CARTEIRA INVÁLIDA PARA TÍTULOS COM RATEIO DE CRÉDITO",
     70: "AGÊNCIA/CONTA - BENEFICIÁRIO NÃO CADASTRADO PARA FAZER RATEIO DE CRÉDITO",
-    78: "AGÊNCIA/CONTA - DUPLICIDADE DE AGÊNCIA/CONTA BENEFICIÁRIA DO RATEIO DE CRÉDITO",
-    80: "AGÊNCIA/CONTA - QUANTIDADE DE CONTAS BENEFICIÁRIAS DO RATEIO MAIOR DO QUE O PERMITIDO (MÁXIMO DE 30 CONTAS POR TÍTULO)",
+    78: "AGÊNCIA/CONTA - DUPLICIDADE DE AGÊNCIA/CONTA BENEFICIÁRIA DO RATEIO DE CRÉDITO",  # noqa
+    80: "AGÊNCIA/CONTA - QUANTIDADE DE CONTAS BENEFICIÁRIAS DO RATEIO MAIOR DO QUE O PERMITIDO (MÁXIMO DE 30 CONTAS POR TÍTULO)",  # noqa
     81: "AGÊNCIA/CONTA - CONTA PARA RATEIO DE CRÉDITO INVÁLIDA / NÃO PERTENCE AO ITAÚ",
-    82: "DESCONTO/ABATI-MENTO - DESCONTO/ABATIMENTO NÃO PERMITIDO PARA TÍTULOS COM RATEIO DE CRÉDITO",
-    83: "VALOR DO TÍTULO - VALOR DO TÍTULO MENOR QUE A SOMA DOS VALORES ESTIPULADOS PARA RATEIO",
-    84: "AGÊNCIA/CONTA - AGÊNCIA/CONTA BENEFICIÁRIA DO RATEIO É A CENTRALIZADORA DE CRÉDITO DO BENEFICIÁRIO",
-    85: "AGÊNCIA/CONTA - AGÊNCIA/CONTA DO BENEFICIÁRIO É CONTRATUAL / RATEIO DE CRÉDITO NÃO PERMITIDO",
-    86: "TIPO DE VALOR - CÓDIGO DO TIPO DE VALOR INVÁLIDO / NÃO PREVISTO PARA TÍTULOS COM RATEIO DE CRÉDITO",
-    87: "AGÊNCIA/CONTA - REGISTRO TIPO 4 SEM INFORMAÇÃO DE AGÊNCIAS/CONTAS BENEFICIÁRIAS DO RATEIO",
-    90: "NRO DA LINHA - COBRANÇA MENSAGEM - NÚMERO DA LINHA DA MENSAGEM INVÁLIDO OU QUANTIDADE DE LINHAS EXCEDIDAS",
-    97: "SEM MENSAGEM - COBRANÇA MENSAGEM SEM MENSAGEM (SÓ DE CAMPOS FIXOS), PORÉM COM REGISTRO DO TIPO 7 OU 8",
-    98: "FLASH INVÁLIDO - REGISTRO MENSAGEM SEM FLASH CADASTRADO OU FLASH INFORMADO DIFERENTE DO CADASTRADO",
-    99: "FLASH INVÁLIDO - CONTA DE COBRANÇA COM FLASH CADASTRADO E SEM REGISTRO DE MENSAGEM CORRESPONDENTE",
+    82: "DESCONTO/ABATI-MENTO - DESCONTO/ABATIMENTO NÃO PERMITIDO PARA TÍTULOS COM RATEIO DE CRÉDITO",  # noqa
+    83: "VALOR DO TÍTULO - VALOR DO TÍTULO MENOR QUE A SOMA DOS VALORES ESTIPULADOS PARA RATEIO",  # noqa
+    84: "AGÊNCIA/CONTA - AGÊNCIA/CONTA BENEFICIÁRIA DO RATEIO É A CENTRALIZADORA DE CRÉDITO DO BENEFICIÁRIO",  # noqa
+    85: "AGÊNCIA/CONTA - AGÊNCIA/CONTA DO BENEFICIÁRIO É CONTRATUAL / RATEIO DE CRÉDITO NÃO PERMITIDO",  # noqa
+    86: "TIPO DE VALOR - CÓDIGO DO TIPO DE VALOR INVÁLIDO / NÃO PREVISTO PARA TÍTULOS COM RATEIO DE CRÉDITO",  # noqa
+    87: "AGÊNCIA/CONTA - REGISTRO TIPO 4 SEM INFORMAÇÃO DE AGÊNCIAS/CONTAS BENEFICIÁRIAS DO RATEIO",  # noqa
+    90: "NRO DA LINHA - COBRANÇA MENSAGEM - NÚMERO DA LINHA DA MENSAGEM INVÁLIDO OU QUANTIDADE DE LINHAS EXCEDIDAS",  # noqa
+    97: "SEM MENSAGEM - COBRANÇA MENSAGEM SEM MENSAGEM (SÓ DE CAMPOS FIXOS), PORÉM COM REGISTRO DO TIPO 7 OU 8",  # noqa
+    98: "FLASH INVÁLIDO - REGISTRO MENSAGEM SEM FLASH CADASTRADO OU FLASH INFORMADO DIFERENTE DO CADASTRADO",  # noqa
+    99: "FLASH INVÁLIDO - CONTA DE COBRANÇA COM FLASH CADASTRADO E SEM REGISTRO DE MENSAGEM CORRESPONDENTE",  # noqa
 }
 
 
@@ -392,7 +393,8 @@ def _lote_400(self, evento, lote_id):
                             "name": evento.nosso_numero,
                             "nosso_numero": evento.nosso_numero,
                             "numero_documento": evento.numero_documento,
-                            "identificacao_titulo_empresa": evento.identificacao_titulo_empresa,
+                            "identificacao_titulo_empresa":
+                                evento.identificacao_titulo_empresa,
                             "credit": float(evento.valor_principal)
                             + float(evento.tarifa_cobranca),
                             "account_id": invoice.payment_mode_id.default_account_id.id
@@ -427,8 +429,9 @@ def _lote_400(self, evento, lote_id):
                                     "name": str(evento.nosso_numero) + " - Tarifa",
                                     "credit": 0,
                                     "debit": float(evento.tarifa_cobranca),
-                                    "account_id": invoice.payment_mode_id.default_tax_account_id.id
-                                    or invoice.account_id.id,
+                                    "account_id": invoice.payment_mode_id.\
+                                                  default_tax_account_id.id
+                                                  or invoice.account_id.id,
                                 }
                             )
                             line_values.append((0, 0, line_dict_tarifa))
@@ -593,7 +596,6 @@ def _reprocessa_lote_400(self, evento, lote_id):
         else:
             cnab_event_id.write(vals_evento)
 
-        amount = 0.0
         codigo_ocorrencia = evento.codigo_ocorrencia
         if codigo_ocorrencia and bank_payment_line_id:
 
@@ -632,7 +634,8 @@ def _reprocessa_lote_400(self, evento, lote_id):
                     "name": evento.nosso_numero,
                     "nosso_numero": evento.nosso_numero,
                     "numero_documento": evento.numero_documento,
-                    "identificacao_titulo_empresa": evento.identificacao_titulo_empresa,
+                    "identificacao_titulo_empresa":
+                        evento.identificacao_titulo_empresa,
                     "date_maturity": datetime.strptime(
                         str(evento.vencimento).zfill(6), STR_EVENTO_FORMAT
                     )

l10n_br_account_payment_cobranca/readme/CONFIGURE.rst

@@ -1,10 +1,2 @@
-[ This file is optional, it should explain how to configure
-  the module before using it; it is aimed at advanced users. ]
-
-To configure this module, you need to:
-
-#. Go to ...
-
-.. figure:: ../static/description/image.png
-   :alt: alternative description
-   :width: 600 px
++ In a multicompany environment, make sure the sequences payment line and payment order are with the company set to the one you'll use to export the payments and charges.
+    If they are not set like this, you'll not be able to add payment lines with regular users.

l10n_br_account_payment_cobranca/readme/CONTRIBUTORS.rst

@@ -1,2 +1,3 @@
-* Firstname Lastname <[email protected]> (optional company website url)
-* Second Person <[email protected]> (optional company website url)
+* Luis Felipe Mileo <[email protected]>
+* Fernando Marcato Rodrigues <[email protected]>
+* Daniel Sadamo <[email protected]>

l10n_br_account_payment_cobranca/readme/CREDITS.rst

@@ -1,7 +1,3 @@
-[ This file is optional and contains additional credits, other than
-  authors, contributors, and maintainers. ]
-
 The development of this module has been financially supported by:
 
-* Company 1 name
-* Company 2 name
+* KMEE INFORMATICA LTDA - www.kmee.com.br

l10n_br_account_payment_cobranca/readme/DESCRIPTION.rst

@@ -1,4 +1,2 @@
-[ This file must be max 2-3 paragraphs, and is required. ]
-
-This module extends the functionality of portal to support Brasilian
-fields.
+This module provide an infrastructure to export payment and debit orders in
+Febraban layout.

l10n_br_account_payment_cobranca/readme/HISTORY.rst

@@ -1,24 +0,0 @@
-[ The change log. The goal of this file is to help readers
-  understand changes between version. The primary audience is
-  end users and integrators. Purely technical changes such as
-  code refactoring must not be mentioned here.
-
-  This file may contain ONE level of section titles, underlined
-  with the ~ (tilde) character. Other section markers are
-  forbidden and will likely break the structure of the README.rst
-  or other documents where this fragment is included. ]
-
-11.0.x.y.z (YYYY-MM-DD)
-~~~~~~~~~~~~~~~~~~~~~~~
-
-* [BREAKING] Breaking changes come first.
-  (`#70 <https://github.com/OCA/repo/issues/70>`_)
-* [ADD] New feature.
-  (`#74 <https://github.com/OCA/repo/issues/74>`_)
-* [FIX] Correct this.
-  (`#71 <https://github.com/OCA/repo/issues/71>`_)
-
-11.0.x.y.z (YYYY-MM-DD)
-~~~~~~~~~~~~~~~~~~~~~~~
-
-* ...

l10n_br_account_payment_cobranca/readme/INSTALL.rst

@@ -1,7 +1,2 @@
-[ This file must only be present if there are very specific
-  installation instructions, such as installing non-python
-  dependencies. The audience is systems administrators. ]
-
-To install this module, you need to:
-
-#. Do this ...
+This module depends on :
+* l10n_br_account_payment_order

l10n_br_account_payment_cobranca/readme/ROADMAP.rst

@@ -1,5 +0,0 @@
-[ Enumerate known caveats and future potential improvements.
-  It is mostly intended for end-users, and can also help
-  potential new contributors discovering new features to implement. ]
-
-* ...

l10n_br_account_payment_cobranca/readme/USAGE.rst

@@ -1,11 +1 @@
-[ This file must be present and contains the usage instructions
-  for end-users. As all other rst files included in the README,
-  it MUST NOT contain reStructuredText sections
-  only body text (paragraphs, lists, tables, etc). Should you need
-  a more elaborate structure to explain the addon, please create a
-  Sphinx documentation (which may include this file as a "quick start"
-  section). ]
-
-To use this module, you need to:
-
-#. Go to ...
+This module adds a Payment type "Cobrança" and related configuration fields in Payment Mode

l10n_br_account_payment_cobranca/wizard/payment_order_create.py

@@ -88,12 +88,12 @@ def _prepare_move_line_domain(self):
         # Exclude lines that are already in a non-cancelled
         # and non-uploaded payment order; lines that are in a
         # uploaded payment order are proposed if they are not reconciled,
-        paylines = self.env["account.payment.line"].search(
-            [
-                ("state", "in", ("draft", "open", "generated", "uploaded")),
-                ("move_line_id", "!=", False),
-            ]
-        )
+        # paylines = self.env["account.payment.line"].search(
+        #     [
+        #         ("state", "in", ("draft", "open", "generated", "uploaded")),
+        #         ("move_line_id", "!=", False),
+        #     ]
+        # )
 
         move_line_domain = ["draft"]
         if self.allow_error: