Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

l10n_it_fatturapa NumeroCivico with empty space: SimpleFacetValueError #1220

Closed
1 of 4 tasks
eLBati opened this issue May 6, 2019 · 14 comments
Closed
1 of 4 tasks

l10n_it_fatturapa NumeroCivico with empty space: SimpleFacetValueError #1220

eLBati opened this issue May 6, 2019 · 14 comments
Labels
11.0 needs fixing Has conflicts or is failing mandatory CI checks tracking

Comments

@eLBati
Copy link
Member

eLBati commented May 6, 2019

Versioni coinvolte:

Passi per riprodurre:

Importare un XML con <NumeroCivico> </NumeroCivico>

Comportamento osservato:

SimpleFacetValueError: Type {http://ivaservizi.agenziaentrate.gov.it/docs/xsd/fatture/v1.2}NumeroCivicoType pattern constraint violated by value

Comportamento atteso:

L'XML viene importato

@sherpya
Copy link
Member

sherpya commented May 6, 2019

ma esattamente quando arriva un civico con uno spazio e basta che si fa? non è che sarebbe meglio eliminare l'entry come sto provando a fare come con #1218? riduciamo anche le patch all'xsd

@eLBati
Copy link
Member Author

eLBati commented May 6, 2019

ma esattamente quando arriva un civico con uno spazio e basta che si fa?

In odoo viene lasciato il campo vuoto

non è che sarebbe meglio eliminare l'entry come sto provando a fare come con #1218? riduciamo anche le patch all'xsd

Quindi anche per i vari StringNType?
Per me OK. Vuoi provare ad aggiungerlo a #1218 ?

@sherpya
Copy link
Member

sherpya commented May 6, 2019

per quelli con minOccurs="0" va bene, se uno obbligatorio è uno spazio che succede? teoricamente l'ade dovrebbe rimbalzarli, se non è opzionale suppongo che debba essere valorizzato

@eLBati
Copy link
Member Author

eLBati commented May 6, 2019

teoricamente l'ade dovrebbe rimbalzarli

riesci a verificare questo?

@sherpya
Copy link
Member

sherpya commented May 6, 2019

teoricamente l'ade dovrebbe rimbalzarli

riesci a verificare questo?

devo chiedere a @letyf1988 io di solito non traffico con l'ADE :D, credo che intanto i minOccurs 0 possono essere eliminati

@sherpya
Copy link
Member

sherpya commented May 6, 2019

ma non c'è un xsd ufficiale? ho appena trovato questo https://github.com/italia/fatturapa-testsdi/blob/master/core/schemas/Schema_del_file_xml_FatturaPA_versione_1.2_cleanup.xsd

rispetto a quello che ho io in più c'è:

      <xs:enumeration value="RF03">
        <xs:annotation>
          <xs:documentation>Regime delle nuove iniziative produttive (art. 13, L. 388/2000)</xs:documentation>
        </xs:annotation>
      </xs:enumeration>

;(

@sherpya
Copy link
Member

sherpya commented May 6, 2019

pare sia stato rimosso nella 1.2.1

@eLBati
Copy link
Member Author

eLBati commented May 7, 2019

@sherpya
Copy link
Member

sherpya commented May 7, 2019

le string obbligatorie sono tante, molte non hanno proprio senso di essere vuote, ma per sempio Indirizzo dentro IndirizzoType potrebbe aver senso provarlo, o meglio ancora Descrizione in DettaglioLineeType, prova a mettere uno spazio vuoto e vedere se se l'accolla.

per quanto riguarda l'xsd sembra l'ultimo ufficiale sia 1.2.1, ma l'autofattura sta in un fantomatico 1.3 (introvabile)

@sherpya
Copy link
Member

sherpya commented May 7, 2019

se ci metto _ o qualche altro testo quando è vuoto?

@eLBati
Copy link
Member Author

eLBati commented May 7, 2019

@sherpya un file con <Descrizione> </Descrizione> non viene scartato da SDI...

se ci metto _ o qualche altro testo quando è vuoto?

Penso possa andare bene (mi piace di più - 🤓 )

@SimoRubi
Copy link
Member

SimoRubi commented Aug 6, 2019

In 12.0 questo dovrebbe essere risolto, c'è anche un test:

@eLBati
Copy link
Member Author

eLBati commented Aug 7, 2019

Sì, era già stato risolto con XML doctor (#1172) e poi successivamente anche refactored con #1312
Penso che questa si possa chiudere

@primes2h primes2h added 11.0 needs fixing Has conflicts or is failing mandatory CI checks tracking and removed help wanted labels Feb 11, 2020
@primes2h
Copy link
Contributor

Chiudo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
11.0 needs fixing Has conflicts or is failing mandatory CI checks tracking
Projects
None yet
Development

No branches or pull requests

4 participants