Push error message to messagebar, stop adding a new layer if an error occured

JuhoErvasti · JuhoErvasti · commit 29b39bacc099 · 2023-12-15T10:41:20.000+02:00
diff --git a/QGISDigitransitGeocoding/build.py b/QGISDigitransitGeocoding/build.py
@@ -18,7 +18,7 @@
     if "test/" not in fil and "test\\" not in fil
 ]
 locales = ["fi"]
-profile = "digi"
+profile = "digitransit"
 ui_files = list(glob.glob("**/*.ui", recursive=True))
 resources = list(glob.glob("**/*.qrc", recursive=True))
 extra_dirs = ["resources"]
diff --git a/QGISDigitransitGeocoding/ui/digitransit_geocoder_dockwidget.py b/QGISDigitransitGeocoding/ui/digitransit_geocoder_dockwidget.py
@@ -304,12 +304,14 @@ def handle_search_response(self, reply):
                 reply.errorString(), "QGISDigitransitGeocoding", Qgis.Warning
             )
         else:
+            self.iface.messageBar().pushMessage("Error", "An error occured. Check the log for details.", level=1, duration=10)
             QgsMessageLog.logMessage(
                 str(error), "QGISDigitransitGeocoding", Qgis.Warning
             )
             QgsMessageLog.logMessage(
                 reply.errorString(), "QGISDigitransitGeocoding", Qgis.Warning
             )
+            return
         
         try:
             if len(self.features) > 0 and self.radiobuttonresultsshowall.isChecked():

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@`
`18`	`18`	`if "test/" not in fil and "test\\" not in fil`
`19`	`19`	`]`
`20`	`20`	`locales = ["fi"]`
`21`		`-profile = "digi"`
	`21`	`+profile = "digitransit"`
`22`	`22`	`ui_files = list(glob.glob("*/.ui", recursive=True))`
`23`	`23`	`resources = list(glob.glob("*/.qrc", recursive=True))`
`24`	`24`	`extra_dirs = ["resources"]`
Original file line number	Diff line number	Diff line change
`@@ -304,12 +304,14 @@ def handle_search_response(self, reply):`
`304`	`304`	`reply.errorString(), "QGISDigitransitGeocoding", Qgis.Warning`
`305`	`305`	`)`
`306`	`306`	`else:`
	`307`	`+ self.iface.messageBar().pushMessage("Error", "An error occured. Check the log for details.", level=1, duration=10)`
`307`	`308`	`QgsMessageLog.logMessage(`
`308`	`309`	`str(error), "QGISDigitransitGeocoding", Qgis.Warning`
`309`	`310`	`)`
`310`	`311`	`QgsMessageLog.logMessage(`
`311`	`312`	`reply.errorString(), "QGISDigitransitGeocoding", Qgis.Warning`
`312`	`313`	`)`
	`314`	`+ return`
`313`	`315`
`314`	`316`	`try:`
`315`	`317`	`if len(self.features) > 0 and self.radiobuttonresultsshowall.isChecked():`