havatv / qgismultidistancebufferplugin

QGIS Plugin for Multi-distance buffering
GNU General Public License v2.0
4 stars 1 forks source link

Plugin saves a ~/D:\multibuffer #1

Closed pcav closed 8 years ago

pcav commented 8 years ago

This seems inappropriate, particularly for non-Windows systems

havatv commented 8 years ago

Thank you for reporting. May be I don't understand the question, but saving of temporary files is done to a temporary directory found using Python't tempfile module, and I thought that this should be portable. I don't have any problems on Windows or Ubuntu. I don't have access to MacOS.

pcav commented 8 years ago

The appropriate dir would be /temp. In your case, it goes to ~/D:\multibuffer

havatv commented 8 years ago

On which platform are you? Which dataset is it that goes to ~/D:\multibuffer?

pcav commented 8 years ago

Debian sid Dir empty, apparently is created at plugin import

havatv commented 8 years ago

OK. I will inspect the code to see what the reason for this can be. Thanks again for reporting.

havatv commented 8 years ago

I am not able to find any trace of "multibuffer" or "D:" in the plugin files. I tried to uninstall and install the plugin with QGIS 2.8.2 on Ubuntu 14.04, but after import I found nothing that looked like ~/D:\multibuffer. Nothing happens when I start QGIS and the plugin. So this is very strange. Can you try to reproduce it?

pcav commented 8 years ago

Just tested:

havatv commented 8 years ago

Very strange, indeed. Could there be other plugins that are causing this? What happens if you first uninstall the Multi-distance buffer plugin, then remove the strange directory (~/D:\multibuffer) and finally restart QGIS?

pcav commented 8 years ago

Oho, you seem to be right! Uninstalling the plugin, rm the dir, restarting qgis recreates the dir again. So it must be something else, unclear what. Sorry about the noise, but the name really pushed be on the wrong track.

havatv commented 8 years ago

Thanks for clarifying! And good to hear that it was a false alarm this time.

havatv commented 8 years ago

The previous commit was supposed to reference issue #2!