[12.0] Edição do Documento Fiscal e Fatura #1146

renatonlima · 2021-02-25T22:37:22Z

replaces #1099

rvalyi

No runbot, tentando criar uma nota de entrada escolhando apenas o parceiro Akretion e o produto customizable desk, se eu nao escolher um tipo de documento fiscal, vem essa mensagem ao salvar:

Se eu escolher algum tipo de document fiscal, como 55, ai ele deixa salvar.

A mesma coisa acontece com a nota de saida.

mileo · 2021-03-02T00:05:29Z

Ta ficando bacana, hoje peguei um fluxo em um cliente que precisou duplicar um documento fiscal para corrigir uma nota já transmitida, não sei se vc já esta pensando em cobrir isso.

Mas ele duplicou, corrigiu umas coisas e cancelou a antiga.

Mas ai o pedido de venda e os outros modelos não estavam lincados ao novo documento fiscal.

marcelsavegnago · 2021-03-05T16:17:08Z

ping @mileo talvez seja do seu interesse
https://travis-ci.com/github/OCA/l10n-brazil/jobs/488495175#L5266

marcelsavegnago · 2021-03-06T20:43:23Z

Ohh q lindo o travis verde :D

rvalyi · 2021-03-07T00:35:11Z

Ohh q lindo o travis verde :D

esse PR estamos com ele em prod desde quinta, ja com dezenas de notas emitidas agora.

marcelsavegnago

Testado no runbot.. a principio Ok.

Farei testes tbm locais mas por mim pode seguir.

rvalyi · 2021-03-08T11:54:56Z

@mileo consegue testar mais? Estamos com a ideia de fazer o merge desse refator hoje..

mileo · 2021-03-08T12:06:36Z

@mileo consegue testar mais? Estamos com a ideia de fazer o merge desse refator hoje..

Vou colocar aqui na minha lista de hj.

rvalyi · 2021-03-08T22:29:13Z

(agora 5 dias em produção em duas empresas com uns 50 usuários)

mileo · 2021-03-09T01:32:59Z

l10n_br_account/models/account_invoice.py

+ def action_move_create(self):
+ result = super().action_move_create()
+ dummy_doc = self.env.ref('l10n_br_fiscal.fiscal_document_dummy')
+ self.mapped('fiscal_document_id').filtered(
+ lambda d: d != dummy_doc).action_document_confirm()
+ return result


O lançamento contábil só deve ser realizado caso o documento fiscal seja autorizado.

@mileo,

O fluxo contábil não alterou neste PR, para enviar o documento fiscal você precisa ter a account.invoice confirmada primeiro e com o status em aberto para ser enviado os valores e duplicatas por exemplo.

Atualmente você consegue até fazer um documento fiscal, transmitir e depois confirmar a fatura, mas isso depende do usuário depois confirmar a fatura.

Eu diria que hoje pode ser configurado para o lançamento não ser postado, ficar como draft e depois que confirmado ele ser postado. Eu queria melhorar esse fluxo depois do PR #1125 porque umas das razões de validar a fatura e ter os lançamentos é pegar os vencimentos para incluir no documento fiscal. que trabalhando no PR de lançamentos podemos mudar isso, mas hoje dá para deixar os lançamentos da fatura não postado e posta-lo depois que o documento fiscal é transmitido.

@mileo para melhorar o fluxo, quando uma fatura é criada é gerada a contabilização mas não é postada, e quando é transmitida e autorizada é postada a contabilização, desta forma resolvemos o problema de ser uma contabilização postada somente quando o documento fiscal é autorizado.

eu tb vi isso com o @renatonlima e accredito que seja a melhor forma: o move do invoice fica de rascunho ate vc transmitir e depois de transmitir ele é postado. Nisso o move ja pega o numero dele (porque senao daria problema se vc deixar varias Nfe's e transmitir elas numa ordem diferente) e tb isso faz algum checagem basica que o lançamento é possivel e depois ele é apenas postado. Tb o codigo disso fica razoavel.
cc @marcelsavegnago

ping @mileo

Vou revisar esse e o dos totais hj

No #806 foi implementado boa parte desses fluxos, de confirmação do lançamento contábil e mais uns outros detalhes.

Tudo bem que tinha aquela funcionalidade embutida do motor de lançamentos contábeis, vou dar uma relembrada do que foi feito lá e ver lembro e mais algum detalhe e comento aqui.

@mileo,

Eu havia visto o PR #806 e ele mantém o fluxo separado entre documento fiscal e invoice e o fluxo esta errado porque edita o documento fiscal e nesse PR eu uso o recurso do "inherits" e o documento fiscal através da fatura (account.invoice).

Um outro ponto, sobre o motor de lançamentos contábeis eu não vejo lógica em fazer uma implementação tão grande e complexa já que o módulo do core do Odoo já faz a contabilização e com pouco código localizado é possível fazer a contabilização de partida dobrada 100% compatível com a legislação brasileira e mantendo a compatibilidade da localização com outros módulos do eco-sistema.

Esse PR também esta com algumas implementações do PR #820, que ainda tem bastante coisas para revisar e corrigir, depois deste PR e do PR dos eventos eu pretendo trabalhar no PR #820, mas antes de lidar com as particularidades dos recebiveis/pagaveis é importante a gente acertar o fluxo da edição do documento fiscal/fatura.

@mileo eu acho que na v12 a gente tem um l10n_br_base e l10n_br_fiscal limpo suficente para vc usar caso a KMEE quiser partir para essa forma de fazer de novo como vcs haviam feito no fork das v10 e v11 de vcs. Ai ja seria melhor do que dar uma viajada 100% fora da OCA como era o caso desse fork v10/v11. Se precisar de alguns hooks para facilitar isso, na medida do possivel a gente bote botar alguns. Mas nao ao ponto de copiar 700 linhas de codigo do account com copyright Odoo SA no modulo fiscal como no PR 820 e que se torna 100% inutil logo que vc instala o modulo account e usa esse tipo de PR aqui que integra bem o fiscal com o account.

E mesmo que seja legitimo refazer milhares de coisas fora do account, eu nao acho que tem lugar para esse tipo de implementaçao na OCA. Em outros paises tem pessoas que fizeram contabilidade ou payroll 100% fora dos modulos do Odoo, usando apenas o Odoo como framework. Porem isso nunca entrou na OCA em repo nehnum porque a filosofia da OCA é de se integrar com os modulos Odoo e ecosistema OCA e como o Renato falou nao é tao dificil, pelo menos a gente ta conseguindo legal.

Se essa forma de vcs era tao boa assim, fica porem a pergunta: porque sera que esse fork da Kmee não vingou? Cade o roadmap a enchação de boca toda para ficar falando desse fork de como isso era tao melhor, onde isso vai e cade os "good first issues" para a turma do Telegram ir resolver nesse fork ai? Como ta a turma que te seguiu nessa hoje? Ta com suporte legal? Tem muitos iniciantes ajudando ai? Ou talvez nao era tao bom assim... Cada uma tire as conclusões.

mileo · 2021-03-09T01:37:58Z

@renatonlima pode descrever melhor o PR? Ele altera vários fluxos.

mileo · 2021-03-09T01:49:21Z

Algo errado ao testar no runbot.

mileo · 2021-03-09T01:50:36Z

Acho que os totais não estão legais, mas pode ser coisa do runbot e dessa empresa.

mileo · 2021-03-09T01:59:45Z

Acho que os totais não estão legais, mas pode ser coisa do runbot e dessa empresa.

Confirmei que esse problema é só na base company

renatonlima · 2021-03-09T11:36:41Z

Algo errado ao testar no runbot.

Esse caso acontece porque o campo se chama state_edoc e o widget statusbar só funciona com o campo que se chama state, a principio só tem impacto visual neste PR, mas também com o PR #1176 a sinalização dos status do documento fiscal ficou mais visível do que o widget state

renatonlima · 2021-03-12T19:30:38Z

@mileo and @gabrielcardoso21 all reviews notes was fixed!

mileo · 2021-03-12T19:32:44Z

@mileo and @gabrielcardoso21 all reviews notes was fixed!

Vou dar uma olhada

mileo · 2021-03-12T19:34:51Z

Mais uma coisa que eu vi, tem uns códigos comentados, não seria melhor já tirar eles?

renatonlima · 2021-03-12T19:36:42Z

@mileo Eu deixei comentado e um todo porque depois pretendo fazer um PR para ter um parametro para escolher a visão padrão...

rvalyi · 2021-03-12T22:25:51Z

bom galera eu acho que foi respondido os pontos que foram colocados pelo @mileo e o @gabrielcardoso21
Tb ja se passou uma semana com o PR bem verdinho pronto para revisar e fazer critica fundamentada se tiver algum problema conceitual grave.
E eu @marcelsavegnago e eu aprovamos entao bora la para dar continuidade nas outras coisas...

marcelsavegnago · 2021-03-12T22:27:54Z

@rvalyi boraaa :D

rvalyi · 2021-03-12T22:30:29Z

/ocabot merge major

OCA-git-bot · 2021-03-12T22:31:14Z

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

rvalyi · 2021-03-12T22:32:03Z

/ocabot sextou

OCA-git-bot · 2021-03-12T22:32:05Z

Hi @rvalyi. Your command failed:

Invalid command: sextou.

Ocabot commands

ocabot merge major|minor|patch|nobump

More information

ocabot documentation
OCA guidelines, specially the "Version Numbers" section.

mileo · 2021-03-12T22:34:56Z

Tava testando ele agora mas manda ver.

marcelsavegnago · 2021-03-12T22:38:12Z

Hi @rvalyi. Your command failed:

Invalid command: sextou.

Ocabot commands

ocabot merge major|minor|patch|nobump

More information

ocabot documentation

OCA guidelines, specially the "Version Numbers" section.

hahahahhahahaha

OCA-git-bot · 2021-03-12T23:15:07Z

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

renatonlima mentioned this pull request Feb 25, 2021

[12.0] Edit Account Invoice and Fiscal Documents #1099

Closed

renatonlima added 12.0 work in progress labels Feb 26, 2021

renatonlima force-pushed the 12.0-l10n_br_fiscal-document-invoice branch from dffa676 to d2b1360 Compare February 26, 2021 12:48

rvalyi previously requested changes Feb 27, 2021

View reviewed changes

renatonlima mentioned this pull request Mar 1, 2021

[NEW] Res Partner Fiscal Document Relation close: #1151 #1164

Closed

renatonlima force-pushed the 12.0-l10n_br_fiscal-document-invoice branch from 4356d6f to a5e75fa Compare March 1, 2021 20:50

renatonlima force-pushed the 12.0-l10n_br_fiscal-document-invoice branch 3 times, most recently from a0ab993 to f6469a7 Compare March 3, 2021 21:06

renatonlima force-pushed the 12.0-l10n_br_fiscal-document-invoice branch from 7960a1d to ba0e473 Compare March 6, 2021 18:58

renatonlima removed the work in progress label Mar 6, 2021

marcelsavegnago approved these changes Mar 7, 2021

View reviewed changes

renatonlima force-pushed the 12.0-l10n_br_fiscal-document-invoice branch 2 times, most recently from 5baa7cf to 57011ef Compare March 8, 2021 23:19

mileo previously requested changes Mar 9, 2021

View reviewed changes

renatonlima added 6 commits March 12, 2021 14:17

[REF] post account move only after fiscal doc is authorized

25295a2

[FIX] fiscal number copy=False

aafcf5f

Unlink fiscal document line from account invoice line

098250c

pep8

2b06db1

[REM] remove action document view

c94a676

[FIX] fiscal document dashboard, button attr, val name

a82fa0b

renatonlima force-pushed the 12.0-l10n_br_fiscal-document-invoice branch from c1e1c98 to a82fa0b Compare March 12, 2021 19:06

[FIX] not use dummy line in invoice line a true display_type

8271ac6

renatonlima removed the needs fixing label Mar 12, 2021

renatonlima force-pushed the 12.0-l10n_br_fiscal-document-invoice branch from 3fab144 to faca84c Compare March 12, 2021 19:35

[FIX] Do not try unlink dummy line

34f31d6

renatonlima force-pushed the 12.0-l10n_br_fiscal-document-invoice branch from faca84c to 34f31d6 Compare March 12, 2021 19:41

OCA-git-bot added the bot is merging ⏳ label Mar 12, 2021

OCA-git-bot merged commit 8d5a673 into OCA:12.0 Mar 12, 2021

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Mar 12, 2021

[12.0] Edição do Documento Fiscal e Fatura #1146

[12.0] Edição do Documento Fiscal e Fatura #1146

Conversation

renatonlima commented Feb 25, 2021

rvalyi left a comment

Choose a reason for hiding this comment

mileo commented Mar 2, 2021

marcelsavegnago commented Mar 5, 2021

marcelsavegnago commented Mar 6, 2021

rvalyi commented Mar 7, 2021

marcelsavegnago left a comment

Choose a reason for hiding this comment

rvalyi commented Mar 8, 2021

mileo commented Mar 8, 2021

rvalyi commented Mar 8, 2021

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rvalyi Mar 12, 2021 • edited Loading

Choose a reason for hiding this comment

mileo commented Mar 9, 2021

mileo commented Mar 9, 2021

mileo commented Mar 9, 2021

mileo commented Mar 9, 2021

renatonlima commented Mar 9, 2021 • edited Loading

renatonlima commented Mar 12, 2021

mileo commented Mar 12, 2021

mileo commented Mar 12, 2021

renatonlima commented Mar 12, 2021

rvalyi commented Mar 12, 2021

marcelsavegnago commented Mar 12, 2021

rvalyi commented Mar 12, 2021

OCA-git-bot commented Mar 12, 2021

rvalyi commented Mar 12, 2021

OCA-git-bot commented Mar 12, 2021

mileo commented Mar 12, 2021

marcelsavegnago commented Mar 12, 2021

OCA-git-bot commented Mar 12, 2021

rvalyi Mar 12, 2021 •

edited

Loading

renatonlima commented Mar 9, 2021 •

edited

Loading