sharpetronics/QGIS
1
0
Fork 0
mirror of https://github.com/qgis/QGIS.git synced 2025-12-16 00:06:09 -05:00
Code Issues Packages Projects Releases Wiki Activity

postprocessing: Use QgsLayerTreeRegistryBridge to add layers

Browse Source 
When a profile tool is already opened, the output of a procssing is
not added to its layer tree. This is because it relies on a
`QgsLayerTreeRegistryBridge`. Indeed, `QgsLayerTreeRegistryBridge`
listens to the `QgsProject::legendLayersAdded()` signal in order to
update the elevation profile tree view. However this signal is not
triggered by the current logic in `Postprocessing.py`because
`QgsProject::addMaplayer` is called with `addToLegend` set to
False. Then, the layer is added to the tree by calling
`QgsLayerTreeGroup::insertChildNode`.

This issue is fixed by creating a
`QgsLayerTreeRegistryBridge::InsertionPoint` to set the insertion
point and then calling `QgsProject::addMaplayer` with `addToLegend`
set to True.
This commit is contained in:
Jean Felder 2025-02-04 15:49:28 +01:00 committed by Nyall Dawson
parent bd4eb2f2cf
commit a46f3e9752
1 changed files with 26 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
python/plugins/processing/gui/Postprocessing.py
View File

@ -35,6 +35,7 @@ from qgis.core import (
QgsLayerTreeLayer, QgsLayerTreeLayer,
QgsLayerTreeGroup, QgsLayerTreeGroup,
QgsLayerTreeNode, QgsLayerTreeNode,
QgsLayerTreeRegistryBridge,
) )
from qgis.utils import iface from qgis.utils import iface
@ -237,8 +238,6 @@ def handleAlgorithmResults(
# output layer already exists in the destination project # output layer already exists in the destination project
owned_map_layer = context.temporaryLayerStore().takeMapLayer(layer) owned_map_layer = context.temporaryLayerStore().takeMapLayer(layer)
if owned_map_layer: if owned_map_layer:
details.project.addMapLayer(owned_map_layer, False)
# we don't add the layer to the tree yet -- that's done # we don't add the layer to the tree yet -- that's done
# later, after we've sorted all added layers # later, after we've sorted all added layers
layer_tree_layer = create_layer_tree_layer(owned_map_layer, details) layer_tree_layer = create_layer_tree_layer(owned_map_layer, details)
@ -277,10 +276,15 @@ def handleAlgorithmResults(
current_selected_node = iface.layerTreeView().currentNode() current_selected_node = iface.layerTreeView().currentNode()
iface.layerTreeView().setUpdatesEnabled(False) iface.layerTreeView().setUpdatesEnabled(False)
# store the current intersection point to restore it later
previous_insertion_point = (
details.project.layerTreeRegistryBridge().layerInsertionPoint()
)
for group, layer_node in sorted_layer_tree_layers: for group, layer_node in sorted_layer_tree_layers:
layer_node.removeCustomProperty(SORT_ORDER_CUSTOM_PROPERTY) layer_node.removeCustomProperty(SORT_ORDER_CUSTOM_PROPERTY)
insertion_point: Optional[QgsLayerTreeRegistryBridge.InsertionPoint] = None
if group is not None: if group is not None:
group.insertChildNode(0, layer_node) insertion_point = QgsLayerTreeRegistryBridge.InsertionPoint(group, 0)
else: else:
# no destination group for this layer, so should be placed # no destination group for this layer, so should be placed
# above the current layer # above the current layer
@ -289,16 +293,32 @@ def handleAlgorithmResults(
current_node_index = current_node_group.children().index( current_node_index = current_node_group.children().index(
current_selected_node current_selected_node
) )
current_node_group.insertChildNode(current_node_index, layer_node) insertion_point = QgsLayerTreeRegistryBridge.InsertionPoint(
current_node_group, current_node_index
)
elif isinstance(current_selected_node, QgsLayerTreeGroup): elif isinstance(current_selected_node, QgsLayerTreeGroup):
current_selected_node.insertChildNode(0, layer_node) insertion_point = QgsLayerTreeRegistryBridge.InsertionPoint(
current_selected_node, 0
)
elif context.project(): elif context.project():
context.project().layerTreeRoot().insertChildNode(0, layer_node) insertion_point = QgsLayerTreeRegistryBridge.InsertionPoint()
if insertion_point:
details.project.layerTreeRegistryBridge().setLayerInsertionPoint(
insertion_point
)
details.project.addMapLayer(layer_node.layer())
if not have_set_active_layer and iface is not None: if not have_set_active_layer and iface is not None:
iface.setActiveLayer(layer_node.layer()) iface.setActiveLayer(layer_node.layer())
have_set_active_layer = True have_set_active_layer = True
# reset to the previous insertion point
details.project.layerTreeRegistryBridge().setLayerInsertionPoint(
previous_insertion_point
)
# all layers have been added to the layer tree, so safe to call # all layers have been added to the layer tree, so safe to call
# postProcessors now # postProcessors now
for layer, details in layers_to_post_process: for layer, details in layers_to_post_process: