snapshotpl / ZfSnapGeoip

Maxmind Geoip module for Zend Framework 2
15 stars 4 forks source link

Don't download database if already installed. #2

Closed sokac closed 10 years ago

sokac commented 10 years ago

Update database on update and on install on the first installation.

snapshotpl commented 10 years ago

@sokac You should change the existing controller instead of adding a new one. Maybe add a parameter that checks if a db file exists?

sokac commented 10 years ago

@snapshotpl good call. I just updated branch, please review.

snapshotpl commented 10 years ago

@sokac now looks perfect! Thanks! :)

snapshotpl commented 10 years ago

@sokac One think. This change break compatibility. I need to change this parameter to --noclobber (like in mv)

sokac commented 10 years ago

@snapshotpl thanks. Feel free to change param.

snapshotpl commented 10 years ago

@sokac I have started many changes and improvements. I will try to release them this weekend.