search

TUFLOW-Support / QGIS-TUFLOW-Plugin

TUFLOW related QGIS Plugins
9 stars 7 forks source link

Fix for currentLayer.type() #18

Closed ZedeN1 closed 4 years ago

ZedeN1 commented 4 years ago

For some of our Linux machines in the office, TUFLOW plugin doesn't work for time series results. The GIS layers are added, the plugin knows about the times, but it doesn't let you select the Result Type → Time Series → Level or Flow or any other (they are greyed out).

I am unable to find the correlation between QGIS version/ plugin version and this bug. Bug only happens on Linux Mint though... Kubuntu and Ubuntu Mate are OK.

The following fixes the issues:

On line 118 in: _QGIS-TUFLOW-Plugin/tuflow/tuflowqgis_tuviewer/tuflowqgistuview.py

Instead of: if self.currentLayer.type() == 0: # vector layer it should be if self.currentLayer.type() == QgsMapLayerType.VectorLayer: # vector layer

TUFLOW-Support commented 4 years ago

Hi, thanks for finding this. I thought I had replaced all instances where an integer had been used instead of an explicit enumerator - obviously not since I scanned the code and found a few more. Hopefully I've now caught all of them.