Merge lp://staging/~openerp-brazil-team/openerp.pt-br-localiz/vanzuita into lp://staging/openerp.pt-br-localiz/6.1
Status: | Needs review |
---|---|
Proposed branch: | lp://staging/~openerp-brazil-team/openerp.pt-br-localiz/vanzuita |
Merge into: | lp://staging/openerp.pt-br-localiz/6.1 |
Diff against target: |
849 lines (+378/-108) 3 files modified
l10n_br_account/account_invoice.py (+369/-106) l10n_br_account/wizard/nfe_export_from_invoice.py (+7/-1) l10n_br_account/wizard/nfe_export_from_invoice_view.xml (+2/-1) |
To merge this branch: | bzr merge lp://staging/~openerp-brazil-team/openerp.pt-br-localiz/vanzuita |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Leonardo Santagada @ Proge.com.br | Needs Fixing | ||
Review via email:
|
Description of the change
O tratamento p/ CST 101 mesmo configurando as regras fiscais ainda não consegui deixa-la 100%, portanto gostaria que esse tratamento não fosse incluído no merge.
Unmerged revisions
- 335. By João Vanzuita
-
Simplifiquei o codigo tirando o controle de precisao decimal que havia adicionado e que nao tinha necessidade
- 334. By João Vanzuita
-
Ajustes na exportacao do XML (ainda não está 100%), ajuste na exportação do TXT p/ que funcione com o UniNFe, nome do arquivo exportado modificado de modo que possa ser importado automaticamente pelo UniNFe, ajuste na criacao da chave NFe, digito verificador corrigido
- 333. By João Vanzuita
-
Melhorias na exportacao do XML de modo que ele possa começar a ser usado
- 332. By João Vanzuita
-
ajuste no valor do desconto na exportacao
- 331. By converge <converge@dev>
-
conflito no merge resolvido
- 330. By converge <converge@dev>
-
conflito resolvido
- 329. By converge <converge@dev>
-
tratamento para nota com desconto
- 328. By converge <converge@dev>
-
tratamento da CST 101 comentado pois a solução ainda não está boa
- 327. By João Vanzuita
-
Salva arquivos da exportação da NFe em TXT e XML no formato nfe_numero-nota.txt (ou .xml)
- 326. By João Vanzuita
-
Quando frete FOB, valor do frete é adicionado ao valor total da nota. Adicionado rateio do valor do frete para produtos da nota
passar o programa pep8 nos arquivos vai ajudar bastante com estilo, mas as principais mudanças que eu vejo necessárias são:
- documentação de função usando docstrings
- nenhum parentese desnecessário, principalmente nos ifs e quando faz formatação de string
- chamadas de str desnecessárias
- deixar a formatação da string para o ultimo passo (não fazer "%.2f" no dicionário mas sim na string final)
- if com 4 clausulas ao invés de usar ifs aninhados
- nfe_key troca de tipo, isso não deve acontecer
- clean_string na verdade é remove_pontuacao porque ela remove partes das strings (tá sendo usada na nfe_key que eu acho que não deveria).
e tem uma opcional que eu já tinha falado sobre num email, de usar parametros com nome na string de formatação dos dicionarios de parametros:
"""
reg_N10c = {
'Orig': 'nacional',
'CSOSN': '34234',
}
str_N10c = 'N10c|% (Orig)s| %(CSOSN) s' % reg_N10c
"""
abraços.