[14.0][FIX] l10n_br_stock_account: Ao "Marcar para Faturar" é preciso atualizar as linhas com a Operação Fiscal e a Linha de Operação Fiscal #3497
Conversation
|
Hi @renatonlima, |
There was a problem hiding this comment.
[Funcional Review] [Approved]
@mbcosta muito obrigado pela atenção.
Fiz o teste:
- Criei um tipo de Operação de Transferencia (stock.picking.type), com uma Operação fiscal padrão
- Criei uma transferência interna usando esse Tipo de Operação.
- Marquei "Set do be invoiced"
- Criado a Fatura
OK Funcionou lindamente.
Evidências:
Uma observação importante, é que para que a Operação Fiscal seja preenchida e possível de ser fatura é que tenha um endereço de entrega (Parceiro). Caso contrário, a fatura, mesmo com os campos disponíveis de Operação e Linha de Operação disponíveis para serem preenchidos, eles não serão persistidos e não será possível.
Ao meu ver, isso pode ser alvo de uma nova PR. Seja para incluir alguma validação no botão "Set do be invoiced" ou na criação da possibilidade de converter essa fatura "Não contém doc Fiscal" para uma fatura "contém doc Fiscal"
|
/ocabot merge patch |
|
This PR looks fantastic, let's merge it! |
|
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-3497-by-rvalyi-bump-patch. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
/ocabot merge patch |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 378cd93. Thanks a lot for contributing to OCA. ❤️ |
Update line Fiscal OP
Ao "Marcar para Faturar" é preciso atualizar as linhas com a Operação Fiscal, PR simples mas aqui é preciso confirmar se atende as possibilidades de formas de uso, por exemplo se o Usuário inicialmente já definir que uma Ordem de Seleção/Picking deve ser Faturada o problema não aparecia e na visão tanto do picking quanto nas Linhas já são mostradas as informações Fiscais
O que parece ser a melhor forma de usar por já mostrar esses campos, porém existe outras formas de uso como foi descrito no issue #3478 que é
Perceba que o Marcar para Faturar é definido no final do processo e da forma que o código estava as Linhas não eram atualizadas com a Operação Fiscal isso criava uma Fatura sem Documento Fiscal e uma vez criada dessa forma não é possível associar a Fatura a um Documento Fiscal.
Agora com esse PR mesmo que o Marcar para Faturar seja informado no final do processo as Linhas serão atualizadas
Como não foi informado antes que o Picking vai gerar uma Fatura os Dados Fiscais não são mostrados
Agora ao Marcar para Faturar o programa vai preencher a Operação Fiscal e chamar o _onchange_product_id_fiscal e assim preencher a Linha de Operação fiscal
Com isso a Fatura criada já deverá ter os Dados Fiscais informados nas Linhas e o Documento Fiscal também será criado
Esse PR deve resolver o issue #3295 e de certa forma vai evitar o erro visto no issue #3478 , porque apesar do Douglas não ter relacionado o erro era justamente a falta da Operação Fiscal que causava o problema
OBS.: Sobre o issue #3478 é preciso avaliar se esse caso onde A partir de uma Fatura gerada de um objeto Sem Operação Fiscal deve ser possível Associar/Criar um Documento Fiscal é um problema/issue que pode ser resolvido ou mesmo se isso é algo sem solução devido a forma que foi feita a implementação, e nesse caso incluir um Warning para Confirmar com o Usuário se deve ou não Gerar um Documento Fiscal e assim evitar o problema. As alterações que foram feitas nos módulos l10n_br_sale, l10n_br_purchase, l10n_br_stock_account e derivados para que a Localização atenda tanto os casos das Faturas Com ou Sem Documento Fiscal acabaram gerando essa situação, porque antes estava sempre criando um Documento Fiscal, e hoje o programa usa a Operação Fiscal para identificar se deve Criar ou Não o Documento Fiscal, então em todos os casos tanto no Pedido de Vendas, Compras ou Ordem de Seleção onde se quer gerar um Documento Fiscal é preciso informar a Operação Fiscal para não ter esse problema, isso já havia sido debatido em outro issue [14.0][BUG][l10n_br_account] Impossibilidade de inserir impostos em uma fatura gerada por uma SO que não tem operação declarada. e no PR #3327 o Wesley também comentou sobre isso com a recomendação de um Warning
Mas isso é algo fora do escopo desse PR, já que uma vez informado a Operação Fiscal no objeto que vai gerar a Fatura essa necessidade de associar/criar um Documento Fiscal deixa de existir.
@douglascstd @kaynnan por favor verifiquem se essas alterações resolvem os erros dos issues mencionados
cc @OCA/local-brazil-maintainers @WesleyOliveira98