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

Intermediario con codice fiscale di San Marino non rispetta il pattern FATTURAPA_IN_REGEX #922

Closed
3 of 4 tasks
sherpya opened this issue Jan 30, 2019 · 19 comments
Closed
3 of 4 tasks
Labels
needs review stale PR/Issue without recent activity, it'll be soon closed automatically. tracking

Comments

@sherpya
Copy link
Member

sherpya commented Jan 30, 2019

mi è arrivata una fattura di un intermediario di San Marino (è possibile?)

dovrebbe essere questo https://shop.passepartout.net/Acquisto/SceltaProdotto

il nomefile diventa SM03473_XXXXX.xml

ma il pattern FATTURAPA_IN_REGEX di aspetta 11 o 16 caratteri, qualcuno ha informazioni a riguardo?

@eLBati
Copy link
Member

eLBati commented Jan 30, 2019

@sherpya prova a scrivere in mailing list
https://github.com/OCA/l10n-italy#contribuire

@sherpya
Copy link
Member Author

sherpya commented Jan 30, 2019

a parte che secondo me questi hanno cannato perché l'identificativo è quello del mittente da usare e non l'intermediario, ma la specifica comunque dice di accettarlo

https://www.fatturapa.gov.it/export/fatturazione/it/c-11.htm

il Codice Paese va espresso secondo lo standard ISO 3166-1 alpha-2 code,
l’Identificativo univoco del Trasmittente, sia esso persona fisica o soggetto giuridico, è rappresentato dal suo identificativo fiscale (codice fiscale nel caso di soggetto trasmittente residente in Italia, identificativo proprio del Paese di appartenenza nel caso di soggetto trasmittente residente all’estero). La lunghezza di questo identificativo è di:

11 caratteri (minimo) e 16 caratteri (massimo) nel caso di codice paese IT,
2 caratteri (minimo) e 28 caratteri (massimo) altrimenti,

@sherpya
Copy link
Member Author

sherpya commented Feb 4, 2019

a quanto pare la specifica è IT + 11,16 (anche se dice da 11 a 16) e per gli altri due lettere più da 2 28, appena posso faccio una PR

il nome file è dato dal mittente, in questo caso l'intermediario, ma potrebbe arrivare una fattura da un'entità non IT

@sherpya
Copy link
Member Author

sherpya commented Feb 4, 2019

tipo una cosa del genere?

FATTURAPA_IN_REGEX = r'''^(IT[A-Z0-9]{11,16}|       # IT + PIVA (11 caratteri) o CF (16 caratteri)
                         [A-Z]{2}[A-Z0-9]{2,28})    # XX + da 2 a 28 caratteri
                         _[A-Z0-9]{,5}              # Progressivo univoco del file: max 5 caratteri alpha'''

re_pattach = re.compile(FATTURAPA_IN_REGEX + r'\.(xml|zip)', re.I | re.X)
re_resp = re.compile(FATTURAPA_IN_REGEX + r'_MT_[A-Z0-9]{,3}', re.I)

putroppo matcha anche IT e 2-28, ma volendo fare questo check da regex divento scemo

@eLBati
Copy link
Member

eLBati commented Feb 5, 2019

@sherpya puoi anche farlo in python il controllo aggiuntivo

@sherpya
Copy link
Member Author

sherpya commented Mar 7, 2019

me ne stanno arrivando altre, io farei {2,28} invece di {11,16} credo che possa bastare alla fine, a proposito perché la regex è compilata dentro la funzione e non fuori?

@eLBati
Copy link
Member

eLBati commented Mar 8, 2019

perché la regex è compilata dentro la funzione e non fuori?

Penso possa essere messa fuori

@sergiocorato
Copy link
Contributor

sergiocorato commented Mar 15, 2019

ho rilevato lo stesso problema , per il momento farei un regex con la specifica di San Marino

@sergiocorato
Copy link
Contributor

@sherpya @eLBati ho caricato la soluzione, ne ho ricevute anche da SE (forse Serbia?), ho anche dovuto ripristinare quelle ignorate finora 😢

@sherpya
Copy link
Member Author

sherpya commented Mar 18, 2019

@sergiocorato {2,28} ?

@sergiocorato
Copy link
Contributor

@sergiocorato {2,28} ?

@sherpya si per le non IT, l'avevi scritto sopra no?

@sergiocorato
Copy link
Contributor

perché la regex è compilata dentro la funzione e non fuori?

Penso possa essere messa fuori

ho spostato fuori la regex

@eLBati
Copy link
Member

eLBati commented Apr 11, 2019

@OCA/local-italy-developers
PR per v12: #1178
Qualcuno intende portare la correzione alla 11?

@labaggio
Copy link
Contributor

@OCA/local-italy-developers
PR per v12: #1178
Qualcuno intende portare la correzione alla 11?

posso farcela! parto da quella della 12.

@eLBati
Copy link
Member

eLBati commented Apr 11, 2019

@labaggio grazie.

PS: hai voglia di iscriverti alla mailing list sviluppo su https://www.odoo-italia.org/groups ?
Così magari aboliamo il gruppo @OCA/local-italy-developers che penso abbia i problemi che ho scritto qua https://www.odoo-italia.org/groups/sviluppo-17/sviluppo-1187

@labaggio
Copy link
Contributor

@labaggio grazie.

PS: hai voglia di iscriverti alla mailing list sviluppo su https://www.odoo-italia.org/groups ?
Così magari aboliamo il gruppo @OCA/local-italy-developers che penso abbia i problemi che ho scritto qua https://www.odoo-italia.org/groups/sviluppo-17/sviluppo-1187

Fatto, me l'ero persa..

@glauco70
Copy link

@OCA/local-italy-developers
PR per v12: #1178
Qualcuno intende portare la correzione alla 11?

posso farcela! parto da quella della 12.

Creata PR per la 11 : #1179

@eLBati
Copy link
Member

eLBati commented Apr 16, 2019

Attenzione: #1189

@github-actions
Copy link

github-actions bot commented Nov 7, 2021

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review stale PR/Issue without recent activity, it'll be soon closed automatically. tracking
Projects
None yet
Development

No branches or pull requests

6 participants