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

[FIX]Campo CPF e CST #81

Merged
merged 2 commits into from
Dec 26, 2017
Merged

[FIX]Campo CPF e CST #81

merged 2 commits into from
Dec 26, 2017

Conversation

carcaroff
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 79.532% when pulling e6642d2 on carcaroff:master into 4f8bf2b on danimaribeiro:master.

self.string(nMr-69, self.nlin+7.5,
format_cnpj_cpf(tagtext(oNode=elem_dest, cTag='CNPJ')))
cnpj_cpf = format_cnpj_cpf(tagtext(oNode=elem_dest, cTag='CNPJ'))
if cnpj_cpf == '..-' or not cnpj_cpf:
Copy link
Owner

Choose a reason for hiding this comment

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

Porque tu faz essa verificação?
cnpj_cpf == '..-'

cCST = tagtext(oNode=el_imp_ICMS, cTag='orig') + \
tagtext(oNode=el_imp_ICMS, cTag='CST')
tagtext(oNode=el_imp_ICMS, cTag='CSOSN')
Copy link
Owner

Choose a reason for hiding this comment

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

Aqui pode ser CST ou CSOSN depende do tipo de empresa, simples ou lucro.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 79.543% when pulling 98900e0 on carcaroff:master into 4f8bf2b on danimaribeiro:master.

@danimaribeiro danimaribeiro merged commit bebc9f9 into danimaribeiro:master Dec 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants