Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0][Anglo-saxon accounting]: Habilita anglo-saxon para COA templates e cria contas intermediárias para COA generic #2667

Merged
merged 3 commits into from
Aug 30, 2023

Conversation

DiegoParadeda
Copy link
Contributor

@DiegoParadeda DiegoParadeda commented Aug 29, 2023

Objetivo

Habilitar anglo-saxon accounting e criar contas auxiliares para o fluxo contábil.

Background


A funcionalidade anglo-saxon foi discutida de modo extensivo no PR #1561 para a versão 12.0, que ativa o modo anglo-saxon e adiciona melhorias no fluxo.

O PR ficou travado por um conflito no conceito anglo-saxon e suas funcionalidades que posso resumir brevemente aqui:

  • "Anglo-saxon accounting" é um termo sem ligação com a contabilidade e aparenta ser utilizado no Odoo apenas por motivos históricos/legado. Esse termo causou confusão e discordância por (especulando) parecer alterar funcionalidades chave da contabilidade.
  • Na realidade esse modo apenas permite a contabilização automática dos custos de estoque* no momento da venda. Este é o padrão Brasileiro (como falarei abaixo). Exemplo: sabe-se que uma compra de mercadoria para revenda gera custos. Estes podem ser lançados de duas formas:
    • No momento da própria compra: funcionamento padrão atual.
    • No momento da venda: funcionamento facilitado pelo modo anglo-saxon
      • Padrão Brasileiro e objetivo desse PR

*Contabilização automática dos custos de estoque -> Pode ser interpretado (no contexto da revenda de mercadorias) como um lançamento contábil com débito na conta Custo das Mercadorias Vendidas (CMV) e crédito na conta Estoque.

Para entender esse fluxo e embasar as soluções técnicas contratamos uma consultoria de contabilidade e realizamos testes e simulações extensas. Seguem comparações entre os fluxos demonstrando os resultados.

Simulação SEM ANGLO-SAXON:


Sem a utilização do modo anglo-saxon a configuração validada com a contabilidade foi a seguinte:
image

Repare na utilização de uma conta transitória, sem ela os lançamentos não ficam corretos (considerando contabilização no CMV na venda), e parece até não haver contas para preencher toda a configuração.

IMPORTANTE: Apesar de ser/parecer um workaround os lançamentos contábeis foram validados como corretos, veja:

Na Compra:

image
image

Na Venda:

image
image

Outras tentativas com a conta CMV no campo Expense Account também ficaram inconsistentes pelos seguintes motivos:

  1. A conta CMV é debitada no momento da compra. Mais especificamente na fatura do fornecedor:
    image

  2. Os lançamentos automáticos de estoque (tanto de entrada como de saída) não ficam certos pois não geram relação nenhuma com a conta CMV (mesmo se utilizar outras contas fora desse exemplo):
    image
    image

Simulação COM ANGLO-SAXON:


Finalmente, com modo anglo-saxon ativado os lançamentos ficam semelhantes aos estratégia da conta transitória. Por se tratar de um módulo padrão do Odoo acredito que a maioria irá concordar que é uma estratégia melhor do que o workaround e outras estratégias mencionadas.

Configuração utilizada:

image

Lançamentos na Compra:

image
image

Lançamentos na Venda:

image
image

  • com impostos zerados dessa vez

O módulo l10n_br_coa_generic é baseado no Livro_Escrituracao_contabil.pdf, de onde retirei os seguintes trechos:

Compra:
image

Venda:
image

As contas utilizadas estão ligeiramente diferentes e no Odoo os lançamentos estoque/financeiro são separados, necessitando as contas de estoque intermediário para juntá-los. De qualquer forma, o resultado dos lançamentos após simplificação bate com o do livro e com a teoria passada pela consultoria que comentei.


NOTAS:

  • Esse funcionamento também já foi testado com 2635
  • O PR do @rvalyi 1561 possui outras melhorias no fluxo como a remoção de linhas de lançamentos contábeis zeradas que também observei em alguns casos. Ainda não olhei como ficaria na 14.0 mas com certeza é válido.

@OCA-git-bot
Copy link
Contributor

Hi @renatonlima, @mileo,
some modules you are maintaining are being modified, check this out!

Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ufa fim dessa novela

@rvalyi
Copy link
Member

rvalyi commented Aug 30, 2023

/ocabot merge major

1 similar comment
@mileo
Copy link
Member

mileo commented Aug 30, 2023

/ocabot merge major

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-2667-by-mileo-bump-major, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 30, 2023
Signed-off-by mileo
@mileo
Copy link
Member

mileo commented Aug 30, 2023

/ocabot merge major

Suspeito que quando solicitado pelo app do github no celular não esta indo, isso aconteceu comigo esses dias, foi o seu caso?

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-2667-by-rvalyi-bump-major, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 30, 2023
Signed-off-by rvalyi
@rvalyi
Copy link
Member

rvalyi commented Aug 30, 2023

/ocabot merge major

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-2667-by-rvalyi-bump-major, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 10791f1 into OCA:14.0 Aug 30, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

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

@marcelsavegnago
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants