sigdeletras / Spanish_Inspire_Catastral_Downloader

A QGIS plugin to download cadastral data from INSPIRE Services of Spanish Cadastral Cartography
GNU General Public License v3.0
23 stars 10 forks source link

Cambios varios #3

Closed All4Gis closed 6 years ago

All4Gis commented 6 years ago

Perdona por tantos cambios.

Changelog:

Todos los cambios han sido probados en python 2/3 y Win/Linux

sigdeletras commented 6 years ago

Hola

Como es lógico no tengo nada que objetar. Pero sí algunas preguntas, para ir enterándome:

Añadida barra de progreso

No tenía ni idea de cómo obtener el valor final para que realizara el progreso. Creo que (medio) entiendo cómo funciona la función reporthook y sus parámetros.

Correccion al descargar zip's donde el path contenia acentos,ñ etc

Aquí no sé muy bien dónde estaba el error. Veo que has creado la función EncodeUrl Pero ¿te ha dado error con algún municipio con ñ?

Añadido icono al dialogo y eliminado el icono de ?

self.dlg.setWindowIcon(QIcon (':/plugins/Spanish_Inspire_Catastral_Downloader/icon.png'));

La carpeta se borra si no se descargan los zips correctamete(Evitamos dejarlo al usuario carpetas incompletas)

Ok

Wait cursor mietras se esta realizando la descarga.

Con QApplication.setOverrideCursor(QCursor(Qt.WaitCursor)) ¿no?

Cambios en el ui, ahora se adapta al tamaño del dialogo.

Todo OK

Para terminar:

+#For Debug
+try:
+    import sys
+    from pydevd import *
+except:
+    None
All4Gis commented 6 years ago

Buenas

Correccion al descargar zip's donde el path contenia acentos,ñ etc

No podia descargarme ningún municipio que tuviese un ñ en su nombre,al hacer urlretrieve fallaba.

Wait cursor mietras se esta realizando la descarga. Si es con ese metodo.

self.dlg.setWindowFlags(Qt.WindowSystemMenuHint | Qt.WindowTitleHint)

Esto sirve para quitar el icono superior,el que esta al lado del aspa de cerrar el dialogo.No sirve para nada ya que nunca se le implemeta la ayuda. Puedes verlo en todos los dialogos de QGIS. Se quita con eso.

Igual con QApplication.restoreOverrideCursor()

Como se pone el wait,esto sirve para eliminarlo cuando termina la acción

En telegram alguien, quizás tú comentó algo de ¿un log de errores? ¿Es esto?¿Cómo funciona?

Lo comente yo.Eso es para debuggear el código.Puedo borrarlo y no subirlo(sería lo elegante)