Fix #4439

wonder-sk · wonder-sk · commit ed20ff368723 · 2011-10-31T09:51:21.000-03:00
diff --git a/src/app/legend/qgslegend.cpp b/src/app/legend/qgslegend.cpp
@@ -1752,7 +1752,12 @@ void QgsLegend::refreshLayerSymbology( QString key, bool expandItem )
   }
 
   //store the current item
-  QModelIndex currentItemIndex( currentIndex() );
+  QTreeWidgetItem* current = currentItem();
+  // in case the current item is a child of the layer, use the layer as current item
+  // because otherwise we would set an invalid item as current item
+  // (in refreshSymbology the symbology items are removed and new ones are added)
+  if ( current->parent() == theLegendLayer )
+    current = current->parent();
 
   double widthScale = 1.0;
   if ( mMapCanvas && mMapCanvas->map() )
@@ -1763,7 +1768,7 @@ void QgsLegend::refreshLayerSymbology( QString key, bool expandItem )
   theLegendLayer->refreshSymbology( key, widthScale );
 
   //restore the current item again
-  setCurrentIndex( currentItemIndex );
+  setCurrentItem( current );
   adjustIconSize();
   setItemExpanded( theLegendLayer, expandItem );//make sure the symbology items are visible
 }
