Merge pull request #6408 from arnaud-morvan/processing_reload_fix

Processing reload fixes

Author: nyalldawson

python/plugins/processing/ProcessingPlugin.py

@@ -253,6 +253,7 @@ def unload(self):
         self.resultsDock.setVisible(False)
         self.iface.removeDockWidget(self.resultsDock)
 
+        self.toolbox.deleteLater()
         self.menu.deleteLater()
 
         # delete temporary output files

python/plugins/processing/core/Processing.py

@@ -85,11 +85,15 @@ def initialize():
         if "model" in [p.id() for p in QgsApplication.processingRegistry().providers()]:
             return
         # Add the basic providers
-        for c in QgsProcessingProvider.__subclasses__():
+        for c in [
+            QgisAlgorithmProvider,
+            Grass7AlgorithmProvider,
+            GdalAlgorithmProvider,
+            SagaAlgorithmProvider,
+            ScriptAlgorithmProvider,
+            ModelerAlgorithmProvider
+        ]:
             p = c()
-            if p.id() in ('native', '3d'):
-                # c++ providers are already registered
-                continue
             if QgsApplication.processingRegistry().addProvider(p):
                 Processing.BASIC_PROVIDERS.append(p)
         # And initialize

python/plugins/processing/modeler/ModelerAlgorithmProvider.py

@@ -63,7 +63,10 @@ def __init__(self):
 
         # must reload models if providers list is changed - previously unavailable algorithms
         # which models depend on may now be available
-        QgsApplication.processingRegistry().providerAdded.connect(self.refreshAlgorithms)
+        QgsApplication.processingRegistry().providerAdded.connect(self.onProviderAdded)
+
+    def onProviderAdded(self, provider_id):
+        self.refreshAlgorithms()
 
     def load(self):
         ProcessingConfig.settingIcons[self.name()] = self.icon()