[14.0][BUG] l10n_br_sale: Frete, Seguro e Outros Custos informados pelos Totais

[mbcosta]

No l10n_br_sale quando o parâmetro para definir os campos Frete, Seguro e Outros Custos por Valor Total ( a outra opção é por Linha ) apenas um campo está sendo salvo por vez, usando os dados de demonstração ao tentar preenche-los por exemplo com o valor 100 acaba ficando da forma abaixo:

Apesar dos três campos estarem preenchidos apenas um é salvo

O calculo para ratear os valores na linha parece certo

Até onde entendi o problema pode estar em algo do método _compute_amount do modulo l10n_br_fiscal https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_fiscal/models/document_fiscal_mixin_methods.py#L38 ou está faltando ou precisa ser incluído algo nos métodos herdados porque é possível verificar que no inicio do método os campos estão com os valores informados mas ao final eles já não tem o mesmo valor, bom isso é apenas uma hipótese estou procurando entender e testar melhor, testei incluindo o código abaixo antes desse doc.update(values) https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_fiscal/models/document_fiscal_mixin_methods.py#L54 é isso funciona

        if doc.company_id.delivery_costs == "total":
            values["amount_freight_value"] = doc.amount_freight_value
            values["amount_insurance_value"] = doc.amount_insurance_value
            values["amount_other_value"] = doc.amount_other_value

Tentei incluir esse mesmo código nos métodos herdados no l10n_br_sale https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_sale/models/sale_order.py#L213 mas não funcionou.

Talvez o código esteja considerando que os valores Totais são sempre a soma dos Valores das Linhas, o que realmente são a maioria dos casos, mas teria essa exceção a ser considerada.

Existe uma questão pendente que pode acabar sendo resolvida devido a esse problema, essa funcionalidade de permitir informar os campos Frete, Seguro e Outros Custos tanto por Linha quanto por Total deveria já ser possível nas Notas Fiscais, quer dizer mesmo que apenas o l10n_br_fiscal ou o l10n_br_account estejam instalados sem o l10n_br_sale isso já deveria ser possível, mas isso pode ser algo no futuro, um Roadmap,

cc @renatonlima @rvalyi @marcelsavegnago @mileo

[douglascstd]

Imagino que para cada um dos campos de custo extra, deveria existir uma seleção de conta contábil para onde seria enviado esse custo, imaginando que se não fosse inserido, ele aplicaria a mesma conta de despesa do produto. O mesmo poderia ser utilizado para os custos gerais:

• Se informado usaria a conta informada
• Se não informado, usaria a conta do produto de cada uma das linhas Acredito que deveria ter um checkbox com a pergunta: Os custos deverão ser rateados nas linhas? ( um rateio entre todas as linhas da fatura )

[antoniospneto]

@douglascstd mas o certo é contabilizar esses valores tudo na conta do produto, da forma que está, não? por exemplo esses valores na nota fiscal vão ser tudo somado na conta "receita bruta dos produtos vendidos" normalmente é essa a conta padrão que está no produto.. O custo de fato desses valores extras vão ser lançados em outro momento, por exemplo o frete, a despesa vai ser lançada quando fazer o registro do conhecimento de frete que vai vir da transportadora.

Depois no balanço contábil que vai ser se calculado a receita liquida, que vai ser a receita bruta que vem da fatura menos as despesas/custos..

bom pelo menos é assim que costumo fazer hehe

[marcelsavegnago]

• Os custos deverão ser rateados nas linhas? ( um rateio entre todas as linhas da fatura )

Acho que isso já tem. No caso a configuração é por empresa mas eh possível forçar para ser por total mesmo que seja por linha

[marcelsavegnago]

Correção.. o forçar ainda nao foi aceito.. mas esta proposto nesta PR https://github.com/OCA/l10n-brazil/pull/1988

https://github.com/OCA/l10n-brazil/pull/1988/commits/ff19b1916403ed7fb9f16dc6873adb5d7bdb6128

https://github.com/OCA/l10n-brazil/pull/1988/commits/d8aae123b618f472a26ba179f9ffa5f5594ac61d

[antoniospneto]

@mbcosta com o merge do https://github.com/OCA/l10n-brazil/pull/2136 esses problemas foram resolvidos ?

[github-actions[bot]]

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.