QgsLegendInterface.addGroup() nests all groups

[anitagraser]

The following code adds four groups to the legend. Unfortunately they're all nested while the goal would be to put them into legend root:

i = qgis.utils.iface
l = i.legendInterface()
groupAnnotation = l.addGroup("Annotation",False)
groupDimensioning = l.addGroup("Dimensioning", True)
groupLocation = l.addGroup("Location", True)
groupTopo = l.addGroup("Topo", True)

Compare: ​http://forum.qgis.org/viewtopic.php?f=5&t=7583

[mbernasocchi]

I m dealing with the same problem in my multiview plugin (​http://hub.qgis.org/projects/multiview/repository/revisions/master/entry/temporalrasterloaderdialog.py) and i noticed that if i create a group and then load a layer the groups get created at the end of the list (look for the hack in the link). So maybe it is a more complex issue/bug.

[mbernasocchi]

Better fix, instead of forcing the element to be at the end, addGroup can receive a parent object. The new method addGroupToCurrentItem adds a new goup to the currentItem

[mbernasocchi]

trying to delete a wrong file

[mhugent]

Applied in r15561. Thanks!

[borysiasty]

I believe it can be closed now.

[mbernasocchi]

Fixing documentation string

[borysiasty]

applied in r15594

[mbernasocchi]

patch to add nesting on gui

[mbernasocchi]

the patch applied was an incomplete one... missunderstanding between Marcos :) here the patch to the patch

ciao

[mhugent]

Applied in r15766 with two small modifications: -added missing return statement in addGroupToCurrentItem() -removed debug msg

[mbernasocchi]

ups... :|

[mbernasocchi]

This patch creates the python binding to addGroup. For the moment it allows only one level of nesting like removeGroup

[mbernasocchi]

added python bindings. it supports only nesting into a top level element like removeGroup. we should think of a more flexible solution.

[mhugent]

Yes, all these group functions with int indices come frome the pre-multiple nesting time. It would probably be better to do a bit of interface redesign for both QgsLegend and QgsLegendInterface after 1.7 (maybe working with QTreeWidgetItems)

[mbernasocchi]

Replying to mhugent:

Yes, all these group functions with int indices come frome the pre-multiple nesting time. It would probably be better to do a bit of interface redesign for both QgsLegend and QgsLegendInterface after 1.7 (maybe working with QTreeWidgetItems)

True, once done with my master we can have a look at it. will you still apply the patch?

[mhugent]

Yes, applied in r15787