[16.0] Módulo l10n_br_account não funciona sem o l10n_sale

[rodmad85]

Branch 16.0

Conforme teste realizado na Issue #3610 . Verifiquei que mesmo desativando a instalação automática do módulo l10n_br_sale e instalando somente o l10n_br_account, ocorre o seguinte erro ao selecionar um imposto na linha do Pedido de Vendas.

Traceback (most recent call last):
  File "/opt/odoo/odoo/odoo/http.py", line 1657, in _serve_db
    return service_model.retrying(self._serve_ir_http, self.env)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/odoo/service/model.py", line 133, in retrying
    result = func()
             ^^^^^^
  File "/opt/odoo/odoo/odoo/http.py", line 1685, in _serve_ir_http
    response = self.dispatcher.dispatch(rule.endpoint, args)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/odoo/http.py", line 1889, in dispatch
    result = self.request.registry['ir.http']._dispatch(endpoint)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/odoo/addons/base/models/ir_http.py", line 154, in _dispatch
    result = endpoint(**request.params)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/odoo/http.py", line 734, in route_wrapper
    result = endpoint(self, *args, **params_ok)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/addons/web/controllers/dataset.py", line 42, in call_kw
    return self._call_kw(model, method, args, kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/addons/web/controllers/dataset.py", line 33, in _call_kw
    return call_kw(request.env[model], method, args, kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/odoo/api.py", line 468, in call_kw
    result = _call_kw_multi(method, model, args, kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/odoo/api.py", line 453, in _call_kw_multi
    result = method(recs, *args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/odoo/models.py", line 6643, in onchange
    if name not in done and snapshot0.has_changed(name)
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/odoo/models.py", line 6426, in has_changed
    return self[name] != record[name]
                         ~~~~~~^^^^^^
  File "/opt/odoo/odoo/odoo/models.py", line 5975, in __getitem__
    return self._fields[key].__get__(self, self.env.registry[self._name])
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/odoo/fields.py", line 1158, in __get__
    self.recompute(record)
  File "/opt/odoo/odoo/odoo/fields.py", line 1370, in recompute
    apply_except_missing(self.compute_value, recs)
  File "/opt/odoo/odoo/odoo/fields.py", line 1343, in apply_except_missing
    func(records)
  File "/opt/odoo/odoo/odoo/fields.py", line 1392, in compute_value
    records._compute_field_value(self)
  File "/opt/odoo/odoo/odoo/models.py", line 4259, in _compute_field_value
    fields.determine(field.compute, self)
  File "/opt/odoo/odoo/odoo/fields.py", line 98, in determine
    return needle(*args)
           ^^^^^^^^^^^^^
  File "/opt/odoo/extra-addons/sale-workflow/sale_discount_display_amount/models/sale_order_line.py", line 52, in _compute_amount
    res = super(SaleOrderLine, self)._compute_amount()
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/addons/sale/models/sale_order_line.py", line 626, in _compute_amount
    tax_results = self.env['account.tax'].with_company(line.company_id)._compute_taxes(
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/odoo/addons/account/models/account_tax.py", line 1126, in _compute_taxes
    to_update_vals, tax_values_list = self._compute_taxes_for_single_line(
                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/odoo/extra-addons/l10n-brazil/l10n_br_account/models/account_tax.py", line 235, in _compute_taxes_for_single_line
    fiscal_taxes=line.fiscal_tax_ids,
                 ^^^^^^^^^^^^^^^^^^^
AttributeError: 'sale.order.line' object has no attribute 'fiscal_tax_ids'

The above server error caused the following client error:
null

[antoniospneto]

O erro parece ser desencadeado por causa do módulo sale_discount_display_amount esse módulo depende do sales, então ele não deveria tá instalado na sua implementação.

[antoniospneto]

Obs: se você não quer o módulo sales, seria interessante você nem disponibilizar o addons do sale-workflow na sua implementação.

[rodmad85]

No caso eu utilizo este módulo sale_discount_display_amount. Eu teria de fazer uma adaptação e enviar um PR?

[antoniospneto]

@rodmad85 mas ai não faz sentido, você não quer o módulo sales instalado mas quer o módulo sale_discount_display_amount que extende a funcionalidade do sales?

[rodmad85]

Eu quero utilizar o modulo Sales sem utilizar o l10n_br_sale. Meu setor de vendas não iria preparar diretamente os dados fiscais p emissão da NFE

[antoniospneto]

@rodmad85 ah acho que agora entendi hehe

você precisa fazer um PR aqui: o l10n_br_account altera o comportamento do modelo account_tax, vai ser necessario
fazer algum tipo de validação ali pra esse compute só funcionar quando for uma linha da fatura, não uma linha de pedido:

File "/opt/odoo/extra-addons/l10n-brazil/l10n_br_account/models/account_tax.py", line 235, in _compute_taxes_for_single_line
fiscal_taxes=line.fiscal_tax_ids,

Edit, talvez apenas verificar se a linha tem um fiscal_tax_ids já resolve, e ai por uma nota: que é para permitir o funcionamaneto quando o l10n_br_sales não tá instalado.

[rvalyi]

vc poderia botar um patch na sua instalação tirando o "auto_install": True aqui: https://github.com/OCA/l10n-brazil/blob/16.0/l10n_br_sale/__manifest__.py#L33

Ou vc consegue viver com um patch local ou pode partir para usar essa ferramenta git-aggregator: https://github.com/acsone/git-aggregator

Porem no geral não pareceria legal tirar o auto_install do l10n_br_sale aqui no projeto pois para 99% do pessoal, o ideal vai ser instalar sim o l10n_br_sale neste tipo de situação e sem o auto_install existira o risco da pessoa esquecer do modulo. Na Akretion estamos com algumas ideias para tornar o l10n_br_sale menos invasivo tb e iremos propor uns PRs neste sentido nas proximas semanas. Ai de repente ele nem iria mais te atrapalhar no seu caso de uso.

[rodmad85]

Por enquanto não irei utilizar o l10n_br_account então irei esperar estas atualizações, caso eu precise utilizar antes eu crio um PR aqui. Obrigado a todos!

[mbcosta]

@rodmad85 recentemente eu fiz um PR no l10n_br_sale para permitir o uso dos casos Sem Operação Fiscal a alteração especifica é d43c2ee

Então se tiver interesse e disponibilidade você pode ver de ajudar na Revisão do PR ou em outros que buscam essa compatibilidade. Para funcionar de acordo com o que você espera será preciso apagar a Operação Fiscal Padrão de Vendas no cadastro da Empresa, com isso o campo apesar de aparecer no Pedido de Vendas deverá estar vazio ao criar novos Pedidos.

Um contexto sobre o assunto, existe um esforço do projeto para que tanto no l10n_br_sale quanto em outros módulos para que casos de uso semelhantes ao seu sejam compatíveis com a localização desde a v14

• [14.0][REF] l10n_br_sale: Adaptando o modulo para manter a compatibilidade com os casos internacionais #2743
• [14.0][REF] l10n_br_sale: Adaptando o modulo para ter compatibilidade com o Caso Internacional ou sem Operação Fiscal #2861

Isso é importante para o caso de uso Fora do Brasil que é onde em uma mesma base de dados de multi-empresas existe tanto uma empresa do Brasil e outra(s) de outro país, multi-nacionais, sobre o seu caso é importante você avaliar

• [14.0][BUG][l10n_br_account] Impossibilidade de inserir impostos em uma fatura gerada por uma SO que não tem operação declarada.  #2877

OBS.: O módulo que você esta testando é o l10n_br_sale sem S, certo? Porque na v18 a Odoo incluiu no repo deles um módulo l10n_br_sales com S então aqui é importante saber se isso não é algo referente a versão enterprise ou algum módulo pago que não faz parte do projeto, porque se for você vai precisar ver o problema em outro repo

[rodmad85]

Utilizo a versão da OCA mesmo, quanto a operação fiscal eu também necessito dela, só que o Departamento de Vendas quer somente fazer as propostas sem precisar de cuidar da parte fiscal dela. Então o ideal seria esse mapeamento parcial mesmo. Creio que mesmo em empresas maiores (no caso somos Simples), a parte fiscal geralmente é em outro setor. Então quanto mais simplificado for a proposta melhor.

[rodmad85]

Tem algum código q faça isso de configurar os produtos baseado na operação fiscal sem a necessidade do preenchimento pelo l10n_br_sale? Na teoria poderia até acabar "eliminando" a janela de preenchimento do produto deixando a checagem dos dados faltantes na emissão da NF.