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

[8.0] NFe 4.00 - Apenas UM Modo de Pagamento por Nota #606

Closed
wants to merge 20 commits into from

Conversation

mbcosta
Copy link
Contributor

@mbcosta mbcosta commented Jun 6, 2018

WIP mapping new/removed/changed fields in the new layout
PRs related
odoo-brazil/odoo-brazil-eletronic-documents#146
odoo-brazil/PySPED#31

IMPORTANT : Libreoffice is a new dependency of PySPED

Trabalho em Andamento, mapeando os campos novos/removidos/alterados, até o momento foi possível gerar uma NFe 4.00 de uma empresa do Simples Nacional para um cliente do mesmo regime tributário no estado de São Paulo

Nota técinica usada - nt_2016_002_v1.42-1.pdf - http://www.nfe.fazenda.gov.br/portal/listaConteudo.aspx?tipoConteudo=tW+YMyk/50s=

Copy link
Member

@mileo mileo left a comment

Choose a reason for hiding this comment

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

O vDup pode receber uma lista de pagamentos exemplo:
Valor da NF - 1050,00

Pagamentos:

  1. Dinheiro: 200,00;
  2. Boleto:
    2.1 - 10/10/2018 : R$ 200,00;
    2.2 - 10/11/2018 : R$ 300,00;
  3. Cartão de Credito:
    3.1 10/11/2018: R$ 100,00;
    3.2 10/12/2018: R$ 100,00;
    3.3 10/01/2019: R$ 100,00;
    3.3 10/02/2019: R$ 100,00;
    Troco: R$ 50,00

E esses dados precisam ser contabilizados no account.

Sugiro você criar um modelo específico para salvar esses dados e depois pensarmos na contabilização.

@mbcosta
Copy link
Contributor Author

mbcosta commented Jun 12, 2018

ola @mileo , obrigado pela revisão, eu vi isso e coloquei um TODO no código( isso não impede testes ou o uso de quem trabalha com apenas uma forma de pagamento em cada NFe). Deixei assim porque acredito que precisamos pensar na melhor forma de implementar e na usabilidade, as possibilidades são criar um novo objeto ou usar o acccount.payment.term ou o payment.mode; pelo que vi até agora a melhor opção seria usar o payment.mode e dentro do objeto criar as linhas com as informações de percentuais que seriam pagos em cada modo de pagamento ( como é feito hoje no payment.term referente aos vencimentos ) e depois teríamos que ver a parte contábil

cc @renatonlima @rvalyi

@mbcosta mbcosta force-pushed the 8.0-nfe_4.00 branch 2 times, most recently from d333c60 to ab6d460 Compare July 11, 2018 19:01
@mbcosta mbcosta changed the title [8.0][WIP] NFe 4.00 [8.0] NFe 4.00 - Apenas UM Modo de Pagamento por Nota Aug 27, 2018
@mbcosta
Copy link
Contributor Author

mbcosta commented Aug 27, 2018

Implementado alterações que entram em 01/09/2018

PySPED - odoo-brazil/PySPED#35

@Daniel-Furcht
Copy link
Contributor

Ola. Ao criar um database e instalar a localização aparece o seguinte erro ao final do processo:
2018-08-28 00:03:52,088 12767 ERROR Computec_04 openerp.addons.base.ir.ir_cron: Call of self.pool.get('account.product.fiscal.classification').update_due_ncm(cr, uid, *()) failed in Job 7
Traceback (most recent call last):
File "/opt/odoo/openerp/addons/base/ir/ir_cron.py", line 137, in _callback
getattr(model, method_name)(cr, uid, *args)
File "/opt/odoo/openerp/api.py", line 268, in wrapper
return old_api(self, *args, **kwargs)
File "/opt/odoo/openerp/api.py", line 372, in old_api
result = method(recs, *args, **kwargs)
File "/opt/odoo/localizacao/l10n-brazil/l10n_br_account_product/models/account_product_fiscal_classification.py", line 281, in update_due_ncm
[1]).ibpt_update_days
File "/opt/odoo/openerp/fields.py", line 841, in get
return record._cache[self]
File "/opt/odoo/openerp/models.py", line 6057, in getitem
return value.get() if isinstance(value, SpecialValue) else value
File "/opt/odoo/openerp/fields.py", line 56, in get
raise self.exception
MissingError: ('MissingError', u'Um dos documentos que voc\xea est\xe1 tentando acessar foi exclu\xeddo. Atualize sua tela.')

@mbcosta
Copy link
Contributor Author

mbcosta commented Aug 29, 2018

ola @Daniel-Furcht obrigado pela revisão

O erro não impede o uso e não está relacionado com esse PR.

Isso é referente a uma funcionalidade que atualiza automaticamente o valor de Impostos Estimados por NCM usando um app do site do IBPT ( http://iws.ibpt.org.br/ ), se você se cadastrar no site você terá um token que deverá ser informado em Configurações > Configuração > Contabilidade

image

Podemos talvez buscar uma forma do programa não tentar atualizar quando não existir token, porém vai ser preciso ser feito em outro PR

('02', u'02 - Cheque'),
('03', u'03 - Cartão de Crédito'),
('04', u'04 - Cartão de Débito'),
('06', u'05 - Crédito Loja'),
Copy link
Member

Choose a reason for hiding this comment

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

O código deveria ser o mesmo da descrição?

Copy link
Member

Choose a reason for hiding this comment

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

Código esta errado os valores corretos do meio de pagamento são:

01=Dinheiro
02=Cheque
03=Cartão de Crédito
04=Cartão de Débito
05=Crédito Loja
10=Vale Alimentação
11=Vale Refeição
12=Vale Presente
13=Vale Combustível
15=Boleto Bancário
90=Sem Pagamento
99=Outros

Copy link
Member

Choose a reason for hiding this comment

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

(repare que não existe mais o código 14=Duplicata Mercantil)

('15', u'15 - Boleto Bancário'),
('90', u'90 - Sem pagamento'),
('99', u'99 - Outros')
], string='Tipo de Pagamento da NF', required=True,
Copy link
Member

Choose a reason for hiding this comment

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

Deve ser removido o required deste campo, pois caso tente criar uma Fatura de Cliente ou Fornecedor (sem ser um documento fiscal) vai gerar erro

('02', u'02 - Cheque'),
('03', u'03 - Cartão de Crédito'),
('04', u'04 - Cartão de Débito'),
('06', u'05 - Crédito Loja'),
Copy link
Member

Choose a reason for hiding this comment

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

Código esta errado os valores corretos do meio de pagamento são:

01=Dinheiro
02=Cheque
03=Cartão de Crédito
04=Cartão de Débito
05=Crédito Loja
10=Vale Alimentação
11=Vale Refeição
12=Vale Presente
13=Vale Combustível
15=Boleto Bancário
90=Sem Pagamento
99=Outros

('02', u'02 - Cheque'),
('03', u'03 - Cartão de Crédito'),
('04', u'04 - Cartão de Débito'),
('06', u'05 - Crédito Loja'),
Copy link
Member

Choose a reason for hiding this comment

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

Código esta errado os valores corretos do meio de pagamento são:

01=Dinheiro
02=Cheque
03=Cartão de Crédito
04=Cartão de Débito
05=Crédito Loja
10=Vale Alimentação
11=Vale Refeição
12=Vale Presente
13=Vale Combustível
15=Boleto Bancário
90=Sem Pagamento
99=Outros

Copy link
Member

Choose a reason for hiding this comment

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

(repare que não existe mais o código 14=Duplicata Mercantil)

('02', u'02 - Cheque'),
('03', u'03 - Cartão de Crédito'),
('04', u'04 - Cartão de Débito'),
('06', u'05 - Crédito Loja'),
Copy link
Member

Choose a reason for hiding this comment

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

(repare que não existe mais o código 14=Duplicata Mercantil)

@mbcosta mbcosta force-pushed the 8.0-nfe_4.00 branch 4 times, most recently from 954ac86 to 7875d14 Compare May 16, 2019 20:52
@mbcosta
Copy link
Contributor Author

mbcosta commented May 16, 2019

Ola @hendrixcosta @renatonlima corrigi os problemas que vocês comentaram, sobre o código 14=Duplicata Mercantil existia no leiaute da Nota Técnica quando foi feito o PR mas depois realmente foi removido nessa outra nota tecnica de 10/09/2018 http://www.nfe.fazenda.gov.br/portal/exibirArquivo.aspx?conteudo=zfWxcJtOf98=

@mileo
Copy link
Member

mileo commented Jan 29, 2020

Can we close?

@mileo
Copy link
Member

mileo commented Apr 7, 2020

Too old, closing.

@mileo mileo closed this Apr 7, 2020
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