Marco Colombo

> Puoi modificare il messaggio del commit inglese? > Il co-authored poi non è corretto perché non riconosce il mio utente, Fatto e fatto. Per la cronaca, devi lasciare una...

@SimoRubi > Se stiamo controllando i dati della fattura in modo che poi la generazione del XML non esploda, allora andrebbe controllato che il codice del fiscal_document_type_id rientri tra quelli...

> > > Vedo che ci sono anche altre modifiche rispetto a #2545, idealmente sarebbero da fare in una PR separata ma può essere accettabile averle anche solo in un...

> > La preventive_check() non deve rifare tutti i controlli dello schema, ma magari beccare i casi più comuni che possono prendere di sorpresa l'utente e suggerigli cosa fare. >...

Aggiungo: ``` import xmlschema _xml_schema_1_2_1 = "Schema_del_file_xml_FatturaPA_versione_1.2.1.xsd" _old_xsd_specs = "xmldsig-core-schema.xsd" validator = xmlschema.XMLSchema( _xml_schema_1_2_1, locations={"http://www.w3.org/2000/09/xmldsig#": _old_xsd_specs}, ) print(validator.types['TipoDocumentoType'].enumeration) ``` ``` ['TD01', 'TD02', 'TD03', 'TD04', 'TD05', 'TD06', 'TD16', 'TD17', 'TD18', 'TD19',...

> Secondo me l'estrazione è da fare in `l10n_it_fatturapa_out` perché i valori estratti dal XML servono solo al controllo in `preventive_checks`, non modificherei `l10n_it_fiscal_document_type`. Ma secondo te ha senso avere...

Nota a latere. L'xsd prevede delle annotation con documentation per i vari valori di un enumeration. Peccato che xmlschema scarti quell'informazioni, per cui non possiamo estrarre (banalmente, per carità, possiamo...

Contro nota. Ho controllato e l'autore dei xmlschema ha aggiunto un modo per accederci: https://github.com/sissaschool/xmlschema/issues/255 però disponibile dalla v1.7 Update: con un po' di fantasia e di immaginazione: ``` import...

> Comunque farei il tutto in un'altra PR di miglioramento per riuscire portare a merge questa FIX. Ho creato #2654

La PR non è cambiata, semplice rimozione di un conflitto dovuto ad un merge recente.