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][BUG][l10n_br_account] Impossibilidade de inserir impostos em uma fatura gerada por uma SO que não tem operação declarada. #2877

Open
augustodinizl opened this issue Jan 17, 2024 · 2 comments
Labels

Comments

@augustodinizl
Copy link
Contributor

Ao tentar editar uma fatura criada por uma SO de serviço onde a
SCR-20240117-mwcu

operação fiscal não foi preenchida, a mesma dá o seguinte erro:
SCR-20240117-mxfl

Module

The name of the module that has a bug.
l10n_br_account

Describe the bug

A clear and concise description of what the bug is.

Com a mudança efetuada pela PR #2743 , o campo de operação deixa de ser obrigatório na SO, causando os seguinte erro ao tentarmos inserir as informações fiscais diretamente na fatura gerada após a confirmação da SO.
SCR-20240117-mxfl

To Reproduce

Affected versions:
14.0

Steps to reproduce the behavior:
1.Crie uma SO sem selecionar a operação
2.Insira um item do tipo serviço
3.Crie a fatura
4.Tente preencher na fatura os dados para a emissão de uma nfs-e
5. Não será possível salvar a mesma

Expected behavior
A clear and concise description of what you expected to happen.
Como não há movimentação de estoque, creio que seja necessário algum ajuste no código que retorna tal erro.
Additional context
Add any other context about the problem here. (e.g. OS, Python version, ...)

Ping: @marcelsavegnago, @douglascstd, @rvalyi

@mbcosta
Copy link
Contributor

mbcosta commented Jan 18, 2024

Valeu @augustodinizl por reportar, é importante entender o caso de uso que você está testando, algumas perguntas:

Por que você está criando um Pedido de Vendas sem a Operação Fiscal se você pretende que a Fatura gere um Documento Fiscal? Tem algum motivo especifico?

Porque a implementação foi pensada para quando você criar um Pedido de Vendas sem a Operação Fiscal fica sub entendido que a Fatura/Invoice não deve gerar Documento Fiscal, é o caso de uma Fatura/Invoice Internacional o padrão original do modulo account, quando tem a Operação Fiscal é o caso que vai gerar o Documento Fiscal então hoje a decisão se vai gerar ou não o Documento Fiscal precisa ser feita já na criação ou antes de Confirmar o Pedido de Vendas, porque no código os métodos que carregam os Dados Fiscais do Brasil são feitos baseados nessa informação aqui https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_sale/models/sale_order.py#L185 veja que isso está depois da verificação se tem a Operação Fiscal.

Por enquanto até onde vi esse comportamento é o ideal porque não deveria existir o caso onde o Pedido de Vendas não tem a Operação Fiscal e portanto não mostra os campos Fiscais do Brasil mas a Fatura criada tem os Dados Fiscais e gera o Documento Fiscal, mas dependendo da justificativa do seu caso de uso podemos considerar ser necessário alterar o codigo para atender.

@augustodinizl
Copy link
Contributor Author

augustodinizl commented Jan 24, 2024

Grande @mbcosta , obrigado pela explicação, o caso que presenciei, é na venda, onde os vendedores nem se preocupam com a classificação fiscal ou impostos pois a margem de lucro praticada pela empresa já prevê o recolhimento de tais impostos, e a SO é encaminhada para outro departamento para os ajustes fiscais da nf-e, onde tais ajustes seriam feitos na fatura para a geração da nf-e.

Por isso penso que podemos usar alguma variável (ou um conjunto de variáveis) tipo o país do parceiro/cliente + alguma outra variável, assim, poderíamos ter os impostos declarados na fatura. ao invés de somente usarmos o campo operação na SO para determinar se na fatura serão ou não declarados os impostos

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

No branches or pull requests

2 participants