Code review comment for lp:~icsergio/openobject-italia/remake-vat-registries

Stavo guardando la query che recupera le righe da stampare:

'SELECT ait.id FROM account_invoice_tax ait, account_invoice ai, account_tax_code atc WHERE (atc.id=ait.tax_code_id AND ai.id=ait.invoice_id AND ai.state IN %s AND ai.period_id IN %s AND ai.journal_id IN %s ) ORDER BY ai.number' , (tuple(invoice_state), tuple(period_id), tuple(journal_id) )

In questo modo però vengono stampate solo le righe generate da fatture. Se l'utente facesse registrazioni dirette sui sezionali di vendita o acquisto, ad esempio tramite causale, queste non verrebbero stampate.

L'approccio "basato sulle fatture" lo avevo seguito anch'io nelle prime versioni di 'l10n_it_vat_registries'. Ma poi spesso ai clienti capitava di fare registrazioni legate all'IVA direttamente in contabilità, ad esempio per il reverse charge o anche per le fatture passive.

Probabilmente modificare 'account_vat_registries_webkit' in modo da non usare 'account_invoice_tax' potrebbe essere oneroso.
A questo punto, almeno per la 7, punterei a correggere i bachi di 'l10n_it_vat_registries'.