go2streetview icon indicating copy to clipboard operation
go2streetview copied to clipboard

QGIS 3.4 - Extension disabled because it caused QGIS to crash

Open Fabien98 opened this issue 6 years ago • 4 comments

Hello, I just freshly installed the new QGIS 3.4 but I encountered an error with this plugin.

When I activate the plugin I have a Window pop-up error which states:

This Windows version (6.3.9600) does not support the required Bluetooth API. Consider updating to a more recent Windows (10.0.10586 or above)

It seems that this issue is related to a pyQT5 version.. (see related issue here)

Fabien98 avatar Oct 30 '18 09:10 Fabien98

I didn't do extensive testing, but simply deleting Qt from the import list of PyQt5 seems to fix the problem.

I don't think go2streetview should need the Bluetooth API. I better fix is expected from the author, or from the PyQt5 team, though.

Ciao.

zezzagio avatar Dec 21 '18 16:12 zezzagio

I created a bug report on the QGIS bugtracker: https://issues.qgis.org/issues/20867

Fabien98 avatar Dec 21 '18 17:12 Fabien98

A plugin should not be able to block the application, for any reason; this is definitely an application bug, in my opinion; in principle, at least

The plugin, though, should get rid of unwanted, and unneeded dependencies. At least, it can, when they are creating problem like in this case.

Instead of the generic "import Qt", which import all (and thus Blutooth, of which I dind't find any reference in the rest of the code) I suggest using more specific imports; all the more because they are already here: it says "import Qt", and then it goes ahead and says "and I mean: QtCore, QtWidgets, QtGui, QtWebKit..." At this point, it seems just natural to me to delete the generic import, and stay with and only with what is needed.

Ciao.

zezzagio avatar Dec 21 '18 20:12 zezzagio

QGIS rejected my bug report and made the same conclusion that @zezzagio . Thanks for your input by the way.

Status changed from Open to Rejected Resolution set to up/downstream

Please file bugreport in the plugin bugtracker, as QGIS itself does not requires Bluetooth. Seems that in the plugin code there is a incorrect import.

Fabien98 avatar Jan 03 '19 23:01 Fabien98