[processing] make items hidden after adding them to the view (fix #17881

)
qgis · Jan 23, 2018 · d7e1813 · d7e1813
1 parent aaf9793
commit d7e1813
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/python/plugins/processing/gui/ProcessingToolbox.py b/python/plugins/processing/gui/ProcessingToolbox.py
@@ -342,16 +342,18 @@ def addRecentAlgorithms(self, updating):
     def addProvider(self, provider_id):
         provider = QgsApplication.processingRegistry().providerById(provider_id)
         providerItem = TreeProviderItem(provider, self.algorithmTree, self)
-        if not provider.isActive():
-            providerItem.setHidden(True)
-            self.disabledProviderItems[provider.id()] = providerItem
 
         for i in range(self.algorithmTree.invisibleRootItem().childCount()):
             child = self.algorithmTree.invisibleRootItem().child(i)
             if isinstance(child, TreeProviderItem):
                 if child.text(0) > providerItem.text(0):
                     break
+
         self.algorithmTree.insertTopLevelItem(i, providerItem)
+        if not provider.isActive():
+            providerItem.setHidden(True)
+            self.disabledProviderItems[provider.id()] = providerItem
+
         provider.algorithmsLoaded.connect(self.updateProvider)
 
     def fillTreeUsingProviders(self):
@@ -375,19 +377,17 @@ 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):
                         if child.text(0) > providerItem.text(0):
                             break
 
                 self.algorithmTree.insertTopLevelItem(i + 1, providerItem)
+                if not provider.isActive():
+                    providerItem.setHidden(True)
+                    self.disabledProviderItems[provider.id()] = providerItem
 
     def addAlgorithmsFromProvider(self, provider, parent):
         groups = {}