search

andreausu / CodiceFiscale

A library to calculate the italian fiscal code (codice fiscale)
MIT License
23 stars 14 forks source link

Aggiunto controllo effettivo sulla validità del codice fiscale #1

Closed nigrosimone closed 10 years ago

nigrosimone commented 10 years ago

Ciao, ho trovato molto utile la tua classe per il calcolo del codice fiscale, tuttavia avendo bisogno anche di una per il controllo della validità ed essendo la sola espressione regolare troppo poco, ho ampliato la classe per il controllo per la verifica completa dei codici fiscali a 16 lettere... perdona il pessimo inglese nei commenti, magari se hai voglia e se ti interessa fare il merge, puoi sistemarli...

nigrosimone commented 10 years ago

ehmm i tuoi test case sono sbagliti i codici fiscali di test sono tutti errati... prova tu stesso https://telematici.agenziaentrate.gov.it/VerificaCF/VerificaCf.do SLLNDR91C06F205S : CODICE FISCALE NON VALIDO SXLNDQ67C48Z210L: CODICE FISCALE NON VALIDO

metre ti aspetti che siamo validi...

andreausu commented 10 years ago

Ciao! Prima di tutto grazie, il controllo sulla validità era molto lasco (e i test lo riflettevano). Due cose prima di poter accettare la PR:

1) La libreria segue lo standard PSR-2 per la scrittura del codice, potresti modificare la classe per renderla compatibile? Questo per evitare di avere parti di codice scritti con uno stile e altre in un altro. 2) Riusciresti ad aggiornare i test con dei case corretti?

Grazie mille!

nigrosimone commented 10 years ago

ok, ho aggiornato la classe per farla aderire allo standard PSR-2 e aggriornato i test

andreausu commented 10 years ago

Ciao, ho sistemato ancora qualcosina e successivamente mergiato in master, tra poco taggo anche il commit che così si può installare da composer. Grazie ancora!

nigrosimone commented 10 years ago

figurati è stato un piacere... ;)