search

andreausu / CodiceFiscale

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

Elenco dei codici catastali #7

Closed nigrosimone closed 9 years ago

nigrosimone commented 9 years ago

Ciao Andrea, ho aggiunto una classe per recuperare i codici catastali a partire dal nome del comune e viceversa ed ho sistemato delle piccole cose per far funzionare il tutto anche con php 5.3 (ma credo che ora funzioni con tutte le versioni 5.x di php). Ho aggiunto anche il relativo test. Scusa se ho incasinato tutto facendo diverse path e pullrequest, ma credo avrai capito che non so usare bene GitHub. Comunuque la patch di cui tener conto è questa. Spero che il mio contributo ti faccia piacere.

andreausu commented 9 years ago

Ciao! Scusa se rispondo cosi' tardi, ma essendomi trasferito in un'altra nazione sono stato parecchio preso, appena ho un attimo di tempo guardo le PR, grazie :)

andreausu commented 9 years ago

Ho dato un'occhiata, grazie per il lavoro fatto. La retrocompatibilita' con PHP 5.3 non mi piace molto, la versione 5.3 e' in end of life e quindi voglio poter utilizzare le nuove feature messe a disposizione dalle versioni di PHP successive.

Potresti aprire un'altra PR che riguardi soltanto i codici catastali? Bisogna anche fixare alcune parti di codice perche' rispettino lo standard PSR-0 (da un'occhiata veloce ho notato in particolare i nomi dei metodi e delle variabili che devono iniziare con la minuscola e gli spazi in generale). Per il momento questa PR la chiudo.

Gazie mille, Andrea