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

alexbruy · alexbruy · commit 092557b16a01 · 2018-03-19T11:55:10.000+02:00
diff --git a/python/plugins/processing/modeler/ModelerDialog.py b/python/plugins/processing/modeler/ModelerDialog.py
@@ -700,12 +700,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):
@@ -714,6 +709,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
