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

Fix for #6658, update duplicateLayers() to work around odd layer group segfault bug

Browse Source 
- Move QgsLegend::findLegendLayer() methods to public
This commit is contained in:
Larry Shaffer 2012-11-08 09:09:15 -07:00
parent 474bd2400f
commit 81f99d6658
2 changed files with 35 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/app/legend/qgslegend.h
View File

@ -206,8 +206,13 @@ class QgsLegend : public QTreeWidget
/** return canvas */
QgsMapCanvas *canvas() { return mMapCanvas; }
public slots:
/**Returns the legend layer to which a map layer belongs to*/
QgsLegendLayer* findLegendLayer( const QString& layerKey );
/**Returns the legend layer to which a map layer belongs to*/
QgsLegendLayer* findLegendLayer( const QgsMapLayer *layer );
public slots:
/*!Adds a new layer group with the maplayers to the canvas*/
void addLayers( QList<QgsMapLayer *> );
@ -372,12 +377,6 @@ class QgsLegend : public QTreeWidget
void mouseReleaseEvent( QMouseEvent * e );
void mouseDoubleClickEvent( QMouseEvent* e );
/**Returns the legend layer to which a map layer belongs to*/
QgsLegendLayer* findLegendLayer( const QString& layerKey );
/**Returns the legend layer to which a map layer belongs to*/
QgsLegendLayer* findLegendLayer( const QgsMapLayer *layer );
/**Checks mPixmapWidthValues and mPixmapHeightValues and sets a new icon size if necessary*/
void adjustIconSize();

53
src/app/qgisapp.cpp
View File

@ -5372,7 +5372,7 @@ void QgisApp::duplicateLayers( QList<QgsMapLayer *> lyrList )
return;
}
QList<QgsMapLayer *> selectedLyrs = lyrList.empty() ? mMapLegend->selectedLayers() : lyrList;
const QList<QgsMapLayer *> selectedLyrs = lyrList.empty() ? mMapLegend->selectedLayers() : lyrList;
if ( selectedLyrs.empty() )
{
return;
@ -5389,16 +5389,6 @@ void QgisApp::duplicateLayers( QList<QgsMapLayer *> lyrList )
unSppType = QString( "" );
layerDupName = selectedLyr->name() + " " + tr( "copy" );
// setup for placing duplicated layer below source layer, regardless of group depth
mMapLegend->blockSignals( true );
if ( !mMapLegend->setCurrentLayer( selectedLyr ) )
{
mMapLegend->blockSignals( false );
continue; // legend item doesn't exist for map layer
}
mMapLegend->blockSignals( false );
QgsLegendLayer *sourcellayer = mMapLegend->currentLegendLayer();
if ( selectedLyr->type() == QgsMapLayer::PluginLayer )
{
unSppType = tr( "Plugin layer" );
@ -5409,7 +5399,7 @@ void QgisApp::duplicateLayers( QList<QgsMapLayer *> lyrList )
if ( unSppType.isEmpty() )
{
QgsVectorLayer *vlayer = qobject_cast<QgsVectorLayer*>( selectedLyr );
// TODO: check for other layer types that won't duplicate correctly
// TODO: add other layer types that can be duplicated
// currently memory and plugin layers are skipped
if ( vlayer && vlayer->storageType() == "Memory storage" )
{
@ -5421,7 +5411,6 @@ void QgisApp::duplicateLayers( QList<QgsMapLayer *> lyrList )
}
}
if ( unSppType.isEmpty() && !dupLayer )
{
QgsRasterLayer *rlayer = qobject_cast<QgsRasterLayer*>( selectedLyr );
@ -5453,23 +5442,39 @@ void QgisApp::duplicateLayers( QList<QgsMapLayer *> lyrList )
continue;
}
// add layer to project and layer registry
// add layer to layer registry and legend
QList<QgsMapLayer *> myList;
myList << dupLayer;
QgsMapLayerRegistry::instance()->addMapLayers( myList );
// duplicate the layer style
copyStyle( selectedLyr );
pasteStyle( dupLayer );
// verify layer has been added to legend
QgsLegendLayer *duplLayer = 0;
duplLayer = mMapLegend->findLegendLayer( dupLayer );
if ( !duplLayer )
{
// some source layers, like items > 4th in a container, have their layer
// registered but do not show up in the legend, so manually add them
QgsLegendLayer* llayer = new QgsLegendLayer( dupLayer );
mMapLegend->insertTopLevelItem( 0, llayer );
// double-check, or move of non-existent legend layer will segfault
duplLayer = mMapLegend->findLegendLayer( dupLayer );
}
// move layer to just below source layer
QgsLegendLayer *dupllayer = mMapLegend->currentLegendLayer();
mMapLegend->moveItem( dupllayer, sourcellayer );
QgsLegendLayer *srclLayer = mMapLegend->findLegendLayer( selectedLyr );
if ( duplLayer && srclLayer )
{
// move layer to just below source layer
mMapLegend->moveItem( duplLayer, srclLayer );
// always set duplicated layers to not visible
// so layer can be configured before being turned on,
// and no map canvas refresh needed when doing multiple duplications
mMapLegend->setLayerVisible( dupLayer, false );
// duplicate the layer style
copyStyle( selectedLyr );
pasteStyle( dupLayer );
// always set duplicated layers to not visible
// so layer can be configured before being turned on,
// and no map canvas refresh needed when doing multiple duplications
mMapLegend->setLayerVisible( dupLayer, false );
}
}
dupLayer = 0;