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

ItaxCode.valid? ritorna false per omocodia cittadini estere #29

Closed
baraidrissa opened this issue Sep 1, 2023 · 4 comments · Fixed by #30
Closed

ItaxCode.valid? ritorna false per omocodia cittadini estere #29

baraidrissa opened this issue Sep 1, 2023 · 4 comments · Fixed by #30
Assignees
Labels
bug Something isn't working

Comments

@baraidrissa
Copy link
Contributor

Ciao @matteoredz , c'è un piccolo problema con i codici fiscale omocodes.

Io ho un codice fiscale reale di un cittadino indiano che finisce con Z2N2Y, però ItaxCode.valid? su questo codice fiscale mi ritorna sempre false.

Ho visto che il metodo ItaxCode::Validator.new(tax_code, data = {}).valid?
controlla che encoded_tax_code sia uguale al tax_code fornito.
Ovviamente encoded_tax_code ritorna un valore che finisce Con codice catastale in questo "Z222N".

Si potrebbe cercare di migliorare questo controllo, altrimenti il programma continuerà a dare false anche per molti altri codici fiscali reali.

Spiego di essermi spiegato.

Grazie mille!

@matteoredz
Copy link
Owner

Ciao @baraidrissa

Initialmente il controllo era fatto in questo modo.

Devo fare un po' di test per vedere se reintrodurre questo comportamento sia sufficiente per risolvere il problema senza introdurne altri.

@matteoredz
Copy link
Owner

@baraidrissa ho analizzato la questione, anche confrontando ItaxCode con altre implementazioni, e sono giunto alla conclusione che la validazione del codice fiscale possa (e debba) essere notevolmente semplificata. Ho indicato le motivazioni nella seguente PR #30.

Data l'entità della modifica, la gemma subirà un bump alla prossima major (attualmente v2.0.0).

@matteoredz matteoredz self-assigned this Sep 30, 2023
@matteoredz matteoredz added the bug Something isn't working label Sep 30, 2023
@matteoredz
Copy link
Owner

La release v2.0.0 è stata appena rilasciata! Grazie per il tuo contributo 😄

@baraidrissa
Copy link
Contributor Author

Ciao @matteoredz,
grazie mille a te del gentile riscontro!

Buon inizio settimana

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants