Merge lp:~scigghia/openobject-italia/adding_cf_piva_stampa_registri_iva into lp:~openobject-italia-core-devs/openobject-italia/italian-addons-7.0

[Andrea Cometa]

[Lorenzo Battistini (elbati)]

Ciao Andrea, puoi descrivere a cosa servono le modifiche del diff?
Grazie

review: Needs Information

[Andrea Cometa (scigghia)]

ho aggiunto sotto ogni riga anche il codice fiscale e la partita iva del cliente, sebbene non siano necessarie possono tornare utili sulla stampa.
Inoltre ho spostato la colonna totale fattura in ultima posizione

[giudalo (g-dalo)]

Di funzionare, funziona.

Concordo con Davide che non siano dati obbligatori e penso anche io che andrebbe fatto un configuratore dei registri su che colonne si vuole. In effetti la nomativa non prevede neanche la stampa del numero di registrazione e del sezionale.
Il sole 24 ore dice :

Il registro deve riportare i seguenti dati:
- data di emissione;
- numero progressivo di emissione;
- indicazione del cliente, ditta, ragione sociale o nominativo, ovvero, nei casi di autofattura, del cedente o prestatore;
- base imponibile distinta per aliquota;
- Iva distinta per aliquota.
e per il registro acquisti aggiunge solo il numero protocollo.

review: Approve

[lp:~scigghia/openobject-italia/adding_cf_piva_stampa_registri_iva]

235. By Andrea Cometa on 2014-02-06

[fix] stampa registri iva, rimosso False quando non presente p\.iva o c\.f\.

[Lorenzo Battistini (elbati)]

Come mai CF e PIVA vengono stampati solo se
line['index']!=0
?

In caso la riga avesse un solo tax.code da stampare, CF e PIVA non verrebbero stampati credo

review: Needs Information

[Andrea Cometa (scigghia)]

in realtà vengono stampati ad ogni riga tranne la 0, ho fatto una prova con due aliquote.
ora correggo e faccio in modo di farli stampare solo se line['index'] == 1

[lp:~scigghia/openobject-italia/adding_cf_piva_stampa_registri_iva]

236. By Andrea Cometa on 2014-02-08

[fix] l10n_it_vat_registries\: stampa cf e piva solo una volta

[Lorenzo Battistini (elbati)]

On 02/08/2014 10:07 AM, Andrea Cometa wrote:
> in realtà vengono stampati ad ogni riga tranne la 0, ho fatto una prova con due aliquote.

E se ci fosse una sola aliquota?

[Andrea Cometa (scigghia)]

idem, il test è fatto con diverse fatture, tra cui una con due aliquote
devo solo allineare gli importi sulle righe>1

[Lorenzo Battistini (elbati)]

On 02/09/2014 06:31 PM, Andrea Cometa wrote:
> idem, il test è fatto con diverse fatture, tra cui una con due aliquote
> devo solo allineare gli importi sulle righe>1

Mi riferivo al caso in cui ci fosse un solo tax code da stampare (non una sola aliquota come dicevo). Nel caso di un solo tax code line['index'] non andrà a 1 e la PIVA non sarà stampata.
Ma assumendo che le imposte siano configurate come in l10n_it, questo non dovrebbe accadere.

Noto però che se

line['index']!=0
o
line['index']!=1

(ovvero dopo le prime due sottorighe di ogni riga del registro) il TD non viene stampato, quindi la riga della tabella ha una colonna in meno.

review: Needs Fixing

[Andrea Cometa (scigghia)]

io uso sempre l10n_it come pdc, non ho avuto modo di testarlo su altre configurazioni, ma lo farò a breve.
per quanto riguarda il TD l'ho notato e provvederò non appena possibile

[Lorenzo Battistini (elbati)]

This project is now hosted on https://github.com/OCA/l10n-italy. Please move your proposal there. This guide may help you https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub

Unmerged revisions

236. By Andrea Cometa on 2014-02-08

[fix] l10n_it_vat_registries\: stampa cf e piva solo una volta

235. By Andrea Cometa on 2014-02-06

[fix] stampa registri iva, rimosso False quando non presente p\.iva o c\.f\.

234. By Andrea Cometa on 2014-02-06

[imp] stampa codice fiscale e partita iva su ogni riga partner, spostato totale sull\'ultima colonna

=== modified file 'l10n_it_vat_registries/reports.xml'
--- l10n_it_vat_registries/reports.xml	2013-05-09 15:34:05 +0000
+++ l10n_it_vat_registries/reports.xml	2014-02-08 09:10:47 +0000
@@ -21,7 +21,7 @@
             }
         </script>
     </head>
-    <body style="border:0; margin: 0;" onload="subst()">
+    <body style="border:0; margin: 0; font:Arial;" onload="subst()">
     <% setLang(company.partner_id.lang or "en_US") %>
     <h1>Registro I.V.A. Vendite</h1>
     <h3>${company.partner_id.name or ''|entity}</h3>
@@ -65,7 +65,7 @@
             }
         </script>
     </head>
-    <body style="border:0; margin: 0;" onload="subst()">
+    <body style="border:0; margin: 0; font:Arial;" onload="subst()">
         <table style="border-top: 1px solid black; width: 100%">
             <tr >
                 <td style="text-align:right;font-size:12;" width="95%">Page ${objects[0].period_id and objects[0].period_id.fiscalyear_id and objects[0].period_id.fiscalyear_id.name or ''|entity}/<span class="page"/></td><td style="text-align:left;font-size:12;"></td>
@@ -102,7 +102,7 @@
             }
         </script>
     </head>
-    <body style="border:0; margin: 0;" onload="subst()">
+    <body style="border:0; margin: 0; font:Arial;" onload="subst()">
     <% setLang(company.partner_id.lang or "en_US") %>
     <h1>Registro I.V.A. Acquisti</h1>
     <h3>${company.partner_id.name or ''|entity}</h3>
@@ -146,7 +146,7 @@
             }
         </script>
     </head>
-    <body style="border:0; margin: 0;" onload="subst()">
+    <body style="border:0; margin: 0; font:Arial;" onload="subst()">
         <table style="border-top: 1px solid black; width: 100%">
             <tr >
                 <td style="text-align:right;font-size:12;" width="95%">Page ${objects[0].period_id and objects[0].period_id.fiscalyear_id and objects[0].period_id.fiscalyear_id.name or ''|entity}/<span class="page"/></td><td style="text-align:left;font-size:12;"></td>
@@ -183,7 +183,7 @@
             }
         </script>
     </head>
-    <body style="border:0; margin: 0;" onload="subst()">
+    <body style="border:0; margin: 0; font:Arial;" onload="subst()">
     <% setLang(company.partner_id.lang or "en_US") %>
     <h1>Registro I.V.A. Corrispettivi</h1>
     <h3>${company.partner_id.name or ''|entity}</h3>
@@ -227,7 +227,7 @@
             }
         </script>
     </head>
-    <body style="border:0; margin: 0;" onload="subst()">
+    <body style="border:0; margin: 0; font:Arial;" onload="subst()">
         <table style="border-top: 1px solid black; width: 100%">
             <tr >
                 <td style="text-align:right;font-size:12;" width="95%">Page ${objects[0].period_id and objects[0].period_id.fiscalyear_id and objects[0].period_id.fiscalyear_id.name or ''|entity}/<span class="page"/></td><td style="text-align:left;font-size:12;"></td>

=== modified file 'l10n_it_vat_registries/templates/registro_iva_acquisti.mako'
--- l10n_it_vat_registries/templates/registro_iva_acquisti.mako	2013-08-22 11:09:52 +0000
+++ l10n_it_vat_registries/templates/registro_iva_acquisti.mako	2014-02-08 09:10:47 +0000
@@ -29,9 +29,9 @@
             <th class="left_without_line">Numero fattura</th>
             <th class="left_without_line">Data fattura</th>
             <th class="left_without_line">Sezionale</th>
-            <th class="right_without_line">Totale fattura</th>
             <th class="right_without_line">Imposta</th>
             <th class="right_without_line">Importo</th>
+            <th class="right_without_line">Totale fattura</th>
             <th></th>
         </tr>
         </thead>
@@ -57,12 +57,12 @@
                     ${ counter | entity}
                 %endif
                 %if line['index']==0:
-                    </td><td class="left_with_line">
+                    </td><td class="left_with_line">${object.partner_id.name or ''| entity}
                 %else:
-                    </td><td class="left_without_line">
-                %endif
-                %if line['index']==0:
-                    ${object.partner_id.name or ''| entity}
+                    %if line['index']==1:
+                        </td><td class="left_without_line" nowrap>${object.partner_id.fiscalcode and 'C.F.: ' or ''| entity}${object.partner_id.fiscalcode or ''| entity}
+                        <br/>${object.partner_id.vat and 'P.IVA: ' or ''| entity}${object.partner_id.vat or ''| entity}
+                    %endif
                 %endif
                 %if line['index']==0:
                     </td><td class="left_with_line">
@@ -96,14 +96,6 @@
                 %if line['index']==0:
                     ${object.journal_id.name or ''| entity}
                 %endif
-                %if line['index']==0:
-                    </td><td class="right_with_line">
-                %else:
-                    </td><td class="right_without_line">
-                %endif
-                %if line['index']==0:
-                    ${ formatLang(invoice_total(object)) | entity}
-                %endif
                 </td>
                 %if line['index']==0:
                     <td class="right_with_line">${ (line['tax_code_name'])  or ''| entity}</td>
@@ -115,6 +107,15 @@
                 %else:
                     <td class="right_without_line">${ formatLang(line['amount'])| entity}</td>
                 %endif
+                %if line['index']==0:
+                    <td class="right_with_line">
+                %else:
+                    <td class="right_without_line">
+                %endif
+                %if line['index']==0:
+                    ${ formatLang(invoice_total(object)) | entity}
+                %endif
+                    </td>
                 </tr>
             %endfor
         %endfor

=== modified file 'l10n_it_vat_registries/templates/registro_iva_corrispettivi.mako'
--- l10n_it_vat_registries/templates/registro_iva_corrispettivi.mako	2013-08-22 11:09:52 +0000
+++ l10n_it_vat_registries/templates/registro_iva_corrispettivi.mako	2014-02-08 09:10:47 +0000
@@ -25,9 +25,9 @@
             <th class="left_without_line">Numero</th>
             <th class="left_without_line">Data registrazione</th>
             <th class="left_without_line">Sezionale</th>
-            <th class="right_without_line">Importo totale</th>
             <th class="right_without_line">Imposta</th>
             <th class="right_without_line">Importo</th>
+            <th class="right_without_line">Importo totale</th>
             <th></th>
         </tr>
         </thead>
@@ -60,14 +60,6 @@
                 %if line['index']==0:
                     ${object.journal_id.name or ''| entity}
                 %endif
-                %if line['index']==0:
-                    </td><td class="right_with_line">
-                %else:
-                    </td><td class="right_without_line">
-                %endif
-                %if line['index']==0:
-                    ${ formatLang(invoice_total(object)) | entity}
-                %endif
                 </td>
                 %if line['index']==0:
                     <td class="right_with_line">${ (line['tax_code_name'])  or ''| entity}</td>
@@ -79,6 +71,15 @@
                 %else:
                     <td class="right_without_line">${ formatLang(line['amount'])| entity}</td>
                 %endif
+                %if line['index']==0:
+                    <td class="right_with_line">
+                %else:
+                    <td class="right_without_line">
+                %endif
+                %if line['index']==0:
+                    ${ formatLang(invoice_total(object)) | entity}
+                %endif
+                    </td>
                 </tr>
             %endfor
         %endfor

=== modified file 'l10n_it_vat_registries/templates/registro_iva_vendite.mako'
--- l10n_it_vat_registries/templates/registro_iva_vendite.mako	2013-08-21 16:15:06 +0000
+++ l10n_it_vat_registries/templates/registro_iva_vendite.mako	2014-02-08 09:10:47 +0000
@@ -28,9 +28,9 @@
             <th class="left_without_line">Numero fattura</th>
             <th class="left_without_line">Data fattura</th>
             <th class="left_without_line">Sezionale</th>
-            <th class="right_without_line">Totale fattura</th>
             <th class="right_without_line">Imposta</th>
             <th class="right_without_line">Importo</th>
+            <th class="right_without_line">Totale fattura</th>
             <th></th>
         </tr>
         </thead>
@@ -56,12 +56,12 @@
                     ${ counter | entity}
                 %endif
                 %if line['index']==0:
-                    </td><td class="left_with_line">
+                    </td><td class="left_with_line">${object.partner_id.name or ''| entity}
                 %else:
-                    </td><td class="left_without_line">
-                %endif
-                %if line['index']==0:
-                    ${object.partner_id.name or ''| entity}
+                    %if line['index']==1:
+                        </td><td class="left_without_line" nowrap>${object.partner_id.fiscalcode and 'C.F.: ' or ''| entity}${object.partner_id.fiscalcode or ''| entity}
+                        <br/>${object.partner_id.vat and 'P.IVA: ' or  ''| entity}${object.partner_id.vat or ''| entity}
+                    %endif
                 %endif
                 %if line['index']==0:
                     </td><td class="left_with_line">
@@ -87,14 +87,6 @@
                 %if line['index']==0:
                     ${object.journal_id.name or ''| entity}
                 %endif
-                %if line['index']==0:
-                    </td><td class="right_with_line">
-                %else:
-                    </td><td class="right_without_line">
-                %endif
-                %if line['index']==0:
-                    ${ formatLang(invoice_total(object)) | entity}
-                %endif
                 </td>
                 %if line['index']==0:
                     <td class="right_with_line">${ (line['tax_code_name'])  or ''| entity}</td>
@@ -106,6 +98,15 @@
                 %else:
                     <td class="right_without_line">${ formatLang(line['amount'])| entity}</td>
                 %endif
+                %if line['index']==0:
+                    <td class="right_with_line">
+                %else:
+                    <td class="right_without_line">
+                %endif
+                %if line['index']==0:
+                    ${ formatLang(invoice_total(object)) | entity}
+                %endif
+                    </td>
                 </tr>
             %endfor
         %endfor