Merge lp:~openobject-italia-core-devs/openobject-italia/l10n_it_invoice_tax_compute into lp:~openobject-italia-core-devs/openobject-italia/italian-addons

[Davide Corio]

[Lorenzo Battistini (elbati)]

Non appena qualcuno conferma che il modulo 'l10n_it_invoice_tax' effettua correttamente il calcolo delle tasse sulle righe della fattura, possiamo approvare il merge sul repository ufficiale

[Lorenzo Battistini (elbati)]

Sembrano esserci problemi con le tasse parzialmente deducibili.
Ho provato a creare una fattura fornitore con una riga da 123,57 e IVA 20% deducibile al 50%.

Cliccando su 'Compute Taxes' ottengo

ERROR:web-services:Uncaught exception
Traceback (most recent call last):
  File "/home/elbati/workspace/openerp/openerp6/server/bin/osv/osv.py", line 122, in wrapper
    return f(self, dbname, *args, **kwargs)
  File "/home/elbati/workspace/openerp/openerp6/server/bin/osv/osv.py", line 176, in execute
    res = self.execute_cr(cr, uid, obj, method, *args, **kw)
  File "/home/elbati/workspace/openerp/openerp6/server/bin/osv/osv.py", line 167, in execute_cr
    return getattr(object, method)(cr, uid, *args, **kw)
  File "/home/elbati/workspace/openerp/openerp6/server/bin/addons/account/invoice.py", line 644, in button_reset_taxes
    for taxe in ait_obj.compute(cr, uid, id, context=ctx).values():
  File "/home/elbati/workspace/openerp/openerp6/server/bin/addons/l10n_it_invoice_tax/invoice.py", line 79, in compute
    amount = tax_obj.compute_all(cr, uid, t['tax'], t['price'], 1)["taxes"][0]["amount"] #MG
KeyError: 'tax'

review: Needs Fixing

[Lorenzo Battistini (elbati)]

Per me ora il modulo 'account_invoice_tax_by_column' funziona

review: Approve

[Lorenzo Battistini (elbati)]

Si potrebbe perfezionare il modulo cercando di usare 'super' ove possibile invece di riscrivere completamente i metodi di calcolo delle tasse

[Lorenzo Battistini (elbati)]

Ecco: http://bazaar.launchpad.net/~openobject-italia-core-devs/openobject-italia/l10n_it_invoice_tax_compute/revision/105
Qualcuno vuole fare dei test?

review: Approve

[Sergio Corato (icsergio)]

Il 20/08/2011 12:10, Lorenzo Battistini - Agile BG - Domsense ha scritto:
> Review: Approve
> Ecco: http://bazaar.launchpad.net/~openobject-italia-core-devs/openobject-italia/l10n_it_invoice_tax_compute/revision/105
> Qualcuno vuole fare dei test?
L'ho provato sulla 6.0.3, per me è tutto regolare.

[Davide Corio (enlightx-deactivatedaccount)]

Allora, lato tecnico (codice, funzionamento wizard, stampe...) mi pare tutto ok, quindi il merge è ok per me.
Possiamo quindi iniziare a lavorare alle raffinatezze e all'usabilità. soprattutto per quanto riguarda wizard e stampe.

Cmq bravo Lorenzo

[Davide Corio (enlightx-deactivatedaccount)]

Ooooops, ho fatto casino :-)

per qualche link confuso stavo testando il branch sui registri iva.

su questo branch invece ho trovato un errore di calcolo con aliquote parzialmente deducibili

[Davide Corio (enlightx-deactivatedaccount)]

In una fattura di acquisto ho inserito una riga da 1000 eur con iva 20% deducibile al 50%.

le righe tassa che vengono fuori sono:

1. 20I5a - IVA al 20% detraibile al 50% (1) base=1000.0 tassa=100.0
2. 20I5b - IVA al 20% detraibile al 50% (2) base=1000.0 tassa=900.0

alla fine il totale fattura viene da 2.000 eur di cui 1.000 di IVA

[Sergio Corato (icsergio)]

Il 23/08/2011 15:31, Davide Corio - agilebg.com ha scritto:
> In una fattura di acquisto ho inserito una riga da 1000 eur con iva 20% deducibile al 50%.
>
> le righe tassa che vengono fuori sono:
>
> 1. 20I5a - IVA al 20% detraibile al 50% (1) base=1000.0 tassa=100.0
> 2. 20I5b - IVA al 20% detraibile al 50% (2) base=1000.0 tassa=900.0
>
> alla fine il totale fattura viene da 2.000 eur di cui 1.000 di IVA
>
Non riesco a riprodurre questo errore, provo a creare un nuovo db

[Lorenzo Battistini (elbati)]

Probabilmente non stai usando le imposte che vengono configurate dal nuovo 'l10n_it' contenuto in questo branch https://code.launchpad.net/~openobject-italia-core-devs/openobject-italia/l10n_it_fix_partially_deductible_vat

[Sergio Corato (icsergio)]

Il 23/08/2011 18:46, Sergio Corato ha scritto:
> Il 23/08/2011 15:31, Davide Corio - agilebg.com ha scritto:
>> In una fattura di acquisto ho inserito una riga da 1000 eur con iva 20% deducibile al 50%.
>>
>> le righe tassa che vengono fuori sono:
>>
>> 1. 20I5a - IVA al 20% detraibile al 50% (1) base=1000.0 tassa=100.0
>> 2. 20I5b - IVA al 20% detraibile al 50% (2) base=1000.0 tassa=900.0
>>
>> alla fine il totale fattura viene da 2.000 eur di cui 1.000 di IVA
>>
> Non riesco a riprodurre questo errore, provo a creare un nuovo db
>
Anche con un db nuovo, installando solo account_invoice_tax_by_column
(nessun _it) me la calcola correttamente

[Davide Corio (enlightx-deactivatedaccount)]

ignoravo l'esistenza di quei fix.
quindi ho fatto solo tanto rumore per nulla, scusate