[processing] correctly handle inactive providers in the modeler (fix #18461)

alexbruy · alexbruy · commit 6a027e7a2e24 · 2018-03-19T11:56:51.000+02:00
(cherry picked from commit 092557b)
diff --git a/python/plugins/processing/modeler/ModelerDialog.py b/python/plugins/processing/modeler/ModelerDialog.py
@@ -695,12 +695,7 @@ def fillTreeUsingProviders(self):
             else:
                 providerItem = TreeProviderItem(provider, self.algorithmTree, self)
 
-                if not provider.isActive():
-                    providerItem.setHidden(True)
-                    self.disabledProviderItems[provider.id()] = providerItem
-
                 # insert non-native providers at end of tree, alphabetically
-
                 for i in range(self.algorithmTree.invisibleRootItem().childCount()):
                     child = self.algorithmTree.invisibleRootItem().child(i)
                     if isinstance(child, TreeProviderItem):
@@ -709,6 +704,10 @@ def fillTreeUsingProviders(self):
 
                 self.algorithmTree.insertTopLevelItem(i + 1, providerItem)
 
+                if not provider.isActive():
+                    providerItem.setHidden(True)
+                    self.disabledProviderItems[provider.id()] = providerItem
+
     def addAlgorithmsFromProvider(self, provider, parent):
         groups = {}
         count = 0
