8.0 backporting fatturazione elettronica da v10 #615

jado95 · 2018-10-20T09:55:18Z

Backporting dei moduli: l10n_it_ipa, l10n_it_causali_pagamento; l10n_it_fiscal_payment_termn; l10n_it_withholding_tax_causali; l10n_it_fatturapa; l10n_it_fatturapa_out

sergiocorato · 2018-11-08T17:08:29Z

l10n_it_fatturapa_out/wizard/wizard_export_fatturapa.py

+                inv.preventive_checks()
+                self.with_context(context_partner).setFatturaElettronicaBody(
+                    inv, invoice_body)
+                fatturapa.FatturaElettronicaBody.append(invoice_body)
                # TODO DatiVeicoli



in caso la sequenza sia pluriennale, a me risulta serva passare il fiscalyear_id nel context, con una cosa simile:
invoices = invoice_obj.browse(invoice_ids) fy_id = invoices.mapped('period_id.fiscalyear_id.id') if len(fy_id) > 1: raise UserError(_('More than 1 fy included in the file!')) number = self.with_context( fiscalyear_id=fy_id[0] ).setProgressivoInvio(fatturapa)

chiudo questa segnalazione, come sulle altre versioni la sequenza non è prevista per essere pluriennale

sergiocorato · 2018-11-08T17:26:02Z

.travis.yml

@@ -31,7 +31,7 @@ install:
  - pip install unicodecsv
  - pip install codicefiscale
  - pip install unidecode==0.04.17
-  - pip install PyXB==1.2.4
+  - pip install PyXB==1.2.5


@eLBati per caso sai come mai il travis non prende la versione aggiornata di PyXB?

con il nuovo commit l'ha preso, ok

l10n_it_fatturapa/data/fatturapa_data.xml

sergiocorato · 2018-11-16T16:27:46Z

l10n_it_ipa/view/partner_view.xml

-                    <field name="ipa_code" placeholder="IPA123" attrs="{'invisible': [('is_company','=', False)]}"/>
+                    <field name="ipa_code" placeholder="IPA123" attrs="{'invisible': [('is_pa','=', False)]}"/>
+                </field>
+                <!--<field name="company_type" position="after">-->


Puoi cancellare questa riga?

sergiocorato · 2018-11-16T16:55:03Z

@jado95 programmi di fare il backport anche di l10n_it_fatturapa_in?

oca-clabot · 2018-11-20T09:16:49Z

Hey @jado95, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/cla
Here is a list of the users:

@SGiuseppe (login unknown in OCA database)

Appreciation of efforts,
OCA CLAbot

sergiocorato · 2018-11-21T17:06:48Z

l10n_it_fatturapa_out/models/account.py

+        'fatturapa.attachment.out', 'FatturaPA Export File',
+        readonly=True, copy=False)
+
+    def preventive_checks(self):


questa parte è stata bypassata

sergiocorato · 2018-11-21T17:23:09Z

l10n_it_fatturapa_out/wizard/wizard_export_fatturapa.py

-            DatiAnagrafici.Anagrafica = AnagraficaType(
+                IdPaese=partner.vat[0:2], IdCodice=partner.vat[2:])
+        # if partner.company_type == 'company':
+        if partner.is_company == True:


if partner.is_company:

ricordati di correggere qui

sergiocorato · 2018-11-21T17:23:29Z

l10n_it_fatturapa_out/wizard/wizard_export_fatturapa.py

                Denominazione=partner.name)
+        # elif partner.company_type == 'person':
+        elif partner.is_company == False:


sergiocorato · 2018-11-21T17:25:13Z

l10n_it_fatturapa_out/wizard/wizard_export_fatturapa.py

+            aliquota = line.invoice_line_tax_id[0].amount
+            AliquotaIVA = '%.2f' % (aliquota * 100)
+            line.ftpa_line_number = line_no
+            prezzo_unitario = self._get_prezzo_unitario(line)
            DettaglioLinea = DettaglioLineeType(
                NumeroLinea=str(line_no),
                Descrizione=line.name,


# can't insert newline with pyxb # see https://tinyurl.com/ycem923t # and '
' would not be correctly visualized anyway # (for example firefox replaces '
' with space Descrizione=line.name.replace('\n', ' '),

sergiocorato · 2018-11-21T17:30:46Z

è stato poi aggiunto sempre nell'esportazione la possibiiità di esportare più fatture da tree view

sergiocorato · 2018-11-21T18:01:17Z

l10n_it_fatturapa_pec/__openerp__.py

+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+{
+    'name': 'Italian Localization - FatturaPA - Emission - PEC Support',
+    'version': '10.0.1.0.0',


correggere la versione

jado95 · 2018-11-22T14:07:56Z

@sergiocorato Ho allineato la v8 con la v8 (tranne le traduzioni per ora)
Daresti un'occhiata adesso?

sergiocorato · 2018-11-22T14:30:07Z

@sergiocorato Ho allineato la v8 con la v8 (tranne le traduzioni per ora)
Daresti un'occhiata adesso?

@jado95 certo

sergiocorato · 2018-11-22T14:53:32Z

l10n_it_fatturapa_out/wizard/wizard_export_fatturapa.py

-        return True
+            raise UserError(
+                _('FatturaPA sequence not configured.'))
+        number = fatturapa_sequence.next_by_id(fatturapa_sequence.id)


fatturapa_sequence.id non serve (è già il record stesso)

In v8 serve passargli fatturapa_sequence altrimenti la funzione va in errore

hai ragione

sergiocorato · 2018-11-22T15:04:29Z

@jado95 manca questa modifica del metodo compute con _
poi da qui in avanti è stato modificato il send e i campi relativi

jado95 · 2018-11-26T16:23:11Z

@jado95 manca questa modifica del metodo compute con _
poi da qui in avanti è stato modificato il send e i campi relativi

@sergiocorato per quanto il riguardo modulo l10n_it_fatturapa_pec non l'ho seguito io e non ho fatto il backporting io. Per quanto riguarda i moduli fattura_pa e fatturapa_out secondo te sono ok?

sergiocorato

@jado95 manca questa modifica del metodo compute con _
poi da qui in avanti è stato modificato il send e i campi relativi

@sergiocorato per quanto il riguardo modulo l10n_it_fatturapa_pec non l'ho seguito io e non ho fatto il backporting io.
@jado95 ok

Per quanto riguarda i moduli fattura_pa e fatturapa_out secondo te sono ok?
@jado95 si, che dici di fare delle PR divise per gli altri? così magari mergiamo questa

sergiocorato · 2018-11-22T14:54:40Z

l10n_it_fatturapa_out/wizard/wizard_export_fatturapa.py

-            DatiAnagrafici.Anagrafica = AnagraficaType(
+                IdPaese=partner.vat[0:2], IdCodice=partner.vat[2:])
+        # if partner.company_type == 'company':
+        if partner.is_company == True:


ricordati di correggere qui

sergiocorato · 2018-11-26T16:31:30Z

@jado95 puoi sistemare il paio di note in sospeso sulla fatturapa out?

oca-clabot · 2018-11-26T17:10:06Z

Hey @jado95,
We acknowledge that the following users have signed our Contributor License Agreement:

@SGiuseppe

Appreciation of efforts,
OCA CLAbot

jado95 · 2018-11-26T17:11:44Z

@jado95 puoi sistemare il paio di note in sospeso sulla fatturapa out?

Sistemato le ultime due cose da sistemare.
Per quanto riguarda gli altri moduli (pec) li ho mergiati già in questo branch.

sergiocorato · 2018-11-27T08:04:24Z

@jado95 ho aggiornato il backport del modulo pec, lo trovi in una PR nel tuo repo

sergiocorato · 2018-11-27T10:56:30Z

@jado95 ho fatto la PR 5 sul tuo repo per correzioni minori

sergiocorato · 2018-11-27T17:18:40Z

@jado95 @eLBati per l'invio della pec, vedo che l'smtp dà errore se l'header non è quello della pec, sul mio sistema funziona perchè ho il modulo mail_outbound_static che era stato proposto ma non accettato v. OCA/social#195 , forse c'è una soluzione migliore?

eLBati · 2018-11-29T14:10:56Z

@sergiocorato io ho testato senza mail_outbound_static.
Quali sono i passi per riprodurre l'errore? Capita solo su 8.0?

primes2h · 2018-11-29T16:19:28Z

Ci sarebbe anche questa eventualmente da includere #677, penso basti un cherry-picking quando verrà fatto il merge.

sergiocorato · 2018-11-29T17:34:10Z

@sergiocorato io ho testato senza mail_outbound_static.
Quali sono i passi per riprodurre l'errore? Capita solo su 8.0?

@eLBati sì solo su 8.0, ma magari è qualcosa di specifico, provo con un'installazione basica

jado95 · 2018-12-03T14:36:41Z

@sergiocorato io ho testato senza mail_outbound_static.
Quali sono i passi per riprodurre l'errore? Capita solo su 8.0?

@eLBati sì solo su 8.0, ma magari è qualcosa di specifico, provo con un'installazione basica

@sergiocorato @eLBati
Ho aggiunto in un commit base64.decodestring che era stato aggiunto per le altre versioni

scigghia · 2018-12-03T16:08:25Z

👍

eLBati

Red travis

eLBati · 2018-12-03T16:10:32Z

@jado95 potresti fare squash dei commit secondo la loro logica? O li volevi introdurre tutti?

sergiocorato · 2018-12-03T16:16:47Z

@jado95 come segnala @eLBati ci sono alcune cose da correggere:
l10n_it_ipa/__openerp__.py:1: [C8201(no-utf8-coding-comment), ] No UTF-8 coding comment found: Use # coding: utf-8or# -- coding: utf-8 -- l10n_it_ipa/__openerp__.py:27: [C8103(manifest-deprecated-key), ] Deprecated key "active" in manifest file ************* Module l10n_it_fiscal_payment_term.__openerp__ l10n_it_fiscal_payment_term/__openerp__.py:7: [C8106(manifest-version-format), ] Wrong Version Format "8.0..0.0" in manifest file. Regex to match: "(8\.0)\.\d+\.\d+\.\d+$"

eLBati · 2018-12-03T16:18:15Z

l10n_it_fatturapa/i18n/es.po

-# Pedro M. Baeza <pedro.baeza@gmail.com>, 2015
-# Serge <serge@preply.com>, 2013,2015
-# Александра Маломуж <omalomug@gmail.com>, 2016
+# OCA Transbot <transbot@odoo-community.org>, 2017


Non bisognerebbe modificare i file .po direttamente, in quanto si potrebbero creare conflitti con weblate

eLBati · 2018-12-03T16:21:42Z

Mancherebbero alcuni commit presenti su 10.0 https://github.com/OCA/l10n-italy/commits/10.0

Ad esempio aeceda7

jado95 · 2018-12-04T14:17:25Z

Mancherebbero alcuni commit presenti su 10.0 https://github.com/OCA/l10n-italy/commits/10.0

Ad esempio aeceda7

Aggiunto il commit come aeceda7

Per quanto riguarda lo squash non so, tu dici di non includere i vari commit?

eLBati · 2018-12-06T08:27:59Z

Per quanto riguarda lo squash non so, tu dici di non includere i vari commit?

@jado95 Non dicevo di eliminare dei commit, ma di farne lo squash.
Ad esempio il commit 202174e ha senso tenerlo separato?

Vedi anche https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#git

eLBati · 2018-12-06T10:51:12Z

@jado95 controlla #697 per favore

[ADD] Aggiunto modulo l10n_it_causali_pagamento [ADD] Aggiunto modulo da v10 [ADD] Aggiunto modulo da v10 backportig da v10 a v8 moduli per fatturazione elettronica Update company.py Sistemazione delle proprietà in configurazione Fix refuso, c'era RF18 invece di RF19 Check mandatory customer fields before exporting XML Prevent to use sequence already used by journals Reformat code and change message in UserError in export fatturapa

ADD backport from 10.0 of l10n_it_fatturapa_pec

- Create an invoice with 1 line with quantity = 24 and price 1.2519 - Export XML In the XML you get Quantità: 24.00 Valore unitario: 1.25 Valore totale: 30.05 While you should have Valore unitario: 1.2519 [ADD] Generate a report for invoice and include in generated XML [IMP] l10n_it_fatturapa_out: Add flag to recognize which invoices have a PDF attached in the Fattura Elettronica XML [FIX] l10n_it_fatturapa_out: Remove confirm label [ADD] tracking attachment changes IMP attachments views IMP l10n_it_fatturapa_out export: allow multiple export , group by partner ALLOW to use newline char in invoice line description IMP l10n_it_fatturapa codice destinatario default 0000000

[8.0] fix sequence and is_company check [IMP] clean and update terms [FIX] parameter value [FIX] version

[8.0] Add base64.decodestring to attachment fatturapa [FIX] change FatturaPA into E-invoice FIX l10n_it_fatturapa: partner checks must be done for customers onlyas only needed for XML generation [FIX] fix error travis

jado95 · 2018-12-06T12:46:12Z

@eLBati tutti i check ok, merge?

sergiocorato · 2019-01-08T00:23:11Z

@eLBati mi dà l'impressione che i test in questa PR siano stati saltati di sana pianta vedi https://travis-ci.org/OCA/l10n-italy/jobs/464292977#L1574

eLBati · 2019-01-08T09:03:29Z

Ah, sì, anche nella build del branch 8.0 https://travis-ci.org/OCA/l10n-italy/jobs/475716999
Male

sergiocorato reviewed Nov 8, 2018

View reviewed changes

l10n_it_fatturapa/data/fatturapa_data.xml Outdated Show resolved Hide resolved

sergiocorato reviewed Nov 16, 2018

View reviewed changes

sergiocorato reviewed Nov 21, 2018

View reviewed changes

sergiocorato reviewed Nov 22, 2018

View reviewed changes

sergiocorato reviewed Nov 26, 2018

View reviewed changes

sergiocorato mentioned this pull request Nov 28, 2018

8.0 add vat ivp #679

Closed

sergiocorato mentioned this pull request Nov 30, 2018

8.0 add account vat statement ivp #686

Closed

sergiocorato approved these changes Dec 3, 2018

View reviewed changes

eLBati requested changes Dec 3, 2018

View reviewed changes

eLBati reviewed Dec 3, 2018

View reviewed changes

jado95 and others added 6 commits December 6, 2018 12:35

ADD backport from 10.0 of l10n_it_sdi_channel

6e1e80d

ADD backport from 10.0 of l10n_it_fatturapa_pec

[8.0] backport fatturapa pec

80ff82f

[8.0] fix sequence and is_company check [IMP] clean and update terms [FIX] parameter value [FIX] version

Update fatturapa_data.xml

cb0cbed

[8.0] Add base64.decodestring to attachment fatturapa [FIX] change FatturaPA into E-invoice FIX l10n_it_fatturapa: partner checks must be done for customers onlyas only needed for XML generation [FIX] fix error travis

FIX not allowed empty Causale

3e271ee

eLBati approved these changes Dec 6, 2018

View reviewed changes

OCA-git-bot added the approved label Dec 6, 2018

eLBati merged commit d1b0f5a into OCA:8.0 Dec 6, 2018

8.0 backporting fatturazione elettronica da v10 #615

8.0 backporting fatturazione elettronica da v10 #615

Conversation

jado95 commented Oct 20, 2018 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sergiocorato commented Nov 16, 2018

oca-clabot commented Nov 20, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sergiocorato commented Nov 21, 2018

Choose a reason for hiding this comment

jado95 commented Nov 22, 2018

sergiocorato commented Nov 22, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sergiocorato commented Nov 22, 2018

jado95 commented Nov 26, 2018

sergiocorato left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sergiocorato commented Nov 26, 2018

oca-clabot commented Nov 26, 2018

jado95 commented Nov 26, 2018

sergiocorato commented Nov 27, 2018

sergiocorato commented Nov 27, 2018

sergiocorato commented Nov 27, 2018

eLBati commented Nov 29, 2018

primes2h commented Nov 29, 2018 • edited

sergiocorato commented Nov 29, 2018

jado95 commented Dec 3, 2018 • edited

scigghia commented Dec 3, 2018

eLBati left a comment

Choose a reason for hiding this comment

eLBati commented Dec 3, 2018

sergiocorato commented Dec 3, 2018

Choose a reason for hiding this comment

eLBati commented Dec 3, 2018

jado95 commented Dec 4, 2018

eLBati commented Dec 6, 2018

eLBati commented Dec 6, 2018

jado95 commented Dec 6, 2018

sergiocorato commented Jan 8, 2019 • edited

eLBati commented Jan 8, 2019

jado95 commented Oct 20, 2018 •

edited

primes2h commented Nov 29, 2018 •

edited

jado95 commented Dec 3, 2018 •

edited

sergiocorato commented Jan 8, 2019 •

edited