planetfederal / qgis-geogiglight-plugin

GNU General Public License v2.0
19 stars 9 forks source link

Unnecessary double dialog/warning #33

Closed gioman closed 8 years ago

gioman commented 8 years ago

if the Geogig server is up, but not publishing any repo then when adding a new connection to this server 2 separate dialogs pop up, "No repositories found at the specified server" and "No geogig server found at the specified url. See the logs for details.". In this specific case the second one does not seems appropriate.

gioman commented 8 years ago

Correction/addenda: this double warning nows also pops up also when adding a new connection pointing to a server that is actually publishing repo/s, even with data in it.

In this case after clicking twice on "ok" the user must refresh the connection list to see the connection tree being correctly populated.

This seems a regression caused by the fix about the the "add geogig server" dialog returning no message in case the URL points to a server not serving any repo.

See https://github.com/boundlessgeo/qgis-geogiglight-plugin/issues/24

gioman commented 8 years ago

Fix confirmed.