-
-
Notifications
You must be signed in to change notification settings - Fork 281
[16.0][IMP] l10n br fiscal: add nfse nacional support #4340
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
base: 16.0
Are you sure you want to change the base?
[16.0][IMP] l10n br fiscal: add nfse nacional support #4340
Conversation
|
Hi @renatonlima, @rvalyi, |
f211dd0 to
ae6184a
Compare
|
@rvalyi deixei mais completa a implementação, basicamente adicional o codigo nacional em todo lugar que aparece o codigo municipal |
|
@rvalyi testando em outra branch ja deu erro o tamanho do nome da tabela de relacionamento |
ae6184a to
90d762d
Compare
90d762d to
2580c39
Compare
antoniospneto
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pessoal, esse novo campo “Código de Tributação Nacional” é exclusivo de NFS-e (serviços / padrão nacional), certo? No l10n_br_fiscal, por ser bem abrangente, a implementação fica genérica e pode criar ambiguidade com outros documentos. Eu sugiro explicitar o escopo no nome/ajuda e/ou colocar em um módulo específico de NFS-e.
o nome correto para o padrao nacional seria codigo_tributacao_nacional_iss o municipal (atual city taxation code) seria codigo_tributacai_municipal_iss Acredito que sim, adicionar um help ja vai ajudar a entender |
2580c39 to
ddbef67
Compare
ccbb113 to
da37d11
Compare
da37d11 to
9848ad9
Compare
|
@antoniospneto encurtei.. o City Taxation Code mantive a mesma string mas adicionei help |
Mas e ai deixamos o codigo no l10n_br_fiscal mesmo? |
Eu entendo que sim. Até porque a tendencia é cada vez mais ir para o NFSe padrão Nacional. |
|
Tentei mesmo achar uma alternativa sem precisar criar o modelo mas nao achei.. Em alguns locais da a atender que seria o NBS mas nao é |
|
Mas poderia ir no l10n_br_fiscal_nfse, não? |
|
Posso trabalhar na separação de tudo do nfse depois em outra PR, mas soh falta esse agora e acho que segregar vai ser meio pesado.. Dai entra agora com o suporte a NFSe Nacional e trabalho na separação em seguida.. bora ? :D |
|
O municpal já temos e não é obrigatório (mas vai de cidade a cidade decidir pelo que entendi). Na primeira versão até usei ele para viabilizar a emissão mas não róla. ai ficou faltando só esse nacional mesmo dos obrigatórios |
|
Além dessa segregação, outra atividade planejada aqui é dar um bom tapa no l10n_br_nfse.. dar uma reorganizada/limpada nele. |
Mas teria outras coisas para separar? Do que vc tinha colocado antes como o l10n_br_fiscal.operation.indicator eu tinha confirmado que isso seria usado por outros documentos fiscais, como CTe então fazia sentido botar no l10n_br_fiscal. Esse l10n_br_fiscal.national.taxation.code eu não fui verificar, mas apenas levantei de novo a colocação do @antoniospneto . Então não entendi bem @marcelsavegnago , supondo que o @antoniospneto estivesse certo, teria alguma outra coisa para extrair pro l10n_br_fiscal_nfse? Talvez não e ai não seria complexo fazer ja. Disclaimer eu não parei para pensar muito sobre ainda. |
|
Se for seguir a logica do que se aplica apenas a tal documento, teriamos outros campos.. o proprio codigo de tributação municipal, o service type talvez.. acho que seria melhor fazer isso com mais calma.. segregar agora apenas um campo não sei se é vantagem.. bomm sem olhar ou pensar muito, nao me parece pratico injetar o campo em todos os locais que precisa. Por isso proponho fazer já visando os demais campos tbm.. e de certa forma, seguindo a mesma logica estariamos caminhando para uma arquitetura do tipo l10n_br_fiscal_cte, _mdfe e até _nfe... o que nao vejo nada de errado, a questao seria definir o que é comum e o que não. Mas faria isso em outra PR. |
|
Outro ponto que fiquei na duvida é como ficaria com o campo nos módulos de contratos, sale e afins ? o ideal seriam aqueles campos irem para uma aba do fiscal igual os impostos para nao ter que implementar nas views de cada módulo.. certo ? |
|
Eu gosto muito da ideia do fatiamento, mas neste caso pode ser apenas resistência minha mas não está me parecendo trivial :D. Vou tentar dormir antes das 2 da madruga e deixar a cabeça processando hehehe |




Extração da PR #4323
Mudanças Principais
Adicionado suporte para Código Tributário Nacional em documentos fiscais
Novos campos e views para gerenciamento de códigos tributários nacionais
Integração com produtos e definições de impostos para NFSe Nacional
Atualização das views de documentos fiscais para suportar NFSe Nacional
Melhorias na interface para seleção e configuração de códigos tributários