Feature/accounting invoice delivery costs

fixes only issue #112 , does not include issue #99.
Do ponto de vista do Odoo:

Anteriormente já era criado um registro no account.invoice.tax mas isso
acontecia apenas durante a criação da invoice a partir do valor lançado no
pedido.

O que causava alguns problemas:
1 - caso vc retirasse um item da invoice, os valores totais não eram
recalculados. Causando rejeição na nfe.
2 - Ao agrupar dois ou mais stock.pinking para gerar uma invoice os
lançamentos não eram corrigidos.  Causando rejeição na NFE.
3 - Não haviam contas diferentes para reembolsos.

Ou seja funcionava somente no fluxo do pedido caso ele fosse faturado sem
alterações.

Colocando o calculo no motor de impostos permitimos que todos os objetos
que utilizam o compute seja beneficiado.

Do ponto de vista contábil continua do mesmo jeito, mapeado apenas para
fretes inclusos no total da NFE. Tem boas explicações no google referentes
aos tipos de frete e a contabilidade.

Colocaremos o aviso durante o sprint dessa semana.

Obrigado!

Abraços

Luis Felipe Miléo
KMEE
+55 35 8876-3663
+55 21 4063-8872
mileo@kmee.com.br
kmee.com.br
Em 13/12/2014 11:24, "Danimar Ribeiro" <notifications@github.com> escreveu:

> In l10n_br_account_product/account.py
> <https://github.com/OCA/l10n-brazil/pull/113#discussion-diff-21788122>:
>
> > @@ -66,6 +66,44 @@ def _compute_tax(self, cr, uid, taxes, total_line, product, product_qty,
> >          result['taxes'] = taxes
> >          return result
> >
> > +    def _compute_costs(self, cr, uid, insurance_value, freight_value, other_costs_value, context=False):
> > +        result = []
> > +        total_included = 0.0
> > +
> > +        company = self.pool.get('res.users').browse(cr, uid, [uid], context=context)[0].company_id
> > +
> > +        costs = {
> > +            company.insurance_tax_id : insurance_value,
> > +            company.freight_tax_id : freight_value,
> > +            company.other_costs_tax_id : other_costs_value,
>
> É exatamente este o problema, os objetos tax são nulos.
> Se ele são obrigatórios ao cálculo então seria melhor lançar uma mensagem
> de erro.
> Qual a necessidade de ter um imposto relacionado ao frete, seguro e outras
> despesas?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OCA/l10n-brazil/pull/113/files#r21788122>.
>